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

Bug#696782: RFS: sequitur-g2p/0.0.r1668-1 [ITP] -- Grapheme to Phoneme conversion tool



Hi Giulio,

I missed your initial ITP but now I became aware of the package.  I also
noticed that you are doing the packaging in

   git://anonscm.debian.org/collab-maint/sequitur-g2p.git

I'd rather recommend to join Debian Science team because it is definitely
related and you can get additional support.

Kind regards

       Andreas.

On Sat, Apr 13, 2013 at 12:31:36AM +0200, Jakub Wilk wrote:
> * Giulio Paci <giuliopaci@gmail.com>, 2013-04-07, 23:00:
> >>"python-dev (>= 2.6.6-9)" - why this version?
> >I took it from here:
> >http://www.debian.org/doc/packaging-manuals/python-policy/ap-build_dependencies.html
> 
> Well, this is weird. I have no idea what was the rationale for such
> version. It was bumped (from ">= 2.3.5-7") in python-defaults
> 2.7.3~rc2-1, but changelog for this version doesn't document this
> modification. 2.6.6-9's changelog mentions only a dh_python2 bugfix
> that should affect only minority of packages.
> 
> Anyway, I'd rather make the build-dependency unversioned, to make
> squeeze backporters' life easier.
> 
> And now for something completely different... Typos:
> 
> analogious -> analogous
> assumtion -> assumption
> cetrainty -> certainty
> Constuctor -> Constructor
> correcly -> correctly
> correctnes -> correctness
> decendent -> descendant (?)
> discout -> discount (?)
> expresion -> expression
> Franlin -> Franklin
> fullfil -> fulfil
> hiearchy -> hierarchy
> immediatelly -> immediately
> Implementaion -> Implementation
> imporatant -> important
> ingored -> ignored
> ingoring -> ignoring
> inhert -> inherit
> micture -> mixture
> modfied -> modified
> necesseary -> necessary
> occured -> occurred
> paramters -> parameters
> precoditions -> preconditions
> preferrable -> preferable
> publically -> publicly
> realtive -> relative
> Recipies -> Recipes
> releationship -> relationship
> smooting -> smoothing
> substitue -> substitute
> verion -> version
> 
> Estimation.cc has this twice:
> #if 1 // DEBUG
> Shouldn't that be "#if 0 ..."?
> 
> Using "except" without specifying exception type is bad idea:
> http://docs.python.org/2/howto/doanddont.html#except
> 
> gOpenIn and gOpenOut functions pass filenames to shell unescaped.
> They probably should just use the gzip module.
> 
> FileWriter, FileReader and StoredCounts classes have the same
> problem. (Although these three are not used in Debian binary
> packages AFAICS.)
> 
> -- 
> Jakub Wilk
> 
> 
> -- 
> To UNSUBSCRIBE, email to debian-mentors-REQUEST@lists.debian.org
> with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
> Archive: [🔎] 20130412223135.GA5946@jwilk.net">http://lists.debian.org/[🔎] 20130412223135.GA5946@jwilk.net
> 
> 

-- 
http://fam-tille.de


Reply to: