---------- Forwarded message --------- From: Ko Ko Ye` <kokoye2...@gmail.com> Date: Fri, Jun 5, 2020 at 9:32 AM Subject: Re: Bug#961899: RFS: wifi-qr/0.1-1 -- WiFi Share and Connect with QR To: Boyuan Yang <073p...@gmail.com>
Thank so much Boyuan On Fri, Jun 5, 2020 at 8:42 AM Boyuan Yang <073p...@gmail.com> wrote: > Control: tags -1 +moreinfo > > Hi kokoye, > > Please find my comments below: > > Ko Ko Ye` <kokoye2...@gmail.com> 于2020年5月30日周六 下午11:57写道: > > > > Package: sponsorship-requests > > Severity: wishlist > > > > Dear mentors, > > > > I am looking for a sponsor for my package "wifi-qr" > > > > * Package name : wifi-qr > > Version : 0.1-1 > > Upstream Author : kokoye2007 <kokoye2...@gmail.com> > > * URL : https://github.com/kokoye2007/wifi-qr > > * License : GPL-3.0+ > > * Vcs : https://github.com/kokoye2007/wifi-qr > > Section : utils > > > > It builds those binary packages: > > 1. Please do *not* submit duplicated RFS bugs. You submitted multiple RFS > requests and the duplicated ones have already been closed by the bartm bot > as you can see in [2]. If you need to make any modifications, please use > the > original bug report and add more information by replying to the report > email > address. > > [2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=962169;msg=7 when i make changelog with ITP Close, have some error. so i am test with new RFS. now follow with #961899 <https://bugs.debian.org/961899> > > > 2. I saw your packaging repo at https://github.com/kokoye2007/wifi-qr. > Please > drop the /.pc directory. This directory is created internally during > package building > and is not supposed to appear anywhere, anytime, especially in your > packaging repo. > Keeping it will cause issues when building the package. > i am upload after function updated. so confuse and make a patch. now clean and fix. > > 3. Since you yourself is the upstream of this software, I don't quite > get it when > you put another patch file at > /debian/patches/remove-duplicate-function. Apparantly > it is not a Debian-specific patch. It is supposed to be applied > directly onto your > source code instead of appearing as a patch for Debian. In this case you > might > should act as the upstream and release a new version of your software, > not as the > packager applying a patch. > same with case 2. > > 4. The #DEBHELPER# placeholder should never appear in the debian/rules > file. > It is the placeholder to be used in the preinst/postinst/prerm/postrm > maintenance > scripts, not the debian/rules Makefile. > > 5. Even in the preinst/postinst scripts, the #DEBHELPER# placeholder should > appear at the bottom of the scripts. > > 6. I highly doubt whether manually invoking update-icon-caches is useful > at all. > I believe it will be handled by the dpkg triggers automatically. Maybe > you do not > need to add any preinst/post scripts at all for this package. > > Thank for very clear message. its fix. > I believe the 6 issues above need to be solved before having this > package uploaded. > Thanks! > > Please kindly check for me. now its available at https://mentors.debian.net/package/wifi-qr https://mentors.debian.net/debian/pool/main/w/wifi-qr/wifi-qr_0.1-1.dsc with regards and respects -- > Regards, > Boyuan Yang > -- with regards *Ko Ko Ye`* +95 97989 22022 +95 94500 22022 +95 9731 47907 kokoye2...@gmail.com kokoye2...@ubuntu.com skype: kokoye2007 jitsi: kokoye2007 http://ubuntu-mm.net http://wiki.ubuntu.com/kokoye2007 http://wiki.ubuntu.com/MyanmarTeam http://loco.ubuntu.com/teams/ubuntu-mm -- with regards *Ko Ko Ye`* +95 97989 22022 +95 94500 22022 +95 9731 47907 kokoye2...@gmail.com kokoye2...@ubuntu.com skype: kokoye2007 jitsi: kokoye2007 http://ubuntu-mm.net http://wiki.ubuntu.com/kokoye2007 http://wiki.ubuntu.com/MyanmarTeam http://loco.ubuntu.com/teams/ubuntu-mm