Generate CheckSum using java.security.MessageDigest
#33
Conversation
This changes allows it to generate the hashes for the JAR's package set without `nix-prefetch-scripts`, but `java.security.MessageDigest` instead. In order to achieve this, the following changes have been applied: * Replaced old `nix-prefetch-url` invokation in `FindArtifactsOfRepo` with `MessageDigest` * Use `Try[T]` for error handling of the artifact's fetcher. * Fixed minor working-directory bug in `tests/template-generation`. * Removed `nix-prefetch-scripts` from `sbtix-tool` closure. * Fixed `sbtix` package build expression. When `.bat` files are present that are non-executable, `wrapProgram` can't create a wrapper. Instead `find . -executable` should be used to wrap files in `$out/bin`. See nightkr#27
Ma27
force-pushed
the
refactoring/#27/get-rid-of-nix-dependency-for-hash-computations
branch
from
e97349c to
762e1fb
Compare
|
@teozkr running the builds in |
Ma27
changed the title
java.security.MessageDigestjava.security.MessageDigest
|
Oh yeah, that's because my old file server's drive died. :/ |
|
is there anything I can do to help you fixing that or should we rather disable the test temporarily? |
|
Probably best to just get it back up again. |
|
Deploying a fix now. |
|
@Ma27 Things should be working again now. I've requested a rebuild of the PR. |
|
first of all thanks a lot for your efforts regarding the private auth test! |
|
Note to self: don't play with Nginx after midnight. It should work now. |
|
@teozkr yeah I can somehow relate to that 🙈 As the build is green now - could you have a look at the changeset please - as it's some time since I started working on this I'm not 100% sure if the change is sufficient and good enough for merge 😅 |
|
@Ma27 Looks good to me. |
Ma27
deleted the
refactoring/#27/get-rid-of-nix-dependency-for-hash-computations
branch
As discussed in #27
Please note that this is just a WIP, it needs a bit more testing and the docs require an update after that.