Support NixOS

Issue #2104 resolved
Erik Schnetter created an issue

NixOS is a new Linux-based distribution. To make things work, a few small changes are necessary.

https://bitbucket.org/cactuscode/cactus/pull-requests/46/flesh-support-nixos/diff

Keyword:

Comments (8)

  1. Erik Schnetter reporter
    • removed comment

    There are only a few lines of real changes to configure.in and make.configuration. The other changes are caused by regenerating configure and updating config.guess to a newer version.

  2. Frank Löffler
    • removed comment

    One of the changes sets LD to CXX if LD was set to 'ld'. Before, if LD was set to anything, that setting was honored. If LD is set to 'ld' on NixOS by default, would it be possible to set LD to something else than 'ld' (CXX) in the configuration file, instead of hard-coding this into Cactus make? That way, on all systems (not just NixOS), the setting of LD would be continued to be honored.

  3. Erik Schnetter reporter
    • removed comment

    Cactus expects LD to be the C++ compiler driver to be used for linking, not the linker. In other words, setting LD to "ld" is almost guaranteed to be wrong, and setting it to the value of CXX is a good default choice. Since the Cactus definition of LD goes against the grain of what LD means everywhere else (e.g. in autotools), handling this problematic case generically is better than applying a NixOS-only solution.

  4. Roland Haas

    I have resumed the review.

    I agree with Frank that I would rather keep LD at whatever value the user set it to (so that it is less surprising) rather than second guess.

    I would prefer config.guess’s update to be a separate commit stating just that “update config.guess to XX.YY version” whichever one it may be (assuming you took it from its official repo at http://savannah.gnu.org/projects/config/).

  5. Roland Haas

    Of the 3 changes:

    • force LD to not be “ld”
    • don’t assume “date” is in “/bin/date”
    • udpate config.guess

    The first one has been rejected, the second applied in git hash da64f045 "Cactus: do not assume that date command is in /bin/date" of cactus and the third is a duplicate of #1452. Closing this as resolved / duplicate.

  6. Log in to comment