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

Retrigger CI on v1.1.3 to track down cross compilation failures #478

Closed
wants to merge 7 commits into from

Conversation

Maxxen
Copy link
Member

@Maxxen Maxxen commented Jan 14, 2025

No description provided.

@Maxxen
Copy link
Member Author

Maxxen commented Jan 14, 2025

@carlopi Seems like arm cross compilation still works on v1.1.3. Im going to re-run it again but this time keep duckdb at 1.1.3 but use latest extension ci tools

@Maxxen
Copy link
Member Author

Maxxen commented Jan 14, 2025

Cool, seems like that somewhere between CMake 3.21 and CMake 3.25 recursive-make cross compilation broke.

@Maxxen
Copy link
Member Author

Maxxen commented Jan 14, 2025

PR is up duckdb/extension-ci-tools#129

@carlopi
Copy link
Contributor

carlopi commented Jan 14, 2025

PR there has been merged, but then I remembered why the CMake bump was required in the first place: to have cmake work correctly for boost::filesystem, needed by the iceberg extension.

I am not sure what's worse.

We might have to have another proper look at this mess.

@Maxxen Maxxen closed this Jan 16, 2025
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.

2 participants