[pkg-php-pear] RFS: php-pubsubhubbub-publisher
James Valleroy
jvalleroy at mailbox.org
Fri Jan 24 01:49:03 GMT 2020
Hi David,
On 1/23/20 7:49 AM, David Prévot wrote:
> Hi James and all,
>
> Le 15/01/2020 à 02:48, James Valleroy a écrit :
>
>> Please review and sponsor php-pubsubhubbub-publisher:
>
> Thank you for your work. I’m OK to upload it as-is, but I wonder if
> you’d like to use a lower version string before doing so (since that’s
> the one thing one can’t decrease once the package is in).
>
> A few details:
>
> Did you forget to push your upstream/ (signed) tag?
I don't have an upstream/ tag yet, because I didn't run gbp import-ref (debian/master was directly branched from upstream).
I can do this after changing the version number.
> A d/gbp.conf file would be useful since you use a non-default gbp scheme
> (debian stuff in a d/master branch instead of master).
Added.
> Don’t you need php-cli rather than php for CI?
Yes, this will result in fewer packages pulled in. I have made this change.
> The upstream version you chose (0.0~git20181009.047b0fa) ranks higher
> than 0 or 0., why not simply use 0~something (or even 0~~something if
> you want to be extra careful)?
> Also, it looks like a long string: I don’t believe providing a short git
> hash in the version string is useful (why not simply provide it inside
> d/changelog?). Instead, it increases the data lentgh in every Debian box
> (via its Package file) for a use case I doubt actually exists (and yes,
> it’s your choice in the end ;). OK, it’s not as crazy as what some
> people do in the nodejs world (e.g.,
> 6.2.1+ds+~0.4.0+~4.0.0+really4.0.0+~1.0.0+~5.0.1+ds+~1.7.0+ds+~0.1.1+~0.3.1+~0.2.0+~0.1.0+~0.3.0+~0.3.0-5
> for acorn ;), I think they even managed to break the upload queue not so
> long ago.
Yes, I can change this. I didn't find any guidelines for picking a version number, so I tried to match uscan's mode=git default.
Do you think I should keep the commit date in the version, like 0~~20181009? I can have uscan match this format with the "pretty" option.
> The upstream README.md could be shipped (I don’t mind either way).
Added to d/docs.
Thanks for reviewing!
--
James
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/pkg-php-pear/attachments/20200123/c9a72dc9/attachment-0001.sig>
More information about the pkg-php-pear
mailing list