Opened 21 months ago
Closed 21 months 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 21 months ago by ryandesign (Ryan Carsten Schmidt)
comment:2 Changed 21 months 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 21 months ago by kakuhen
Owner: | set to kakuhen |
---|---|
Resolution: | → fixed |
Status: | new → closed |
See wiki:PortfileRecipes#fetchwithgetparams