Hi Guido,
On Sun, Nov 23, 2008 at 19:48, Guido Loupias <[EMAIL PROTECTED]> wrote:
> Hey Sandro, I've updated the package. Could you take a look at it?
> Thanks.
- bug #488396 should be an ITA not an ITP (so the changelog has to be updated)
- ok, you took too serious to be verbose in the debian/changelog :)
let's find a compromise like: remove all the depends package added
(it's enough you explain that new deps are added since it's a new app
now), add the info about removed files from upstream tarball in
README.source (I think it's best to use it)
- debian/README.source is not we expect to be; first you don't need to
state the copyright/license (it's debian/copyright to explain the
packaging legal stuff), and we expect to find in this file the
information to "use" the source package that *differs* from the normal
way; so, no need to say "use dpkg-source -x .dsc" to obtain the debian
package, we know how to do it since it's the normal way to do it. What
I expect is something like "we manage to patch the upstream source
code with dpatch, take a look at dpatch docs in /usr/share/doc/dpatch;
to obtain the upstream tarball used to prepare this package, run
./debian/rules get-orig-source". And this is the place were to
document what file are removed from upstream checkout.
- how are you putting config.{sub,guess} into orig tarball?
- did you send the manpage upstream?
Pretty easy to fix, so we are really near to upload it :)
Kidnly,
--
Sandro Tosi (aka morph, Morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
