Opened 7 years ago
Closed 7 years ago
#53266 closed enhancement (fixed)
py-pywcs: mark as obsolete, replaced by py-astropy
| Reported by: | lpsinger (Leo Singer) | Owned by: | robitaille@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | kurthindenburg (Kurt Hindenburg) | |
| Port: | py-pywcs |
Description
This is a request to obsolete the py-pyfits port, replaced by py-astropy. There are no ports that depend on py-pywcs.
Attachments (2)
Change History (11)
comment:1 Changed 7 years ago by lpsinger (Leo Singer)
comment:2 follow-up: 3 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
| Cc: | kurthindenburg added; py-pywcs removed |
|---|
You should follow the example in https://guide.macports.org/#development.obsolete-portgroup - it looks like you are at least missing the replaced_by line
comment:3 Changed 7 years ago by lpsinger (Leo Singer)
Replying to kurthindenburg:
You should follow the example in https://guide.macports.org/#development.obsolete-portgroup - it looks like you are at least missing the replaced_by line
Oops, wrong patch attached. One moment.
Changed 7 years ago by lpsinger (Leo Singer)
| Attachment: | Portfile.diff added |
|---|
Diff relative to current Portfile
comment:4 Changed 7 years ago by lpsinger (Leo Singer)
Please see new patches. Originally I had uploaded patches for the wrong port.
comment:5 follow-up: 6 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
If you read the docs, you'll see you have to have the replace_by at the top - you shouldn't need the "if" as well
comment:6 Changed 7 years ago by lpsinger (Leo Singer)
Replying to kurthindenburg:
If you read the docs, you'll see you have to have the replace_by at the top - you shouldn't need the "if" as well
I'll put it at the top, but the "if" is necessary so that the Python subports map to the correct replacements.
comment:7 Changed 7 years ago by lpsinger (Leo Singer)
Hmm, that "if" statement has to come after the "name" line.
comment:8 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
You might be correct - look at other python obsolete ports - I can test it later
comment:9 Changed 7 years ago by lpsinger (Leo Singer)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Looks like this was fixed in commit 33760b18251b93a20a560c77bbe2675f5b4aba44.
Please see attached patch.