Skip to content

Conversation

@mgorny
Copy link
Contributor

@mgorny mgorny commented Nov 12, 2025

PR Checklist:

  • note any issues closed by this PR with closing keywords
  • if you are adding a new page under docs/ or community/, you have added it to the sidebar in the corresponding _sidebar.json file
  • put any other relevant information below

CC @isuruf

@mgorny mgorny requested a review from a team as a code owner November 12, 2025 20:10
@netlify
Copy link

netlify bot commented Nov 12, 2025

Deploy Preview for conda-forge-previews ready!

Name Link
🔨 Latest commit 5b130f0
🔍 Latest deploy log https://app.netlify.com/projects/conda-forge-previews/deploys/6914e9a341f98000080863c8
😎 Deploy Preview https://deploy-preview-2654--conda-forge-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 79
Accessibility: 96
Best Practices: 100
SEO: 89
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

### Building packages using configure scripts written in shell script

Some packages do not provide build systems with first-hand support for Windows, in particular these using `configure` scripts using autotools, or hand-written in shell scripts.
You can use the [autotools_clang_conda](https://github.com/conda-forge/autotools_clang_conda-feedstock?tab=readme-ov-file#about-autotools_clang_conda-feedstock) to provide a build environment with a set of MSYS2 packages and a Clang toolchain configured to build packages compatible with MSVC.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The README in autotools_clang_conda-feedstock is not properly rendered. Can you link to https://github.com/conda-forge/autotools_clang_conda-feedstock/blob/main/recipe/meta.yaml#L41-L74 ?

Copy link
Contributor Author

@mgorny mgorny Nov 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I was going to ask if we should add some "```"s to make it render as Markdown. The problem with linking to specific lines is that the link's going to become slightly wrong when the line numbers change. I could link to specific commit, but then the instructions would eventually become outdated.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you can get the README to render correctly, go for it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this good to go now?

@isuruf isuruf merged commit 364a5de into conda-forge:main Nov 14, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants