Hei, I created the new patch to fix license error reported by lintian. With this new patch, I have an error during package-building : (sorry, it's in french)
dpkg-source: info: construction de postr à partir de ./postr_0.13.orig.tar.gz patching file debian/copyright Reversed (or previously applied) patch detected! Skipping patch. 1 out of 1 hunk ignored dpkg-source: info: les décalages (« fuzz ») ne sont pas autorisés lors de l'application des correctifs dpkg-source: info: si le correctif « 05-fix_missing_license_paragraph.patch » est correctement appliqué par quilt, utiliser « quilt refresh » pour le mettre à jour dpkg-source: erreur: LC_ALL=C patch -t -F 0 -N -p1 -u -V never -g0 -E -b -B .pc/05-fix_missing_license_paragraph.patch/ --reject-file=- < postr.orig.oMZtof/debian/patches/05-fix_missing_license_paragraph.patch a produit une erreur de sortie de type 1 I don't understand why ? Do you have an idea ? Regards, Yoann 2014-06-11 14:55 GMT+02:00 Alexander Alemayhu <alexan...@bitraf.no>: > On Wed, Jun 11, 2014 at 01:56:59PM +0200, Yoann Gauthier wrote: > > Hello, > > > > Hei, > > > Okay, I will fix missing-license-paragraph-in-dep5-copyright gpl-3 during > > this week. > > > > To generate debian package you use : git-buildpackage ? And what is the > > complete lintian command ? > > > > Yes. git-buildpackage makes it easy to build a package. It also has some > nice > features like re-creating the upstream tarballs and much more. For lintian > i run 'lintian -EvIL +pedantic'. I am not sure how mentors.debian.net does > it. > > > Thanks, > > > > Yoann > > > > > > > > 2014-06-10 23:04 GMT+02:00 Alexander Alemayhu <alexan...@bitraf.no>: > > > > > On Tue, Jun 10, 2014 at 09:15:49PM +0200, Yoann Gauthier wrote: > > > > Hello, > > > > > > > > > > Hei, > > > > > > > I updated the changelog file as asked. Thanks for the push access, > it's > > > > easier for me to make changes now :). > > > > > > > > My changelog needs a review I think, because I am not familiar with > > > > changeLog conventions. > > > > > > > > > > It looks good IMO :) > > > > > > > If you have new task to delegate, I am available. > > > > > > > > > > After running lintian today a new warning appears: > > > W: postr source: missing-license-paragraph-in-dep5-copyright gpl-3 > > > (paragraph at line 3) > > > > > > The warning does appear on earlier commits aswell. Maybe something has > > > changed > > > in lintian, could you look into it? > > > > > > I talked with a DD and was recommended to fix the autotools warnings > > > reported by > > > lintian. He also proposed to handle the python bytecode generation in a > > > more > > > correct way. I will be looking at using dh-autoreconf to make the > package > > > free > > > of the autools warnings. It might take a couple of days... > > > > > > > > > > > Regards, > > > > > > > > Yoann > > > > > > > > > > > > 2014-06-10 7:57 GMT+02:00 Alexander Alemayhu <alexan...@bitraf.no>: > > > > > > > > > On Mon, Jun 09, 2014 at 08:29:51PM +0200, Yoann Gauthier wrote: > > > > > > Hello, > > > > > > > > > > > > > > > > Hei. > > > > > > > > > > > I fixed the desktop-entry-lacks-keywords-entry as asked. > > > > > > Find attached the patch. I used quilt to create my patch : > > > > > > 04-fix_desktop_keywords.patch and then git format to create patch > > > file. > > > > > > > > > > > > > > > > Good. You should also update the debian/changelog to include an > entry > > > which > > > > > mentions the patch. Please take a close look at the existing lines > to > > > get > > > > > an > > > > > idea of what to write. > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > PS : have access (push request) to your repository will be easier > > > for me. > > > > > > > > > > > > > > > > Full access granted. You should now be able to push directly. > > > > > > > > > > > Yoann > > > > > > > > > > > > > > > > > > 2014-06-03 22:50 GMT+02:00 Alexander Alemayhu < > alexan...@bitraf.no>: > > > > > > > > > > > > > On Tue, Jun 03, 2014 at 11:45:27AM +0200, Yoann Gauthier wrote: > > > > > > > > Hi Alexander, > > > > > > > > > > > > > > > > What are the next tasks I can do to help for the packaging ? > > > > > > > > > > > > > > > > > > > > > > You could look at the desktop-entry-lacks-keywords-entry[0] > > > lintian tag > > > > > > > and try > > > > > > > to fix it. I was planning on fixing it but wanted to > prioritize the > > > > > > > autotools > > > > > > > warnings. Hopefully we can get some advice from David. > > > > > > > > > > > > > > It looks like you would need to create a debian patch file to > > > modify > > > > > > > data/postr.desktop.in.in. If you are unfamiliar with quilt, > take a > > > > > look > > > > > > > at this > > > > > > > introduction[1]. > > > > > > > > > > > > > > PS: Remember to fetch the latest changes, before making new > > > changes. It > > > > > > > might > > > > > > > help in avoiding merge conflicts. > > > > > > > > > > > > > > PPS: desktop-entry-lacks-keywords-entry only occurs when using > > > `lintian > > > > > > > -EvIL > > > > > > > +pedantic` on my machine. > > > > > > > > > > > > > > [0]: > > > > > > > > > > http://lintian.debian.org/tags/desktop-entry-lacks-keywords-entry.html > > > > > > > [1]: > > > > > > > > > > > > > > > > > > > > > > > http://raphaelhertzog.com/2012/08/08/how-to-use-quilt-to-manage-patches-in-debian-packages/ > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > Yoann > > > > > > > > > > > > > > > > > > > > > > > > 2014-06-02 21:26 GMT+02:00 Alexander Alemayhu < > > > alexan...@bitraf.no>: > > > > > > > > > > > > > > > > > On Mon, Jun 02, 2014 at 10:30:40AM +0200, Yoann Gauthier > wrote: > > > > > > > > > > Hi, > > > > > > > > > > > > > > > > > > > > For man page license, I propose to distribute it under > GNU > > > GLP > > > > > (v3). > > > > > > > Do > > > > > > > > > you > > > > > > > > > > agree ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > I am no license expert, but I think it is okay assuming you > > > mean > > > > > GNU > > > > > > > GPL. > > > > > > > > > > > > > > > > > > > Okay for the maintener part, I will set the maintainer to > > > Python > > > > > > > > > > Applications Packaging Team. > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > Yoann > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2014-06-01 18:26 GMT+02:00 Alexander Alemayhu < > > > > > alexan...@bitraf.no>: > > > > > > > > > > > > > > > > > > > > > On Sun, Jun 01, 2014 at 03:50:27PM +0200, Yoann > Gauthier > > > wrote: > > > > > > > > > > > > Le 31 mai 2014 20:00, "Alexander Alemayhu" < > > > > > alexan...@bitraf.no> > > > > > > > a > > > > > > > > > > > écrit : > > > > > > > > > > > > > > > > > > > > > > > > > > On Sat, May 31, 2014 at 03:29:16PM +0200, Yoann > > > Gauthier > > > > > wrote: > > > > > > > > > > > > > > Hello, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hei, > > > > > > > > > > > > > > > > > > > > > > > > > > > I wrote the man page (attachment). I have no > access > > > to > > > > > the > > > > > > > > > repository > > > > > > > > > > > > > > today, I will upload the page to repository > tomorrow. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Nice. I decided to use github as the Vcs-* since > > > > > collab-maint > > > > > > > > > seems to > > > > > > > > > > > be > > > > > > > > > > > > for > > > > > > > > > > > > > people packing together with DDs. Maybe we should > ask > > > for > > > > > > > access > > > > > > > > > and > > > > > > > > > > > push > > > > > > > > > > > > our > > > > > > > > > > > > > changes there? > > > > > > > > > > > > > > > > > > > > > > > > I don't understand Vcs, what is it ? > > > > > > > > > > > > > > > > > > > > > > A akronym for Version Control System[0]. > > > > > > > > > > > > > > > > > > > > > > > > I think you have a typo 'MAINTENERS' should be > > > > > 'MAINTAINERS'. > > > > > > > It > > > > > > > > > could > > > > > > > > > > > be > > > > > > > > > > > > a > > > > > > > > > > > > > good idea to ask for others to review the man page. > > > Which > > > > > > > license > > > > > > > > > do > > > > > > > > > > > you > > > > > > > > > > > > want > > > > > > > > > > > > > to distribute the man page with? we have to > mention the > > > > > > > license in > > > > > > > > > > > > > debian/copyright. > > > > > > > > > > > > > > > > > > > > > > > > Agree for the typo, this evening I will read again > the > > > man > > > > > page > > > > > > > and > > > > > > > > > add > > > > > > > > > > > > more information maybe. Okay for the license, i will > > > think. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > In order to stay consistent with the debian/control > file we > > > > > should > > > > > > > set > > > > > > > > > the > > > > > > > > > > > maintainer to Python Applications Packaging Team > > > > > > > > > > > <python-apps-t...@lists.alioth.debian.org> > > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yoann > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [0]: http://en.wikipedia.org/wiki/Version_control > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2014-05-30 12:36 GMT+02:00 Yoann Gauthier < > > > > > > > > > yoann.gauthi...@gmail.com > > > > > > > > > > > >: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Alexander, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yes, I am going to write the man page. It will > be > > > > > finished > > > > > > > > > today or > > > > > > > > > > > > > > > tomorrow. > > > > > > > > > > > > > > > Okay for the RFS. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yoann > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2014-05-30 10:43 GMT+02:00 Alexander Alemayhu < > > > > > > > > > alexan...@bitraf.no > > > > > > > > > > > >: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hei Yoann, > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> below is a message I recived after > reuploading to > > > > > > > > > > > mentors.debian.net. > > > > > > > > > > > > > > >> The warnings I mentioned earlier are visible > in > > > the > > > > > > > package > > > > > > > > > page. > > > > > > > > > > > > > > >> Are you going to write a man page? I would > like to > > > > > send a > > > > > > > RFS > > > > > > > > > > > after > > > > > > > > > > > > fixing > > > > > > > > > > > > > > >> some more of the warnings to the PAPT list. > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> ----- Forwarded message from " > mentors.debian.net" > > > < > > > > > > > > > > > > > > >> supp...@mentors.debian.net> ----- > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> Date: Wed, 28 May 2014 21:33:31 +0000 (UTC) > > > > > > > > > > > > > > >> From: "mentors.debian.net" < > > > > > supp...@mentors.debian.net> > > > > > > > > > > > > > > >> To: alexan...@bitraf.no > > > > > > > > > > > > > > >> Subject: postr uploaded to mentors.debian.net > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> Hi. > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> Your upload of the package 'postr' to > > > > > mentors.debian.net > > > > > > > was > > > > > > > > > > > > > > >> successful. Others can now see it. The URL of > your > > > > > > > package is: > > > > > > > > > > > > > > >> http://mentors.debian.net/package/postr > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> The respective dsc file can be found at: > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > http://mentors.debian.net/debian/pool/main/p/postr/postr_0.13-1.dsc > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> If you do not yet have a sponsor for your > package > > > you > > > > > may > > > > > > > > > want to > > > > > > > > > > > go > > > > > > > > > > > > to > > > > > > > > > > > > > > >> > > > http://mentors.debian.net/sponsors/rfs-howto/postr > > > > > > > > > > > > > > >> and set the "Seeking a sponsor" option to > > > highlight > > > > > your > > > > > > > > > package > > > > > > > > > > > on > > > > > > > > > > > > the > > > > > > > > > > > > > > >> welcome page. > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> You can also send an RFS (request for > > > sponsorship) to > > > > > the > > > > > > > > > > > > debian-mentors > > > > > > > > > > > > > > >> mailing list. Your package page will give your > > > > > > > suggestions on > > > > > > > > > how > > > > > > > > > > > to > > > > > > > > > > > > > > >> send that mail. > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> Good luck in finding a sponsor! > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> Thanks, > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> -- > > > > > > > > > > > > > > >> mentors.debian.net > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> ----- End forwarded message ----- > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > From 6f2a4a41373d5b029e552fbf47fecd944bf6a25d Mon Sep 17 00:00:00 > > > 2001 > > > > > > From: Yoann Gauthier <yoann.gauthi...@gmail.com> > > > > > > Date: Mon, 9 Jun 2014 20:24:03 +0200 > > > > > > Subject: [PATCH 2/2] Fix desktop-entry-lacks-keywords-entry > lintian > > > tag > > > > > > > > > > > > --- > > > > > > debian/patches/04-fix_desktop_keywords.patch | 13 > +++++++++++++ > > > > > > debian/patches/series | 1 + > > > > > > 2 files changed, 14 insertions(+) > > > > > > create mode 100644 debian/patches/04-fix_desktop_keywords.patch > > > > > > > > > > > > diff --git a/debian/patches/04-fix_desktop_keywords.patch > > > > > b/debian/patches/04-fix_desktop_keywords.patch > > > > > > new file mode 100644 > > > > > > index 0000000..9e5a24c > > > > > > --- /dev/null > > > > > > +++ b/debian/patches/04-fix_desktop_keywords.patch > > > > > > @@ -0,0 +1,13 @@ > > > > > > +Description: Fix Desktop Keyword warning > > > > > > + Fix desktop-entry-lacks-keywords-entry lintian tag for clean > > > package > > > > > > +Author: Yoann Gauthier <yoann.gauthi...@gmail.com> > > > > > > +--- > > > > > > +This patch header follows DEP-3: > http://dep.debian.net/deps/dep3/ > > > > > > +--- a/data/postr.desktop.in.in > > > > > > ++++ b/data/postr.desktop.in.in > > > > > > +@@ -13,3 +13,4 @@ > > > > > > + X-GNOME-Bugzilla-Product=postr > > > > > > + X-GNOME-Bugzilla-Component=general > > > > > > + X-GNOME-Bugzilla-Version=@VERSION@ > > > > > > ++_Keywords=flickr;upload;picture;postr;image;tool;gnome; > > > > > > +\ No newline at end of file > > > > > > diff --git a/debian/patches/series b/debian/patches/series > > > > > > index a78f286..bc61a4f 100644 > > > > > > --- a/debian/patches/series > > > > > > +++ b/debian/patches/series > > > > > > @@ -2,3 +2,4 @@ > > > > > > 01-fix_encoding.patch > > > > > > 02-fix-glade-path.patch > > > > > > 03-handle-python-bytecode-compilation.patch > > > > > > +04-fix_desktop_keywords.patch > > > > > > -- > > > > > > 1.7.10.4 > > > > > > > > > > > > > > > > > > > >
