On 04/06/2016 21:39, Jakub Wilk wrote: > * Giulio Paci <giuliop...@gmail.com>, 2016-06-04, 18:42: >> git clone git://anonscm.debian.org/collab-maint/opengrm-ngram.git > > Let me see: > >> + * Drop 1001_fix_compilation_issues.patch. >> + No more needed. > > I'm not a native speaker of English, but shouldn't it be s/more/longer/?
I did a quick search and it seems you are right. >> +Maximum number of ngrams to leave in model after pruning.Value less than >> zero means no target number, just use theta. > > Space is missing between "pruning." and "Value". Added a patch and updated the manpage. > There's an unholy mixture of tabs and spaces in update-man.sh: the line you > added is indented by 8 spaces, whereas the surrounding lines are indented by > tabs. Fixed. > There's a similar mixture of tabs and spaces in the definition of BACKUP in > debian/rules. Fixed. > In #687563, I asked you to talk to upstream about insecure use of /tmp in the > test suite. Did anything happen about this? Just my notification and a quick aknoweldge. I reproposed the issue to upstream. > Now that OpenFST is in unstable, should we upload this package to unstable > too? I think so: updated changelog accordingly. Bests, Giulio
