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.

Sign up for GitHub

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

Jump to bottom

[dev] Reinstate libllvm-c{maj} as part of llvmdev #298

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from
Open

[dev] Reinstate libllvm-c{maj} as part of llvmdev #298

wants to merge 12 commits into from
+21 −35

Conversation

h-vetinari
Copy link
Member

@h-vetinari h-vetinari commented Nov 19, 2024 •
edited
Loading

Forward-port #301

@conda-forge-admin
Copy link
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.

h-vetinari and others added 7 commits November 27, 2024 07:45
@h-vetinari
Reinstate libllvm-c{maj} as part of llvmdev
0683357 
The removal in b4501ac did not take effect, and following through would
require clobbering CMake metadata (as otherwise the targets aren't present),
see conda-forge#297 for attempted implementation and discussion.

This reverts commit 55b5f72.

This reverts commit b4501ac.
@h-vetinari
use new smithy config to install ATL components
41c372e
@h-vetinari
MNT: Re-rendered with conda-build 24.11.1, conda-smithy 3.44.8, and c…
6eeee96 
…onda-forge-pinning 2024.11.26.07.33.56
@conda-forge-admin
Reinstate libllvm-c{maj} as part of llvmdev (conda-forge#301)
c59e6bc 
automerged PR by conda-forge/automerge-action
@h-vetinari
Merge remote-tracking branch 'upstream/main' into dev
42dede0
@h-vetinari
MNT: Re-rendered with conda-build 24.11.1, conda-smithy 3.44.8, and c…
bb7e8d1 
…onda-forge-pinning 2024.11.26.23.29.58
@h-vetinari
build llvm v20 off of llvm/llvm-project@74449ab
bcddef4
@h-vetinari h-vetinari changed the title [dev] Actually remove LLVM-C.{dll,lib} from llvmdev [dev] Reinstate libllvm-c{maj} as part of llvmdev Nov 27, 2024
@conda-forge-admin
Copy link
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). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12043256729. 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
Reviewers

@JohanMabille JohanMabille Awaiting requested review from JohanMabille JohanMabille is a code owner

@SylvainCorlay SylvainCorlay Awaiting requested review from SylvainCorlay SylvainCorlay is a code owner

@inducer inducer Awaiting requested review from inducer inducer is a code owner

@isuruf isuruf Awaiting requested review from isuruf isuruf is a code owner

@jakirkham jakirkham Awaiting requested review from jakirkham jakirkham is a code owner

@mingwandroid mingwandroid Awaiting requested review from mingwandroid mingwandroid is a code owner

@timsnyder timsnyder Awaiting requested review from timsnyder timsnyder is a code owner

@xhochy xhochy Awaiting requested review from xhochy xhochy is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@h-vetinari @conda-forge-admin