search

srvk / eesen

The official repository of the Eesen project
http://arxiv.org/abs/1507.08240
Apache License 2.0
824 stars 342 forks source link

Cannot install openfst-1.4.1 #215

Closed JeromeNi closed 4 years ago

JeromeNi commented 4 years ago

Hello,

after cloning eesen, I went to tools directory and typed make. However, it was encountering an error when installing openfst-1.4.1

The error was shown as below:

./../include/fst/replace.h: In constructor 'fst::ArcIterator<fst::ReplaceFst<A, T, C> >::ArcIterator(const fst::ReplaceFst<A, T, C>&, fst::ArcIterator<fst::ReplaceFst<A, T, C> >::StateId)': ./../include/fst/replace.h:1253:57: error: expected ';' before '::' token ->template CacheBaseImpl<typename C::State, C>::InitArcIterator( ^~ Makefile:328: recipe for target 'fst.lo' failed

I've attached a full output from terminal here out_make.txt

AdolfVonKleist commented 4 years ago

This is a known issue with older versions of openfst and recent version of g++. I encountered this last week with the same version of OpenFst, although not in the context of eesen. Following this patch fixed the issue for me (ubuntu 18 + g++ 7.4):

https://github.com/hfst/hfst/commit/134d5a5aac3fd015c4955b6f5fdfede42159e402

2019年9月15日(日) 20:02 JeromeNi notifications@github.com:

Hello,

after cloning eesen, I went to tools directory and typed make. However, it was encountering an error when installing openfst-1.4.1

The error was shown as below:

./../include/fst/replace.h: In constructor 'fst::ArcIterator<fst::ReplaceFst<A, T, C> >::ArcIterator(const fst::ReplaceFst<A, T, C>&, fst::ArcIterator<fst::ReplaceFst<A, T, C>

::StateId)': ./../include/fst/replace.h:1253:57: error: expected ';' before '::' token ->template CacheBaseImpl<typename C::State, C>::InitArcIterator( ^~ Makefile:328: recipe for target 'fst.lo' failed

I've attached a full output from terminal here out_make.txt https://github.com/srvk/eesen/files/3614099/out_make.txt

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/215?email_source=notifications&email_token=AABVUA5SODPJDC5DMIZUEYTQJZ2ETA5CNFSM4IW3OQD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HLOJQIQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AABVUA43XEOW7VAFPIYQDIDQJZ2ETANCNFSM4IW3OQDQ .

JeromeNi commented 4 years ago

Thank you, the patch worked for me!