Opened 10 years ago
Closed 10 years ago
#47395 closed enhancement (fixed)
cantera: download from github
Reported by: | macports@… | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | maintainer haspatch | Cc: | |
Port: | cantera |
Description
Cantera is moving to GitHub (see https://groups.google.com/forum/?fromgroups#!topic/cantera-users/-JyScUZm0O0)
This update just changes the download source.
Attachments (2)
Change History (8)
Changed 10 years ago by macports@…
Attachment: | cantera_patch_portfile.diff added |
---|
comment:1 Changed 10 years ago by mf2k (Frank Schima)
Cc: | mf2k@… removed |
---|---|
Keywords: | maintainer added |
Type: | update → enhancement |
Version: | 2.3.3 |
Thanks! Per the guidelines, please use a unified diff format for your patch.
comment:2 Changed 10 years ago by macports@…
Thank you for pointing me to the right paragraph in the guidelines!
comment:3 Changed 10 years ago by larryv (Lawrence Velázquez)
I haven’t looked at this at all, but make sure that you don’t have to account for a stealth update here (PortfileRecipes#stealth-updates).
Changed 10 years ago by macports@…
Attachment: | Portfile-cantera.diff added |
---|
Cantera 2.1.2 on GitHub - Diff
comment:4 Changed 10 years ago by macports@…
Thanks, that should definitely be there! I added that line.
comment:5 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Keywords: | haspatch added |
---|---|
Owner: | changed from macports-tickets@… to ryandesign@… |
Status: | new → assigned |
Summary: | Update cantera port for new source → cantera: download from github |
If a stealth update had occurred, the checksums would have changed, but you did not indicate changed checksums. In fact the checksums do change if we use your patch as-is, because your patch makes the port download from a tarball automatically generated by github from the git repository. Since this project also offers release downloads, we should use those instead, by using github.tarball_from releases
. The checksums match those from sourceforge, so no stealth update has occurred and there is no need to change the dist_subdir
.
Note also that the github.setup
line is meant to replace, not supplement, the name
and version
lines; you would only keep the name
or version
lines if their values differed from those specified in the github.setup
line, but in this case (and in most cases) they don't.
The cantera web site still tells people to download from SourceForge. You might want to let them know they should update that to refer people to github instead.
comment:6 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Cantera 2.1.2 on GitHub - Diff