Opened 8 years ago

Closed 8 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)

Portfile (542 bytes) - added by lpsinger (Leo Singer) 8 years ago.
Proposed Portfile
Portfile.diff (1.4 KB) - added by lpsinger (Leo Singer) 8 years ago.
Diff relative to current Portfile

Download all attachments as: .zip

Change History (11)

comment:1 Changed 8 years ago by lpsinger (Leo Singer)

Please see attached patch.

comment:2 Changed 8 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 in reply to:  2 Changed 8 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 8 years ago by lpsinger (Leo Singer)

Attachment: Portfile added

Proposed Portfile

Changed 8 years ago by lpsinger (Leo Singer)

Attachment: Portfile.diff added

Diff relative to current Portfile

comment:4 Changed 8 years ago by lpsinger (Leo Singer)

Please see new patches. Originally I had uploaded patches for the wrong port.

comment:5 Changed 8 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 in reply to:  5 Changed 8 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 8 years ago by lpsinger (Leo Singer)

Hmm, that "if" statement has to come after the "name" line.

comment:8 Changed 8 years ago by kurthindenburg (Kurt Hindenburg)

You might be correct - look at other python obsolete ports - I can test it later

comment:9 Changed 8 years ago by lpsinger (Leo Singer)

Resolution: fixed
Status: newclosed

Looks like this was fixed in commit 33760b18251b93a20a560c77bbe2675f5b4aba44.

Note: See TracTickets for help on using tickets.