-
Notifications
You must be signed in to change notification settings - Fork 16
feat: use compliant llvm + tket artifacts from hugrverse-env #1471
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
Merged
Merged
Changes from 36 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
2665af6
Use hugrverse-env to source llvm builds, bring manylinux compliance b…
jake-arkinstall dc6ff98
Neaten up python-wheels.yml
jake-arkinstall c9539f0
Correct mkdir invocation in python-wheels.yml
jake-arkinstall 9474d90
Fix devenv path, allow lib and lib64 paths for shared libraries, avoi…
jake-arkinstall 86d3f96
Bump to hugrenv 0.3.1 (GMP removal)
jake-arkinstall 8b4cc3c
Remove conan as a default dev package, test wheels in a fresh location
jake-arkinstall 60c49aa
Set DYLD_LIBRARY_PATH to point to hugrenv when running delocate on macos
jake-arkinstall 473b006
LD_LIBRARY_PATH fixes, add explicit tket.lib dependency on windows in…
jake-arkinstall 1104c73
uv sync, update snapshots
jake-arkinstall e83908b
Don't flood LD_LIBRARY_PATH for non-build environments (as pytket's l…
jake-arkinstall dfaa475
Add extra windows deps, don't overwrite pytket deps when testing python
jake-arkinstall d3f4667
Try new windows beta release of hugrverse-env
jake-arkinstall 389973d
Re-apply copying of tket_c_api.dll to wheel in windows
jake-arkinstall 7d53ab2
Add hugrenv utilities
jake-arkinstall 38f433d
Move install-hugrenv action to actions dir
jake-arkinstall c2e0b75
Terminate for loop in install-hugrenv action
jake-arkinstall a11052e
mkdir
jake-arkinstall 38f34f5
Add hugrverse_env to path env vars, fixup python-qis-wheels.yml
jake-arkinstall ec56c93
Remove unused caching in qis-compiler, s/macos/macosx/, correct paths…
jake-arkinstall afb7664
Make install-hugrenv detect platform rather than accept info as inputs
jake-arkinstall 179de4c
Correctly set install_path
jake-arkinstall 39254e2
Move extraction path to /tmp/hugrverse to avoid macos permission issues
jake-arkinstall 2b73bd4
Move LLVM to use hugrenv's version rather than nixpkgs' version, unbi…
jake-arkinstall 9ecacc7
Correct windows PATH in CIBW
jake-arkinstall 07aea4f
Fix windows PATH in CIBW (again)
jake-arkinstall 11ee8d9
Restore manylinux_2_28 flag on ARM python-wheels builds, set windows …
jake-arkinstall 494c81a
Don't weakly prefer uv install tket from the built wheel - require it…
jake-arkinstall fd5bf7e
Use a venv instead of uv for testing python wheel import, and manuall…
jake-arkinstall 6d778a8
Try using remote action for installing hugrenv
jake-arkinstall 5e251d5
Typo fix
jake-arkinstall 17ac5be
Typo fix
jake-arkinstall e1cd48a
Set MACOSX_DEPLOYMENT_TARGET, use unified venv invocation when testin…
jake-arkinstall e71d05d
Update hugrenv to 0.5.0 (with macos 11 compliance), utilise new hugre…
jake-arkinstall 3cdfe5d
Merge branch 'main' into feat/compliant-llvm
jake-arkinstall 9bceef6
Use main branch of install-hugrenv
jake-arkinstall 650c55b
Update hugrenv.nix to take the macosx deployment target 15->11 change…
jake-arkinstall 398d2f3
Remove unnecessary LD_LIBRARY_PATH stripping in ci.yml - python wheel…
jake-arkinstall c9e7f6a
Update comments in python-wheels.yml for clarity
jake-arkinstall b45679b
feat: add local hugrenv fetch command and dev docs (#1500)
Copilot f879689
Merge branch 'main' into feat/compliant-llvm
jake-arkinstall 8c89fd9
Point to new location for the install-hugrenv-action
jake-arkinstall 613eee9
Satisfy mypy in scripts/fetch_hugrenv.py
jake-arkinstall 3d7119a
Satisfy ruff
jake-arkinstall 9386975
Satisfy ruff (cont.d)
jake-arkinstall File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,6 +5,7 @@ rust-core: &rust-core | |
| - "tket/**" | ||
| - "Cargo.toml" | ||
| - "Cargo.lock" | ||
| - "hugrenv.lock" | ||
|
|
||
| rust: | ||
| - *rust-core | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a nitpick, but we could have
install-hugrenvat the root of the repo so the callpoints are cleaner.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
agree, will do that now