Skip to content

Conversation

@wolfv
Copy link
Member

@wolfv wolfv commented May 27, 2025

I am trying to fix an issue where the Gst typelib files are not installed properly for the osx-arm64 (cross-compiled) package.

@conda-forge-admin
Copy link
Contributor

conda-forge-admin commented May 27, 2025

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

I wanted to let you know that I linted all conda-recipes in your PR (recipe/recipe.yaml) and found some lint.

Here's what I've got...

For recipe/recipe.yaml:

  • ❌ Selectors in comment form no longer work in v1 recipes. Instead, if / then / else maps must be used. See lines [18].

For recipe/recipe.yaml:

  • ℹ️ The feedstock is lowering the image versions for one or more platforms: {'linux_64': 'cos7'} (the default is alma9). Unless you are in the very rare case of repackaging binaryartifacts, consider removing these overrides from conda-forge.yml in the top feedstock directory.

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

@Tobias-Fischer
Copy link

Hi @wolfv - by coincidence we've just been running into this issue as well. Any idea of what's going on?

@wolfv
Copy link
Member Author

wolfv commented May 28, 2025

I think cross-compilation with gi-introspection is really tricky. For example, on Nix it's also often disabled under cross-compilation (NixOS/nixpkgs#72868).

However, we can add gstreamer as a native build to .cirun! Then things should work fine :)

@Tobias-Fischer
Copy link

I didn’t know we can have native osx-arm64 builds these days - amazing, how does that work?

@wolfv
Copy link
Member Author

wolfv commented May 28, 2025

Need to make a PR like this: conda-forge/.cirun#102 and change conda-forge.yml like this: https://github.com/conda-forge/pixi-pack-feedstock/pull/39/files

If someone has time :) 👀 would love some help

@wolfv
Copy link
Member Author

wolfv commented May 29, 2025

Thanks @Tobias-Fischer - I just reached out to cirun folks to see if they know what the issue is about the job not starting :)

@Tobias-Fischer
Copy link

Seems like it's running now but failing:

  ++++ dirname /Library/Developer/CommandLineTools/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk
  +++ mkdir -p /Library/Developer/CommandLineTools/Platforms/MacOSX.platform/Developer/SDKs
  mkdir: /Library/Developer/CommandLineTools/Platforms: Permission denied

@wolfv
Copy link
Member Author

wolfv commented May 30, 2025

That was fixed by setting the OSX_SDK_DIR=~/sdk which is writable in the runner. Unfortunately, the build currently fails with an error that I don't understand :/ Locally on macos ARM64 it passes fine.

@conda-forge-admin
Copy link
Contributor

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

I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@wolfv wolfv force-pushed the test-osx-arm64 branch from a9effa3 to 35bf2f7 Compare May 31, 2025 05:19
@conda-forge-admin
Copy link
Contributor

conda-forge-admin commented May 31, 2025

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/recipe.yaml) and found it was in an excellent condition.

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

For recipe/recipe.yaml:

  • ℹ️ The feedstock is lowering the image versions for one or more platforms: {'linux_64': 'cos7'} (the default is alma9). Unless you are in the very rare case of repackaging binaryartifacts, consider removing these overrides from conda-forge.yml in the top feedstock directory.

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

@hmaarrfk
Copy link
Contributor

Sorry I should have waited for this for the version update.

@wolfv wolfv changed the title Test osx arm64 [DO NOT MERGE] Test osx arm64 Jun 2, 2025
@wolfv
Copy link
Member Author

wolfv commented Jun 2, 2025

I manually checked the package locally and it seems fine. There is a window that pops up that I could not read (it goes away quickly) but it makes me think that maybe execution on the runner doesn't work because we don't have "sudo" permissions (same reason maybe that I ahve to point the OSX_SDK dir to ~/sdk instead of /opt/sdk).

Not sure what to do. We could try to work with @aktech to fix the issue in the VM or we could accept that the VM doesn't run the binaries and just merge it and see (without tests).

@wolfv
Copy link
Member Author

wolfv commented Jun 2, 2025

Just to note, do not merge this PR as it's missign a lot of stuff :)

@hmaarrfk hmaarrfk marked this pull request as draft June 2, 2025 14:06
@hmaarrfk
Copy link
Contributor

hmaarrfk commented Jun 2, 2025

made it a draft to avoid a "1 click accidental merge"

@yishai1999
Copy link
Contributor

Closing this due to inactivity. @wolfv reopen if necessary.

@yishai1999 yishai1999 closed this Nov 15, 2025
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.

5 participants