Opened 9 years ago
Last modified 2 years ago
#50229 new enhancement
py-ngl: use compilers portgroup; update and download automatically
Reported by: | dstrubbe (David Strubbe) | Owned by: | petrrr |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.4 |
Keywords: | haspatch, portgroup | Cc: | |
Port: | py-ngl |
Description
py-ngl should use the compilers portgroup instead of the obsolete recipe for Fortran variants, as in the attached patch. It can also be updated to version 1.4.0, and have automatic downloading enabled, as was done for py-nio and as discussed in ticket #42541.
Attachments (1)
Change History (11)
Changed 9 years ago by dstrubbe (David Strubbe)
Attachment: | Portfile-py-ngl.diff added |
---|
comment:1 Changed 9 years ago by dstrubbe (David Strubbe)
comment:2 Changed 9 years ago by dstrubbe (David Strubbe)
Summary: | py-ngl: use compilers portgroup → py-ngl: use compilers portgroup; update and download automatically |
---|
comment:3 follow-up: 4 Changed 9 years ago by petrrr
Hi, sorry for reacting so late on this, but I got pretty busy lately and could not look after Macports that much.
I had started work on update and automatic update already quite some time ago. But there were some issues with the build/linking so I never push these updates out. Feel free to fix these, but please make sure that the builds are really okay, that linking is okay and all dependencies are listed. Otherwise I will merge you proposal with what I have in work.
I you are particularly interested in this port you can take over maintainer-ship.
comment:4 Changed 8 years ago by dstrubbe (David Strubbe)
I had started work on update and automatic update already quite some time ago. But there were some issues with the build/linking so I never push these updates out. Feel free to fix these, but please make sure that the builds are really okay, that linking is okay and all dependencies are listed. Otherwise I will merge you proposal with what I have in work.
Hi Petr, in order to remove the manual download requirement, I just need to know what the actual URL is for the files, that you only get after registering on their website. So, if you have that, we can try it out.
comment:5 Changed 8 years ago by petrrr
Sure, I will commit only the download change for now. I am not sure I will find the time to fix the other issue right now, but feel free to look into that.
comment:6 Changed 8 years ago by petrrr
Automatic fetching done in r149298.
Fetch works, but I was not able to test build and installation it due to some problem with dependency ncarg
, will look into the details asap.
comment:7 Changed 8 years ago by dstrubbe (David Strubbe)
Build and installation worked fine for me, using ncarg @6.3.0_6+gcc5. I just added a test phase, though it doesn't work for me. It says unable to import Ngl.
comment:8 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
There were numerous problems with the test phase, including a bug that caused portindex to fail. I've fixed what I could in r149328, however the test phase still fails to work, saying:
Error: I am unable to import the NumPy version of Ngl via 'import Ngl'.
comment:9 Changed 8 years ago by dstrubbe (David Strubbe)
Thanks for fixing the PYTHONPATH more generally. The test seems to reveal a problem in the way the port installs.
comment:10 Changed 2 years ago by mascguy (Christopher Nielsen)
Keywords: | portgroup added |
---|
Add keyword portgroup, to pg-related tickets
Maintainer timeout, Fortran variants fixed r144693. Update and automatically downloading still needs to be done.