Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#43536 closed update (fixed)

sextractor: please update to new version 2.19.5

Reported by: cdeil (Christoph Deil) Owned by: tim@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: sextractor

Description

Could you please update SExtractor to the new version 2.19.5 ?

http://www.astromatic.net/2014/03/20/package-updates

Attachments (7)

Portfile-missfits.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-sextractor.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-skymaker.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-stiff.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-stuff.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-swarp.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.
Portfile-weightwatcher.diff (1.2 KB) - added by cdeil (Christoph Deil) 10 years ago.

Download all attachments as: .zip

Change History (19)

comment:1 Changed 10 years ago by cdeil (Christoph Deil)

The other packages, like SWarp have also been updated ... should I file a separate issue for each one?

comment:2 Changed 10 years ago by mf2k (Frank Schima)

Cc: tim@… removed
Owner: changed from macports-tickets@… to tim@…
Type: defectupdate
Version: 2.2.1

They probably should be separate requests unless they all have the same maintainer and are related.

comment:3 Changed 10 years ago by cdeil (Christoph Deil)

This is the complete list of ports of http://www.astromatic.net/software , all maintained by Tim:

Package       Macports  Available
eye           1.4.1     1.4.1
missfits      2.2.11    2.8.0
scamp         1.6.2     2.0.1
sextractor    2.8.6     2.19.5
skymaker      3.3.3     3.10.5
stiff         1.12      2.4.0
stuff         1.19      1.26
swarp         2.17.6    2.38
weightwatcher 1.8.10    1.12

All except eye have been updated (​http://www.astromatic.net/2014/03/20/package-updates) and should be updated in Macports.

I looked through the Portfiles and I think it's just a matter of changing the version number of checksum in all cases (unless there are build errors with the new versions).

Tim, do you have time to do this soon or should I give it a try?

comment:4 Changed 10 years ago by cdeil (Christoph Deil)

I noticed one more thing ... the license of all these ports has been changed from CeCILL to GPL3+ ... so this also needs to be changed in the Portfile when doing the update.

See e.g. here:

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-missfits.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-sextractor.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-skymaker.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-stiff.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-stuff.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-swarp.diff added

Changed 10 years ago by cdeil (Christoph Deil)

Attachment: Portfile-weightwatcher.diff added

comment:5 Changed 10 years ago by cdeil (Christoph Deil)

Updated Portfiles are attached. All are simple updates and work for me on Mac OS X 10.9.

Could someone please review and if OK commit the updates?

comment:6 Changed 10 years ago by cdeil (Christoph Deil)

There's one more Astromatic tool calles psfex that's currently missing ... the request to add a Portfile for it is #43537 .

The updated Portfile for scamp is in #34746 because it fixes that issue.

Last edited 10 years ago by cdeil (Christoph Deil) (previous) (diff)

comment:7 Changed 10 years ago by cdeil (Christoph Deil)

I think these ports might be abandoned ... #35195 has been open for 22 month without a reply and is fixed here.

Could a Macports committer please review these patches?

Would it be helpful if I add "openmaintainer" for these ports in this issue or a separate one?

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

Have these ports been abandoned?

The patches look fine. I checked the license change upstream, and it is correct. I can test building these ports tonight, and commit the patches under maintainer timeout.

comment:9 in reply to:  8 Changed 10 years ago by mf2k (Frank Schima)

Replying to aronnax@…:

Have these ports been abandoned?

See #43755.

comment:10 Changed 10 years ago by lpsinger (Leo Singer)

Resolution: fixed
Status: newclosed

Addressed in r120317, r120318, r120319, r120320, r120321, r120322, and r120323.

I two minor modifications to the patches. First, it looked like the checksums were indented by an extra space; the prevalent style seems to be that the checksum type is aligned on a four-column grid, flush with the arguments of other keywords. I removed the extra space. Please check that the whitespace looks correct.

Second, I dropped the revision on swarp.

Thank you!

comment:11 in reply to:  10 Changed 10 years ago by lpsinger (Leo Singer)

Replying to aronnax@…:

I two minor modifications to the patches. First, it looked like the checksums were indented by an extra space; the prevalent style seems to be that the checksum type is aligned on a four-column grid, flush with the arguments of other keywords. I removed the extra space. Please check that the whitespace looks correct.

Oh, I see that some of the Portfiles themselves aren't 4-column aligned.

comment:12 Changed 10 years ago by cdeil (Christoph Deil)

Thanks for committing these patches!

About the 4-column alignment ... I copy & pasted to checksums with the alignment as they are printed on the console, because I didn't know if that would be preferable or whether I should align with the rest of the Portfile. Let me know if I should fix the alignment (and which way) and post patches here or if it's OK as is.

Note: See TracTickets for help on using tickets.