<div dir="ltr"><div>Hi David,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 2 mars 2020 à 12:30, David Prévot <<a href="mailto:david@tilapin.org">david@tilapin.org</a>> a écrit :</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I doubt you tested your package,</blockquote><div><br></div><div>I tested it but not enough. :-(</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> it can’t even be uninstalled. You<br>
should at least use debian/pkg-php-tools-overrides to set the<br>
appropriate php-email-validator package name, drop the<br>
php-symfony-polyfill* one(s), and set the proper dependency instead. You<br>
may also need to loud your dependencies.<br>
<br>
I also looked quickly at debian/ before building the package, and there<br>
is also some cleanup needed IMHO:<br>
- please, run the upstream test suite (at build time and in CI),<br>
- you probably should install the doc rather than the tests in u/s/d,<br>
- MIT should probably be Expat (d/copyright),<br>
- debian/dirs seems useless,<br>
- transitional package seems unneeded since at least Jessie;<br>
- reference to a pkg-php-tools version satisfied since even before that.<br>
- what is the point of “Depends: php-cli”?<br>
- ditto for “Suggests: libapache2-mod-php | php”?<br>
- UNRELEASED is not a suitable version (d/changelog).<br>
<br>
I haven’t looked at the upstream part yet.<br>
<br></blockquote><div><br></div><div>Thanks for taking time to review my packaging. I will work on it.</div><div><br></div><div>Regards,</div><div>Nicolas</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Regards<br>
<br>
_______________________________________________<br>
pkg-php-pear mailing list<br>
<a href="mailto:pkg-php-pear@alioth-lists.debian.net" target="_blank">pkg-php-pear@alioth-lists.debian.net</a><br>
<a href="https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-php-pear" rel="noreferrer" target="_blank">https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-php-pear</a></blockquote></div></div>