Skip to content

Build a variant without features to use latest sysroot outside conda-build (redux) - #88

Merged
beckermr merged 2 commits into
conda-forge:mainfrom
h-vetinari:features
Jul 16, 2025
Merged

beckermr merged 2 commits into
conda-forge:mainfrom
h-vetinari:features

Conversation

@h-vetinari

@h-vetinari h-vetinari commented Jul 15, 2025 •

Copy link
Copy Markdown
Member

Rebase of #75, since @isuruf said in #87:

This would prevent me from cross compiling for newer sysroots from older sysroots. Correct fix is #75

but hasn't responded to

@h-vetinari: OK cool. Would you mind rebasing that PR, or do you want me to do it?

Also, if you wouldn't mind explaining, I'd be curious what the use-case is for needing an older sysroot in build: than the one for the target.

All but the core commit were obsoleted by changes in the meantime (I didn't squash the others, just dropped them, since not applicable anymore).

@conda-forge-admin

conda-forge-admin commented Jul 15, 2025 •

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.
  • ℹ️ License exception is not an SPDX exception.

Documentation on acceptable licenses can be found here.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/16306447828. Examine the logs at this URL for more detail.

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.

4 participants