Opened 6 years ago
Closed 4 years ago
#56787 closed enhancement (fixed)
gdal: add fits variant
Reported by: | mazhe (Matthieu Volat) | Owned by: | Veence (Vincent) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | |
Port: | gdal |
Description
Hello,
The gdal library can have support for the FITS image format used in astronomy through the cfitsio project. While gdal already have a lot of variants, it would be super useful to have this one added.
Attached is a patch to the Portfile that allows this, could this be commited to the port?
Thank you very much for your time,
Attachments (1)
Change History (9)
comment:1 Changed 6 years ago by kencu (Ken)
Keywords: | haspatch added; gdal fits removed |
---|---|
Owner: | set to Veence |
Port: | gdal added; gis/gdal removed |
Status: | new → assigned |
Summary: | add fits variant to gis/gdal → request to add fits variant to gis/gdal |
comment:2 Changed 6 years ago by kencu (Ken)
Changed 6 years ago by mazhe (Matthieu Volat)
Attachment: | gdal_fits_variant.diff added |
---|
Simpler patch.
comment:3 Changed 6 years ago by mazhe (Matthieu Volat)
Ho, thanks. I'm not that familiar with Portfile syntax, I just copied what was done in other parts of the file.
Here's an updated version.
comment:4 Changed 6 years ago by kencu (Ken)
Yeah, I think that should do it. Let's give Vince (the maintainer) time to check it out now. -- K
comment:5 Changed 6 years ago by mazhe (Matthieu Volat)
Hi, is there any progress or comment regarding this request? priority is not that high, but it would be nice to have it done!
Thanks again,
comment:6 Changed 6 years ago by kencu (Ken)
Vince hasn't noticed this one so far I guess.
I'll put together a PR for this (or you can if you like). Then it will get committed after a short while if it passes muster on the CI system.
comment:8 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Summary: | request to add fits variant to gis/gdal → gdal: add fits variant |
A cfitsio variant was added in [e3f7c230c2e0baf64bf5cb715dc16b70f2d8ed72/macports-ports]
To make this variant work the way you want it to, you'd have to leave
as it is in the default configure arguments (as you want it off by default).
Then, in your variant, you can do this replacement: