xraylib 2.16.0_1: missing license

[ci42]

Please state the license in the portfile (license keyword). If permitted by the license MacPorts provides binary packages for Snow Leopard and Lion.

[Tom.Schoonjans@…]

I've included the license information (and fixed a small bug in the port file).

Best,

Tom

[ci42]

Thanks, r97991. Binary packages are now available: ​http://packages.macports.org/xraylib/

[Tom.Schoonjans@…]

Thanks!

The binary packages do not seem to carry the correct revision number though: I think it should be 2...

[ci42]

The revision number comes directly from the portfile. So revision 1 is correct.

[Tom.Schoonjans@…]

Replying to ciserlohn@…:

The revision number comes directly from the portfile. So revision 1 is correct.

Oh, it appears you took the first Portfile.diff... I uploaded two: only the second one has the correct revision number (saw it too late myself). I'm not sure it really matters though since no rebuild is necessary

[ci42]

Replying to Tom.Schoonjans@…:

Replying to ciserlohn@…:

The revision number comes directly from the portfile. So revision 1 is correct.

Oh, it appears you took the first Portfile.diff... I uploaded two: only the second one has the correct revision number (saw it too late myself). I'm not sure it really matters though since no rebuild is necessary

The revision must be increased only if the files the port installs change or a rebuild should be enforced (e.g if a port/lib linked against has an incompatible change). Neither is the case with your changes, so we can stay with revision 1.