stdenv/cygwin: add cross bootstrap tools#475998
stdenv/cygwin: add cross bootstrap tools#475998corngood wants to merge 1 commit intoNixOS:masterfrom
Conversation
0cf2aa3 to
b8d334b
Compare
b8d334b to
13cf382
Compare
This comment was marked as outdated.
This comment was marked as outdated.
I'm sure it's possible, but I think it's premature when the only stdenv currently doing it is x86 linux. I suspect we'd want to refactor and share a lot of linux/minimal-bootstrap. I don't think cygwin is the platform we want to use to build that abstraction. Darwin or freebsd would probably be better. |
Very fair, then feel free to discard my comments. The work here looks good! |
|
I've marked this ready because all the original dependencies are merged, and I think the change is still good. Unfortunately it doesn't build at the moment due to regressions, which I'll fix and link in the top comment. |
|
Can we just cross compile from a pure bootstrap platform and say, well, you need multiple types of builders if it isn't cached? I don't see why not. |
I did have a stdenv based on stdenv/cross before this one. It would be very frustrating for an end user IMO. This is quite similar to what freebsd and darwin do. This change doesn't even have the stdenv yet. I really just want all the things required by stdenv to be in hydra, so we can keep them building while we do the rest of the work. I think long-term, minimal-bootstrap would be a great way to go, but it's very linux-centric at the moment. |
|
#123095 (comment) to be clear, something like this is what I am thinking. We can make a "make bootstrap tools" of sorts, but rather than using it to produce something we fetch in a separate eval, we would just directly use it in the native bootstrap. We'd rely on regular substitution rather than fetching a fixed output. |
Like with minimal-bootstrap, I'm not against this idea, but I'd rather start off with a conventional stdenv (like darwin/freebsd) until the concept has been proven. |
I've built a working stdenv based on this, but I think this is a good starting point to get it merged.
Once this is in we should be able to get a tarball from hydra.
I'm marking this draft until I enumerate the package changes required for it to build.
Build with:
Dependencies:
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.