Bug#711833: [support@mentors.debian.net: postr uploaded to mentors.debian.net]
On Thu, Jun 12, 2014 at 12:22:25AM +0200, Yoann Gauthier wrote:
> Hei,
>
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
>
>
Taking a quick look at your changes it looks like you have fuzz in the patches
and according to dpkg fuzz is not allowed. Try updating the patch file.
Please do not push changes which are obviosly broken to the repository! It is
much better if you do things locally and only push when you see it works as
expected.
>
> I don't understand why ? Do you have an idea ?
>
> Regards,
>
> Yoann
>
>
> 2014-06-11 14:55 GMT+02:00 Alexander Alemayhu <alexander@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 <alexander@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 <alexander@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 <
> > alexander@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 <
> > > > alexander@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 <
> > > > > > alexander@bitraf.no>:
> > > > > > > > > > >
> > > > > > > > > > > > On Sun, Jun 01, 2014 at 03:50:27PM +0200, Yoann
> > Gauthier
> > > > wrote:
> > > > > > > > > > > > > Le 31 mai 2014 20:00, "Alexander Alemayhu" <
> > > > > > alexander@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-team@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.gauthier9@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 <
> > > > > > > > > > alexander@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"
> > > > <
> > > > > > > > > > > > > > > >> support@mentors.debian.net> -----
> > > > > > > > > > > > > > > >>
> > > > > > > > > > > > > > > >> Date: Wed, 28 May 2014 21:33:31 +0000 (UTC)
> > > > > > > > > > > > > > > >> From: "mentors.debian.net" <
> > > > > > support@mentors.debian.net>
> > > > > > > > > > > > > > > >> To: alexander@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.gauthier9@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.gauthier9@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
> > > > > > >
> > > > > >
> > > > > >
> > > >
> >
Reply to: