Dockerfile: Use leiningen from Debian#323
Conversation
We do not need latest and greatest leiningen. So we don't have to pull it from GitHub. And the Debian apt repo is more trustworthy than pulling a random script with wget from HEAD branch. Signed-off-by: Tim Meusel <tim@bastelfreak.de>
|
The rpm/deb packages and the JAR file for openvox-server are available in a zip archive: |
|
I recently set the min lein version to 2.12 in openvox-server and openvoxdb project.clj. Not sure what the version is in the deb repo. We don't absolutely need the latest, it was just set to very old versions (2.7 or 2.9 or something) that included profile merging bugs which I figured might bite us. |
|
Debian 12 & 13 only have leiningen 2.10 :( But if there is a minimal version we're below that, the build should fail 🤔 . I will have a look tomorrow. |
|
If we're wget'ting something at all, ideally we should have pinned versions and/or checksums. |
|
Yeah, you'd think it would. Weird. In any case, I think it was 2.9 that had a profile merge bug, so 2.10 is probably fine. |
3 participants
We do not need latest and greatest leiningen. So we don't have to pull it from GitHub. And the Debian apt repo is more trustworthy than pulling a random script with wget from HEAD branch.
Pull Request (PR) description
This Pull Request (PR) fixes the following issues