-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Added libqasm recipe #17719
Added libqasm recipe #17719
Conversation
…20_compilation_errors.patch, and updated conandata.yml as suggested in the pull request review.
…ow to use 'using' type aliases).
This reverts commit a94d7b1.
This reverts commit 05fbdb8.
…e)." This reverts commit 788b028.
…sn't allow to use 'using' type aliases)." This reverts commit 73c9294.
…fix_cpp_20_compilation_errors.patch, and updated conandata.yml as suggested in the pull request review." This reverts commit 6e1cafe.
This reverts commit 22fc695.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…o 3.20 (for cxx_std_23). conandata.yml: added '0.5.1' to sources. conanfile.py: - layout: 'src_folder' set to '.'. - source: get sources from conandata.yml. - validate: updated to be compliant with Conan 1.x.
This comment has been minimized.
This comment has been minimized.
* pixman/all/: bump deps * pixman/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericlemanissier.github.io/conan-center-index-bump-deps/)
@SpaceIm @SSE4 @RubenRBS @uilianries anyone please do a second code review? Thanks! |
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.
Looking good, only small details.
Many thanks @uilianries! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
LGTM
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.
Hi! Thanks a lot, I have a few last minute change requests, but other than that it looks finally ready :)
I have also a request. The library seems aware of wasm
. Have you locally tested it in such arch? If so, could you please provide compilation logs of the default build (So just conan create ...
) and shared? (conan create ... -o="*:shared=True"
)? Thanks a lot! It helps tremendously with future maintenance and traceability
Hi @RubenRBS. I am not sure if we have what you ask for because we only cross-compile for emscripten/WASM. libqasm WASM binaries are already being tested/used in the future Quantum Inspire 2 website. You can find the binaries for the latest libqasm release here. Should you want to build them and run them:
You also have GitHub logs for the And, for the shared build, please find attached the logs: conan__build__pr_emscripten__pr_b_release__o_shared_True__b_missing.txt I hope this helps! |
This comment has been minimized.
This comment has been minimized.
@RubenRBS sorry for the stalking haha but I'm just so much willing this to go forward. Thanks! |
Conan v1 pipeline ✔️All green in build 41 (
Conan v2 pipeline ✔️
All green in build 41 (
|
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.
Thanks a lot for taking the time to create the recipe, we really appreciate it :)
You won't believe me but I've been dreaming about this moment since more than one year ago. 😄 Long live the Conan team! |
Specify library name and version: libqasm/0.6.5
libqasm is a library for handling cQASM files using C++ or Python (https://github.com/QuTech-Delft/libqasm, https://libqasm.readthedocs.io/en/latest/).
It is used by OpenQL, a framework for high-level quantum programming in C++/Python (https://github.com/QuTech-Delft/OpenQL, https://openql.readthedocs.io/en/latest/).
Both libqasm and OpenQL are developed by QuTech-Delft (TU Delft), for which I work as a software engineer.
It is a new addition to ConanCenter.
I basically want OpenQL users to be able to use libqasm in a simple way (i.e. without needing to install libqasm's dependencies manually).
I've used conan 2.0.4.
I've tried building both in Linux and Windows in release and debug modes. These are the commands I use in each platform:
conan create all/conanfile.py --version 0.5.0
conan create all/conanfile.py --version 0.5.0 -o libqasm/*:build_type=Debug