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

Re: RFS: ldtp (updated package)



On Sat, Jun 14, 2008 at 1:00 PM, Vincent Bernat <bernat@debian.org> wrote:
> Hi Kartik!
>
> You copy config.sub and config.guess from autotools-dev as stated in the
> README.Debian of this package. However, copying those files in the clean
> target changes the diff.gz. Instead,  you might want to add this snippet
> in the config.status target:
<snip>

Done. Thanks.

> Moreover, you can safely remove CFLAGS settings in debian/rules. This is
> now done by dpkg-buildpackage.

Done. Thanks.

> We  try to  avoid an  uppercased letter  at the  beginning of  the short
> description. This will be difficult to avoid for ldtp package, but maybe
> you might find a way to reword the short description of python-ldtp.

Lintian will warn about spelling mistake if Python is renamed to
python. I have reword other part of short description.

Thanks.

> You  can  also update  Standards-Version  to  3.8.0  which was  released
> recently.

Done.

Updated package can be found at,
http://mentors.debian.net/debian/pool/main/l/ldtp/ldtp_1.1.0-1.dsc

:)

-- 
 Cheers,
 Kartik Mistry | 0xD1028C8D | IRC: kart_
 Blogs: {ftbfs,kartikm}.wordpress.com


Reply to: