[pkg-php-pear] RFS: php-nesbot-carbon

Robin Gustafsson robin at rgson.se
Mon Apr 13 16:29:44 BST 2020


Hi David,

Thank you for the speedy upload and the great feedback. Much appreciated.

I've fixed your remarks and prepared a new version on Salsa [1].


> - I notice there is a more recent upstream version (2.32.2), why not
>   using the latest version?

No reason. I just failed to keep up with upstream. Fixed.

> - You should include the bin/carbon executable that is now provided
>   upstream (eventually changing the executable name, that sound very
>   generic, even there are none of the same name yet in Debian).

I included it as "nesbot-carbon". It's missing a man page though. I'll
file a bug myself once it's uploaded and contact upstream, in
accordance with policy.

>   + please drop the version on pkg-php-tools, it’s already satisfied
>     since before oldoldstable.

For my own future reference, is "drop the version [if] it's satisfied
since before oldoldstable" a general rule?

>   + why do you use a license for debian/ that is incompatible with
>     upstream? (Changed in be6b8f7c4d0a6259a6bcf347b1e89bf7c504fe07, no
>     indication that the initial author agrees).

I thought GPL-2+ was preferred for debian/*, probably because it's
dh_make's default. I have no personal preference, so I've now changed
it back.

As for the initial author's agreement, I believe the MIT license
itself would allow me to sublicense under GPL.

> - You could base your next update on upstream Git repository.

I prefer not to intertwine the repo histories.

> - Two tests are skipped with the following rationale:
>
>         In Symfony < 4, NotLocaleAwareException will never been thrown.
>
>   yet php-symfony-translation version 4.4 is used, so it needs some
>   investigation to understand what is going wrong.

It should say "Symfony < 5". It's now patched upstream and will be
fixed in their next release.


[1] https://salsa.debian.org/php-team/pear/php-nesbot-carbon

Regards,
Robin



More information about the pkg-php-pear mailing list