Add cabal-hash revision info when displaying recommended packages when failing to construct buildplan

[tdietert]

This PR addresses the following issue #3925

TODO:

• Add changes to ChangeLog.md
• Test the output of failed build plan package recommendations

Tested by removing the rio extra-dependency in the stack.yaml file and then trying to build the package with the newly generated stack executable.

. . .

  * Recommended action: try adding the following to your extra-deps in /home/thomas/github/stack/stack.yaml:

rio-0.1.2.0@sha256:cb2a65cee1c0450815002ca890633215e4544b5c1c9f7091a950142c6efe8f3f

[snoyberg]

Can you add a note to the ChangeLog about the change and which issue it resolves?

[tdietert]

I have updated the changelog 👍

[snoyberg]

Regarding our discussion: can you replace the NonEmpty CabalHash with either [CabalHash] or Maybe CabalHash and remove the usage of partial functions below? Alternatively, you could modify the original [CabalHash] to be a NonEmpty CabalHash.

[tdietert]

Unfortunately, any modification to the [CabalHash] type that represents one or two cabal hashes (the latter encoding both the hash of the cabal file with and without carriage returns) returns the error:

/home/thomas/github/stack/src/Stack/PackageIndex.hs:416:21: error:
    • Exception when trying to run compile-time code:
        For Stack.Types.PackageIndex.PackageCache GHC.Tuple.()

Since I don't really have the time to dig into the template-haskell code right now, I will remove the use of the partial function Data.List.NonEmpty.fromList and instead use Data.List.NonEmpty.nonEmpty to capture the notion that the version may not have a properly parsed tarball file neither/nor correctly parshes/hashes the cabal revisions.

[tdietert]

This PR should be ready to go barring no other requested changes 🎉

[snoyberg]

Nice join!