
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"><html><head><meta content="text/html;charset=UTF-8" http-equiv="Content-Type"></head><body ><div style="font-family: Verdana, Arial, Helvetica, sans-serif; font-size: 10pt;"><div>Hi David,<br></div><div><br></div><div>Thank you for having a look !<br></div><div><br></div><div>> (it sounds like you have everything <br></div><div>under control)<br></div><div><br></div><div>This is true, everything builds and works fine ^^<br></div><div>Uploading to experimental is a very good idea, if anyone has my message in time I would need someone to enable me ACLs to the packages I maintain in order to be able to upload.<br></div><div>I have no idea why since I am a DM uploading does not work and gets rejected each time saying I do not have access.<br></div><div><br></div><div>Yes, the tests run fine and GitLab CI is running autopkgtests so I am sure everything works.<br></div><div>I pushed fixes for phpunit 9 on the google-recaptcha package and again the GitLab CI validated your fix for d/latest and b-backports.<br></div><div><br></div><div>> am not used to deal with Salsa webui (but I guess I can learn)<br></div><div><br></div><div>Well, it is quite cool. It feels like a GitHub and using issues and MR(PR) and the CI makes the packaging experience just perfect.<br></div><div><br></div><div>Also, I pushed a fix to make "php-twig (<< 4~~)," just to be sure next major will not be used by mistake. And dropped the useless restriction diff on our internal package.</div><div><br></div><div>Thanks for the feedback on the metro theme, removing it or fix lintian-overrides is the question now.<br></div><div>The file debian/phpmyadmin.lintian-overrides has some overrides and <a target="_blank" href="https://nm.debian.org/person/yadd/">https://nm.debian.org/person/yadd/</a> pointed our that I should fix them.<br></div><div>But I have no idea how It should be done.<br></div><div>Overrides: font-in-non-font-package, font-outside-font-dir, truetype-font-prohibits-installable-embedding<br></div><div><br></div><div>So basically If I would make a checklist it would be:<br></div><div>- [ ] Have a DD let me know the package is Debian correct<br></div><div>- [ ] Find out how to fix the font-* overrides<br></div><div>- [ ] Be able to upload or have an uploader<br></div><div>- [ ] Link some more JS packages</div><div><br></div><div class="zmail_extra" data-zbluepencil-ignore="true" style=""><div>Thank you for your help<br></div><div><br></div><div>William</div><div id="Zm-_Id_-Sgn1">---- Le mer., 30 déc. 2020 20:03:45 +0100 <b>David Prévot <taffit@debian.org></b> écrit ----<br></div><div><br></div><blockquote style="margin: 0px;"><div>Hi William, <br> <br>Le 30/12/2020 à 06:33, William Desportes a écrit : <br> <br>> I lack reviews and would need some expert eyeballs to have a look to <br>> <a href="https://salsa.debian.org/phpmyadmin-team/phpmyadmin/-/merge_requests/30" target="_blank">https://salsa.debian.org/phpmyadmin-team/phpmyadmin/-/merge_requests/30</a> <br>> I hope you can help me :) <br> <br>I just skimmed at the MR and the content of d/patches and am not sure <br>what you are actually expecting (it sounds like you have everything <br>under control). I’ve not tested the resulting package either but if you <br>did, what is preventing you from an upload, at least to experimental in <br>order to gather more feedback? I’ve also checked that the build time <br>tests look fine with PHPUnit 9, so that’s a relief ;). <br> <br>Just a remark: simplify-composer-or-versions.patch seems currently <br>useless: only php-twig-i18n-extension version 3 seems currently <br>available, and if both php-twig 2 (from testing/unstable) and 3 (from <br>experimental are usable, no version should be needed. OK, in order to <br>allow smooth upgrade easier backports, “>2.9” may be worth it (but <br>preventing the use of php-twig 3 seems counter-productive unless you’re <br>already aware of existing issues), making it “>2.9, <4” should translate <br>properly the current “^2.9 || ^3” without losing information when <br>handled by dh_phpcomposer (as long as #765899 is not fixed). <br> <br>> Known issues: <br>> - Metro theme has some fonts that I found out after another dev told me <br>> Apache2 was not GPL2 compatible as beeing Apache2 files (extreme <br>> possible solution: drop the theme on Debian): <br>> <a href="https://github.com/phpmyadmin/phpmyadmin/pull/16547" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/16547</a> <br> <br>I’m not sure the issue is actually relevant: both licenses are DFSG <br>compliant, and you’re not building a binary file with content from both <br>license (the usual headache with incompatible licenses). As long as <br>you’re providing content via separate files with a properly documented <br>d/copyright, you’re as free to do it in the phpmyadmin package as the <br>debian-cd team is free to provide packages from both licenses in the <br>same ISO file. <br> <br>> Please let me know what you think about that <br> <br>I’ve just subscribed to the [team] list, am not used to deal with Salsa <br>webui (but I guess I can learn), but please point mo to the relevant <br>discussion media you’re using if I can help. <br> <br>team: <a href="https://tracker.debian.org/teams/phpmyadmin-team/" target="_blank">https://tracker.debian.org/teams/phpmyadmin-team/</a> <br> <br>Regards <br> <br>David <br></div></blockquote></div><div><br></div></div><br></body></html>