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

enable mongodb to use cirun #949

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Conversation

h-vetinari
Copy link
Member

Current mongodb v7 manages to compile about half the objects before running into the 6h timecap (example; around [2385+2/4544] on linux-64, [2398+2/4489] on win-64, etc.) - completely hopeless to get it over the line.

CC @jaimergp

@h-vetinari h-vetinari requested a review from a team as a code owner February 23, 2024 12:42
@jaimergp
Copy link
Member

The server can help with Linux builds but we can't do much for Windows, though. I wonder if it's just that mongodb has increased in complexity over time, or whether there's something off in the build.

Also, tagging @conda-forge/mongodb to follow protocol (but I guess we won't hear back, the feedstock has been inactive for a while).

@h-vetinari
Copy link
Member Author

The server can help with Linux builds but we can't do much for Windows, though.

We could conceivably cross-compile from windows from linux. We have linux-64/clang_win-64, but I've never tried to cross-compile across OSes. Perhaps @isuruf or @xhochy have thoughts.

Also, tagging @conda-forge/mongodb to follow protocol (but I guess we won't hear back, the feedstock has been inactive for a while).

I did a big push late last year to revive it, but didn't get all the way up-to-date due to the timeouts. Now that we have the cirun server, I thought I'd give it another shot.

@xhochy
Copy link
Member

xhochy commented Feb 26, 2024

We could conceivably cross-compile from windows from linux. We have linux-64/clang_win-64, but I've never tried to cross-compile across OSes. Perhaps @isuruf or @xhochy have thoughts.

The only feedstock where we do this is https://github.com/conda-forge/polars-feedstock There is one major constraint though: You cannot have a host environment in that situation. There is a major conda-build bug with prefix replacement. I would rather wait until we have rattler-build in conda-forge than tackling this.

@jaimergp jaimergp merged commit 9d39cab into conda-forge:main Feb 26, 2024
1 check passed
@h-vetinari h-vetinari deleted the cirun_mongodb branch February 26, 2024 07:04
@h-vetinari
Copy link
Member Author

There is one major constraint though: You cannot have a host environment in that situation. There is a major conda-build bug with prefix replacement.

It seems Isuru recently tackled (at least part of) this: conda/conda#13687 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants