[DEVOPS-810] [DEVOPS-908] Inject gitrev using file-embed

[rvl]

Description

This solves the problem of nix rebuilds with identical sources but different git revisions. It doesn't require creating extra cabal projects. The code in utils gets a little simpler as well. The code could get even more simpler if HasCompileInfo were removed -- maybe I should do that?

There is also the addition of an all-cardano-sl attribute.

Linked issue

https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-810
https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-908

Type of change

• 🐞 Bug fix (non-breaking change which fixes an issue)
• 🛠 New feature (non-breaking change which adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
• 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
• 🔨 New or improved tests for existing code
• ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

• I have read the style guide document, and my code follows the code style of this project.
• If my code deals with exceptions, it follows the guidelines.
• I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

• I have added tests to cover my changes.
• All new and existing tests passed.

QA Steps

nix-build -A cardano-sl-auxx
./result/bin/cardano-auxx --version
nix-build -A cardano-sl-auxx --argstr gitrev 1234
./result/bin/cardano-auxx --version

[jmitchell]

I like where you're going with this. Using dummySpaceWith and injectWith to embed the git revision is much more cache-friendly.

There is also the addition of an all-cardano-sl attribute but this is not quite working properly.

How so?

I also like how it simplifies the targets in the CI script, in part by using the reasonable isCardanoSL heuristic.

[rvl]

Have fixed the all-cardano-sl attribute (it was building everything twice).

[parsonsmatt]

Nice, I like it!