-
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
xerces-c: conan v2 support #13502
xerces-c: conan v2 support #13502
Conversation
Hooks produced the following warnings for commit e5428dexerces-c/3.2.3
xerces-c/3.2.2
|
raise ConanInvalidConfiguration( | ||
"Option '{option}={value}' is only supported on {os}".format( | ||
option=option, value=value, os=os | ||
) | ||
) |
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.
raise ConanInvalidConfiguration( | |
"Option '{option}={value}' is only supported on {os}".format( | |
option=option, value=value, os=os | |
) | |
) | |
raise ConanInvalidConfiguration(f"Option '{option}={value}' is only supported on {os}") |
An observation, would like some assistance in confirming our theory... Since this new revision has been added we're finding that despite My colleague has suggested it is related to the use of However, the official build containers (at least the one's that we're using) provide earlier versions. In terms of evidence, here is what I observe from rrev [full_settings]
arch=x86_64
build_type=Release
compiler=clang
compiler.libcxx=libstdc++
compiler.version=12
os=Linux
[full_requires]
[full_options]
char_type=uint16_t
fPIC=True
message_loader=inmemory
mutex_manager=posix
network_accessor=socket
shared=False
transcoder=gnuiconv The
In rrev [full_settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.libcxx=libstdc++
compiler.version=11
os=Linux
[full_requires]
[full_options]
char_type=uint16_t
fPIC=True
message_loader=inmemory
mutex_manager=posix
network_accessor=socket
shared=False
transcoder=gnuiconv The
In our experience, building from this recipe has caused our products to fail. They link, but we're getting a dynamic link a shared object which isn't available in our runtimes.
|
It doesn't come from presets, but indeed there is an issue and it's because of https://github.com/apache/xerces-c/blob/v3.2.3/cmake/XercesDLL.cmake#L23 (see conan-io/conan#11937 & conan-io/conan#11840). I've pushed a hotfix #13616 |
Specify library name and version: lib/1.0
This is also a good place to share with all of us why you are submitting this PR (specially if it is a new addition to ConanCenter): is it a dependency of other libraries you want to package? Are you the author of the library? Thanks!