sextractor: please update to new version 2.19.5

[cdeil (Christoph Deil)]

Could you please update SExtractor to the new version 2.19.5 ?

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

[cdeil (Christoph Deil)]

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

[mf2k (Frank Schima)]

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

[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?

[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:

• ​http://astromatic.iap.fr/wsvn/public/filedetails.php?repname=public+software.sextractor&path=%2Fbranches%2Fmulti%2FLICENSE
• ​https://lists.fedoraproject.org/pipermail/devel/2014-April/197478.html

[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?

[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.

[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?

[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.

[mf2k (Frank Schima)]

Replying to aronnax@…:

Have these ports been abandoned?

See #43755.

[lpsinger (Leo Singer)]

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!

[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.

[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.