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

Bug#663916: New phonetisaurus package available

* Giulio Paci <giuliopaci@gmail.com>, 2012-10-09, 02:16:

The ugly hack in debian/rules is indeed ugly. :)

Could you split it into multiple lines? 410 characters is really too long.

Why is the filename between two -e expressions?

Last but not least, why do you need to recover this file? It looks like it shouldn't have been included in the upstream tarball in the first place.

Oh, and Google sparse hash implemented is already packaged in Debian. Please build-depend on libsparsehash-dev and make sure that the system-wide copy is used, not the bundled one.

Packaging the UTF-8 library (currently in src/3rdparty/utf8/) might be also worth considering, in order to comply with Policy §4.13.

Jakub Wilk

Reply to: