Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove outputs that moved to ctng-compilers-feedstock #110

Merged
merged 1 commit into from
May 2, 2024
Merged

Conversation

h-vetinari
Copy link
Member

Following conda-forge/ctng-compilers-feedstock#127, based on this comment. Probably we can close conda-forge/clang-compiler-activation-feedstock#118 after this PR.

Intentionally not incrementing the build number as nothing about the other outputs should change here.

@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-webservice.

It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.

Please close this pull request and remake it from a fork of this feedstock.

Have a great day!

@conda-forge-webservices
Copy link

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

Copy link
Member

@beckermr beckermr left a comment

Choose a reason for hiding this comment

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

Weird build failures for s390x but otherwise LGTM and thank you!

@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-webservice.

It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.

Please close this pull request and remake it from a fork of this feedstock.

Have a great day!

@h-vetinari
Copy link
Member Author

Weird build failures for s390x but otherwise LGTM and thank you!

Yeah, the s390x failures equally appeared in the most recent CI run on #98. Not sure what's happening there.

Also, sorry for creating a branch here - I'm guessing I must have made a C&P error while cloning, so that I ended up getting the wrong remote for origin. The good thing is that this didn't push new builds, because nothing else about the recipe changed - so I'll leave this for now, and fix my git remote setup in the meantime.

@beckermr
Copy link
Member

beckermr commented May 2, 2024

Yep, all good! Let's merge!

@beckermr beckermr merged commit fab3a82 into main May 2, 2024
57 of 62 checks passed
@beckermr beckermr deleted the no_plain_gcc branch May 2, 2024 11:15
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.

Harmonize symlinks structure between platforms
2 participants