[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: daisy-player (next try, new upstream version)


Paul Gevers, le Sun 06 Feb 2011 20:11:10 +0100, a écrit :
> On 01/30/11 17:15, Samuel Thibault wrote:
> > - daisy-player.desktop changes should be forwarded upstream.
> Done, and included in the latest release (still needs a typo patch thou).

That's fine to patch in your package, provided that it's submitted
upstream and will be trivially fixed with next upstream upload.

> > - you should ask upstream to remove -s from LDLIBS, and rather use
> >   install -s instead. Bonus: when building without installing, you have
> >   the debugging symbols.
> Forwarding done (in the mean time I patch the Makefile to not use the -s).


> > - you should ask upstream to use a -DPREFIX=/usr/local instead of
> >   hardcoding it.
> Forwarding done. I thought about this before, but I preferred that he
> fixed other issues first.

Right :)

> >> Known issues (comments welcome):
> >> - I have asked upstream to add license and copyright notices to the
> >> translation files.
> > 
> > Indeed, that's missing. Do you know if the notices are just missing and
> > debian/copyright is actually accurate already?
> Upstream has added the notices. My d/copyright was already accurate.

Ok, good!

> >> - I have asked upstream to move the .daisy-player.rc file to the already
> >> existing .daisy-player directory.
> > 
> > I don't see this?
> The file is created when you run daisy-player, so it is not present in
> the packaging. Upstream didn't respond to this request yet.

Ah, ok.

> >> - Upstream ships the icons in a zipped tar file. Should I ask him to
> >> ship them directly (in a sub-directory for instance) or is unzipping at
> >> build time also fine?
> > 
> > Is there really a reason for them to ship it in a tarball? If not,
> > they'd rather just unpack it. Else it's not too bad to have it as a
> > tarball.
> It is now unpacked in the upstream tarball.

Good.  Actually it might be useful you know that source format
version 3 supports several upstream tarballs, see the festvox-mbrola for
an example.

> >> - Get an agreement with upstream on the description of daisy-player in
> >> the desktop and manual file.
> > 
> > I guess it is related with my comment above.
> Upstream uses my description, so: done.


> > The only barring point is whether debian/copyright has proper
> > information for the translation files. The rest should be OK for a first
> > upload.
> Good to hear.
> I have updated the package with the latest upstream release which fixes
> the issues mentioned above. Available from mentors and alioth:

I'll have a look and probably upload.


Reply to: