Opened 2 years ago
Closed 2 years ago
#65647 closed enhancement (fixed)
leiningen: handling codeberg releases
Reported by: | kakuhen | Owned by: | kakuhen |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.7.2 |
Keywords: | Cc: | easye | |
Port: | leiningen |
Description
Upstream recently moved to Codeberg. In the previous update to Leiningen, I noted to grab releases from Codeberg from now on. However, neither upstream nor I anticipated attachments in Codeberg to be UUIDs.
Here is an example involving Leiningen release 2.9.9.
GitHub: https://github.com/technomancy/leiningen/releases/download/2.9.9/leiningen-2.9.9-standalone.jar
Codeberg: https://codeberg.org/attachments/60dddfb3-74f1-4177-945a-a4ccfe8f7d88
Of course, we can easily fix this issue by setting a variable that contains the UUID then finding a way to translate this into a concrete file name, which seems to be provided in HTTP/2 headers. Example below
~ % curl -I https://codeberg.org/attachments/60dddfb3-74f1-4177-945a-a4ccfe8f7d88 HTTP/2 200 access-control-expose-headers: Content-Disposition cache-control: private, max-age=300 content-disposition: inline; filename*=UTF-8''leiningen-2.9.9-standalone.jar content-length: 13201118 content-type: application/octet-stream ...
Thoughts?
Change History (3)
comment:1 Changed 2 years ago by ryandesign (Ryan Carsten Schmidt)
comment:2 Changed 2 years ago by kakuhen
Alright. Looks like we'll just have to stick with the UUIDs until Codeberg changes their attachment URLs to something more convenient. With that said, there is a serious bug(1) in version 2.9.9 and upstream discourages using it(2). For that reason, I am not going to update this port until 2.9.10 is released. Due to the severity of the bug, the release should arrive soon this week.
(1) https://codeberg.org/leiningen/leiningen/issues/5
(2) No formal source; here's a relevant log of #clojure on Libera Chat
<technomancy> psa: might want to hold off on upgrading to lein 2.9.9 https://codeberg.org/leiningen/leiningen/issues/5 <technomancy> working on a fix
comment:3 Changed 2 years ago by kakuhen
Owner: | set to kakuhen |
---|---|
Resolution: | → fixed |
Status: | new → closed |
See wiki:PortfileRecipes#fetchwithgetparams