search

prosodylab / Prosodylab-Aligner

Python interface for forced audio alignment using HTK and SoX
http://prosodylab.org/tools/aligner/
MIT License
331 stars 77 forks source link

change order of operations in the install? #39

Closed Kedersha closed 9 years ago

Kedersha commented 9 years ago

In the HTK install portion of the instructions, the following order is given:

$ make clean $ export CPPFLAGS=-UPHNALG $ ./configure --disable-hlmtools --disable-hslab $ make -j4 all $ sudo make -j4 install

However, make clean doesn't work if you don't run the configure first--could we switch the order to:

$ export CPPFLAGS=-UPHNALG $ ./configure --disable-hlmtools --disable-hslab $ make clean $ make -j4 all $ sudo make -j4 install

I didn't want to change it without checking, just in case I had this wrong!

kylebgorman commented 9 years ago

On Feb 9, 2015, at 5:25 PM, Kedersha notifications@github.com wrote:

In the HTK install portion of the instructions, the following order is given:

$ make clean $ export CPPFLAGS=-UPHNALG $ ./configure --disable-hlmtools --disable-hslab $ make -j4 all $ sudo make -j4 install

However, make clean doesn't work if you don't run the configure first--could we switch the order to:

$ export CPPFLAGS=-UPHNALG $ ./configure --disable-hlmtools --disable-hslab $ make clean $ make -j4 all $ sudo make -j4 install

Oh, good point, that should be changed.

The only function of make clean is that if the person has previously tried to compile HTK and failed for some other reason, they have to make clean first (which deletes already-compiled bits). People starting from scratch have no particular reason to do that.

Kyle

Kedersha commented 9 years ago

Thanks! :) I've also been running into some trouble trying to run the newest version of the aligner, but I'll ask about it later. Are github issues or email a better way to get in touch?

kylebgorman commented 9 years ago

Issues are better. Use email if you need to send me data so I can reproduce the error.

K

On Feb 9, 2015, at 5:41 PM, Kedersha notifications@github.com wrote:

Thanks! :) I've also been running into some trouble trying to run the newest version of the aligner, but I'll ask about it later. Are github issues or email a better way to get in touch?

— Reply to this email directly or view it on GitHub.

Kedersha commented 9 years ago

Great, thanks