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

Re: Bug#773611: Fwd: RFS: histring/1.1.1-1 [ITP] - general purpose highlighting tool



On 21/12/14 07:02, Tong Sun wrote:
> Package: sponsorship-requests
> Severity: wishlist
> 
> Dear mentors,
> 
> Please sponsor for my newly uploaded package "histring".
> 
> It's a general purpose highlighting tool that works with arbitrary
> input text file (e.g. all sorts of log files), and highlight different
> keywords in different colours (e.g., warnings and errors are
> highlighted with different colours). You can use this single small
> tool to replace all kinds of special purpose log parsing tools, and it
> can do much more, like highlighting results from vcsdiff, etc. E.g.,
> 
> http://sfxpt.wordpress.com/2013/06/02/highlighting-strings-in-text-output-with-histring/
> 
> Once again, I am looking for a sponsor for my newly upgraded package
> "histring". It's *lintian clean*, and without any build problems. Ref
> http://mentors.debian.net/package/histring
> 
>  * Package name    : histring
>    Version         : 1.1.1-1
>   * URL             : https://github.com/suntong001/histring
>  * License         : GPL-2.0+
>    Section         : utils
> 
>   It builds those binary packages:
> 
>     histring   - highlight strings using ANSI terminal escape sequences
> 
>   To access further information about this package, please visit the
> following URL:
> 
>   http://mentors.debian.net/package/histring
> 
> 
>   Alternatively, one can download the package with dget using this command:
> 
>     dget -x http://mentors.debian.net/debian/pool/main/h/histring/histring_1.1.1-1.dsc
> 
>   More information about hello can be obtained from
> 
> http://sfxpt.wordpress.com/2014/11/30/use-new-dbab-to-set-proxy-automatically/#advantages;.
> 
> Thanks &
> Best Regards,
> 
> Tong Sun

Hi! I'm not a DD, so I can't sponsor your package, but here are some
things that I've noticed:

* It may be a good idea to make releases as tarballs still, instead of
just making Debian releases, so that other distros can benefit as well.

* As far as I can see, you can fix "deprecated-configure-filename". All
you would need to do is change the name of configure.in to configure.ac.

* Unless you have a compelling reason, the priority of the package (in
d/control) should be "optional", not "extra".

* This isn't important, but it would be nice if you put the histring
manpage outside of the Debian directory when you make your next upstream
release. This way, other distros and people who build from source will
also benefit from the manpage.

* There are a few minor grammatical errors in histring --help:
	"useage" should be "usage"
	"your self" should be "yourself"
	"hilighting" should be "highlighting"

Good luck getting your package into Debian!

Riley Baird


Reply to: