On Tue, Aug 30, 2016, at 02:07 PM, Marius Bakke wrote: > I don't have a Guix system with audio atm so someone else will have to > review it :)
It works for me on GuixSD OOTB. Brendan Tildesley wrote: > + (home-page "http://praat.org") The docs point to "http://www.praat.org", so I think we should include the www. > + (synopsis "Doing phonetics by computer") > + (description > + "Praat is a program for linguists to analyse, synthesize, and > +manipulate speech, generate graphs for scientific papers, and more.") I know it's Praat's tagline, but the synopsis doesn't match the rest of our packages. Something like "Phonetic analysis program" would be better. The description could probably be a bit more fleshed out too, but I think it's also fine if you don't want to bother. > + (license license:gpl2))) This should be gpl2+. It looks like praat bundles some third-party libraries under external/ with various licenses. These should be unbundled if possible, or else their licenses should be added to praat's license field. Thanks for contributing, hope it's not too much trouble cleaning up these last few details! -- Alex Griffin