PetarKirov / dlang.nix

Nix expressions for building D compilers
MIT License
6 stars 2 forks source link

Porting changes from OldDDerivations, attempt 2. #4

Closed dukc closed 11 months ago

dukc commented 11 months ago

Note: Draft since some C++ test is still failing at late phase (testing 2.102, NixOS with the current stable channel)

This supersedes #2

I have pulled the changes done to this repo since the original PR and reimplemented my changes on top of the latest main version.

@PetarKirov please do something this time once I get this out of draft. I do understand you want to implement the automatic hash fetcher script that will make the per-version nix files needless. I know your intention was to rebase my original MR yourself, which would solve the problem if waiting for it wasn't blocking me from doing anything else in the meantime.

However, compiler versions and NixPkgs changes keep coming, and I don't want to be blocked for months before dealing with them. In the present, I'd have to push updates to OldDDerivations with knowledge that I'll have to do them again when it comes the time to port them here. This is just unworkable - I should have to do my job only once. I think there are three possibilities:

dukc commented 11 months ago

Ping @PetarKirov this now works, ready for review.

PetarKirov commented 11 months ago

Hi @dukc, thanks for this PR (and sorry for causing you the trouble of having to redo #2). I'll try to finish reviewing it today.

Btw, thanks for the ping, I just saw it by chance. In the future, can you try o reach out to me on either Slack or Discord, so you don't end up waiting? In general, I am very rarely able to keep up with GH notifications.

PetarKirov commented 11 months ago

In the mean time, could you review #3 (commit by commit, as looking at the total diff would be quite difficult to follow). I think many of the changes there should be fine in isolation.

dukc commented 11 months ago

LGTM. There 3 things I'd like to do before merging: [snip]

I reworded the commits and changed the Libgcc dependency.

But I don't wish to include (or wait for) a CI in this PR. I very much do want a CI, but it's the sort of thing that should be done independently in a separate PR. Your existing CI work should need no adaptations for rebasing on this one, because the command for building default.nix is not changing.

dukc commented 11 months ago

In the future, can you try o reach out to me on either Slack or Discord, so you don't end up waiting?

Oh I'll try to keep in mind. Maybe I'll still ping too but when I really want attention then Slack it is.

dukc commented 11 months ago

In the mean time, could you review https://github.com/PetarKirov/dlang-nix/pull/3 (commit by commit, as looking at the total diff would be quite difficult to follow). I think many of the changes there should be fine in isolation.

I will, at some point.

PetarKirov commented 11 months ago

I just came back from travel abroad and found a few spare minutes to fix the remaining issues (not using ${dmdPrefix} everywhere and attempting to delete runnable_cxx/cppa.d). I also did a final interactive rebase to polish the commit messages.

PetarKirov commented 11 months ago

@dukc I had to downgrade the default DMD version back to 2.100.2, as 2.102.2 didn't pass the macOS CI.

I have logged #8 to track this issue.