ah, that's where the mgizapp name comes from. I'll ask qin to make the 
automake & cmake exec names consistent (ie. mgiza).

On 22/06/2012 05:53, Tom Hoar wrote:
> Hieu,
>
> Another observation about the changes. Compiling the new MGIZA++ with 
> cmake on my system yesterday created the binary $prefix/bin/mgizapp. 
> Line 228 of the most recent train-model.perl reads: $GIZA = 
> "$_EXTERNAL_BINDIR/mgiza"
>
> The current/valid binary name of MGIZA++ has been a topic of 
> discussion here before. However, since the file name changes 
> unexpectedly between builds and this topic is "changes to the training 
> scripts", would it be better if train-model.perl tests for both 
> possible filenames for MGIZA++ binary?
>
_______________________________________________
Moses-support mailing list
Moses-support@mit.edu
http://mailman.mit.edu/mailman/listinfo/moses-support

Reply via email to