On Mon, 14 Jun 2010 10:14:32 +0200 Jonathan Michalon <johndescs@gmail.com> wrote: > Le Mon, 14 Jun 2010 04:41:42 +0800, > Chow Loong Jin <hyperair@ubuntu.com> a écrit : > > Yes, it does appear that it is required by GeanyGenDoc, and I've > > taken a look at the package though I can't upload it. Some comments: > > > > * Please don't run autogen.sh, the configure script itself looks > > sufficient. Your debian/rules should just have the %: rule, then. > > * The stuff in README.Debian should be in README.source, but with > > reference to the previous point, please remove it. > > * With reference to the first point, please also drop > > autoconf/automake dependencies, but keep autotools-dev for > > config.guess and config.sub. > > * Drop debian/*.dirs, they're unnecessary. > > * Drop lintian-overrides, that's also unnecessary. > > > > I'm CC-ing Evgeni; he sponsors my geany-plugins packages and I'm > > sure he'd be interested to look at ctpl, especially if there will > > be no geany-plugins 0.19 binaries without ctpl. ;-) > > Hi, > > Thank you for your comments! > I've followed all of your hints even if I'm very surprised that the > advices from autotools-dev should no more be taken into account. > New version available at the same URLs (no debian version bump as it > didn't get upladed before). The configure script and provided Makefile.in's should be used if possible, failing which autoconf/automake should be run to re-build these. For packages which are properly created via "make distcheck" that don't have any patches that touch the Makefile.am/configure.ac files, you shouldn't regenerate the configure and Makefile.in files using the method you used. The advice in autotools-dev is only for the proper way of going about regenerating these files, in the event that you've made changes that need to be propagated over. > > CC-ing you and our providential sponsor as I don't know whether you > are subscribed to bug report or not. > [[ It's evgeni@debian.org, I for it wrong the first time. ]] -- Kind regards, Chow Loong Jin
Attachment:
signature.asc
Description: PGP signature