-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[package] arrow/15.0.0: user options are not respected #22506
Labels
bug
Something isn't working
Comments
This was referenced Jan 23, 2024
Indeed, it's a regression coming from PR which has migrated arrow to conan v2: #19380 (comment) There is a fix in #21669 I think. |
Yes, I have verified the fix works on my side. |
wgtmac
added a commit
to wgtmac/conan-center-index
that referenced
this issue
Feb 25, 2024
wgtmac
added a commit
to wgtmac/conan-center-index
that referenced
this issue
Feb 26, 2024
wgtmac
added a commit
to wgtmac/conan-center-index
that referenced
this issue
Feb 28, 2024
Can be closed after #23163 was merged. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
A set of self.options.xxx options are initialized in config_options: https://github.com/conan-io/conan-center-index/blob/master/recipes/arrow/all/conanfile.py#L161-L175. However, this is a misuse. The config_options() method is used to configure or constrain the available options in a package before assigning them a value. The config_options() method executes:
Check https://docs.conan.io/2.0/reference/conanfile/methods/config_options.html for detail.
I've opened a PR in the arrow community: apache/arrow#39729. It would be good to make it consistent to the upstream.
Package and Environment Details
Conan profile
[settings]
arch=armv8
build_type=Release
compiler=clang
compiler.cppstd=gnu17
compiler.libcxx=libc++
compiler.version=17
os=Macos
Steps to reproduce
conan create . --profile:host default --profile:build default --build=missing --version 15.0.0 -o parquet=True -o with_brotli=True -o with_bz2=True -o with_flight_rpc=True -o with_glog=True -o with_jemalloc=True -o with_json=True -o with_lz4=True -o with_snappy=True -o with_zstd=True
Logs
Click to expand log
The text was updated successfully, but these errors were encountered: