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

Re: RFS: kplayer



On Martes 26 Mayo 2009 14:45:23 Patrick Matthäi escribió:
> David Palacio schrieb:
> > On Sábado 16 Mayo 2009 04:40:17 Patrick Matthäi escribió:
> >> David Palacio schrieb:
> >>>>> - URL: http://mentors.debian.net/debian/pool/main/k/kplayer
> >>>>
> >>>> * debian/control:
> >>>> - The bugs field is useless here
> >>
> >> Now format is useless
> >>
> >>>> - Please add the homepage field
> >>
> >> There is a doubled http://
> >>
> >>>> * debian/copyright:
> >>>> - You may leave your own copyright there
> >>>> - The files in doc/ are licensed with the GFDL, see COPYING-DOCS
> >>>
> >>> They are not being installed. Or do you want them out of sources too?
> >>
> >> If they are in the tarball, they have to be listed in copyright.
> >>
> >> Also here is another warning:
> >> P: kplayer: experimental-to-unstable-without-comment
> >>
> >> You may add a ITP or overtake an existing and close it with your firstz
> >> initial debian upload.
> >>
> >> You also want to add a menu file.
> >
> > Done.
> >
> > Thanks.
>
> * debian/control:
> - You suggest a non-existing package kplayer-doc.
>
> * debian/menu:
> - The icon file is missing there, it should be a xpm. If no .xpm is
> available, you may simply generate it with gimp.
>
> * debian/patches/*
> - Have you sent them to upstream?
Those already are fixed upstream, as noted in the diff comments.

> * debian/copyright:
> There are still bigger issues for which the package will get a reject.
> 1) I can not see from your copyright which files/dirs are licensed under
> the terms of the GPLv3+ and which are licensed unter the GFDL.
> 2) I don't believe that every copyright holder has his copyleft in *all*
> files.
>
> Write it e.g. this way:
> http://packages.debian.org/changelogs/pool/main/k/kdenlive/current/copyrigh
>t
>
> Or much better follow the proposal copyright format:
> http://wiki.debian.org/Proposals/CopyrightFormat
I did it instead pkg-kde style (I have worked there a little). But it still 
addresses your two points.


Reply to: