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.

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

dcmtk: maintenance changes #25997

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

jcar87
Copy link
Contributor

@jcar87 jcar87 commented Nov 21, 2024

@jcar87 jcar87 force-pushed the lcc/maintenance/dcmtk-macos-crossbuild branch from bbebe28 to fa2c737 Compare November 21, 2024 14:13
@@ -0,0 +1,58 @@
#ifndef CONFIG_ARITH_H
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is generated doing a native build on macOS x86_64
Following the guidance at https://support.dcmtk.org/redmine/projects/dcmtk/wiki/Cross_Compiling#Generated-file-arithh

@luadebug luadebug mentioned this pull request Nov 21, 2024
3 tasks
@jcar87 jcar87 marked this pull request as ready for review November 22, 2024 14:36
@ErniGH ErniGH merged commit 73a8c11 into conan-io:master Nov 22, 2024
10 checks passed
@jcar87 jcar87 deleted the lcc/maintenance/dcmtk-macos-crossbuild branch November 22, 2024 15:18
@jcar87 jcar87 mentioned this pull request Nov 22, 2024
3 tasks
raise ConanInvalidConfiguration("MacOS crossbuilding is only supported to target x86_64")
else:
# Note: other cross-building scenarios have not been tested and may also need to be marked as invalid
self.output.warning("Crossbuilding has not been tested and may not work")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cross-compilation is currently only possible if can_run() is true: valgur@0cb08c5

OMGtechy pushed a commit to OMGtechy/conan-center-index that referenced this pull request Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants