hey tom,
it works on my ubuntu g++ 4.3.3 and with the newer g++ 4.4 on fedora 11,
after similar changes. I suppose a few more includes won't hurt so added
it into the main branch,
http://mosesdecoder.svn.sourceforge.net/viewvc/mosesdecoder?view=rev&revision=2399
Thx
Hieu
Tom Hoar wrote:
> I asked a friend who does professional system support for Ubuntu's
> parent company to review Moses for updates to compile on the newer
> g++4.3. He forwarded the comments below with his permission to forward
> to the Moses team for inclusion in an update. Hope someone can verify
> the suggestions make use of this in an upcoming update.
>
> Tom
>
> ---------- Forwarded message ----------
>
> ... Recent gcc versions have been tightening up the way standard
> headers include other standard headers, so you don't get the effect of
> including any that you didn't ask for.
>
> So a few extra includes are needed:
>
> moses/src/gzfilebuf.h: #include <cstring>
> moses/src/mempool.cpp: #include <cstring>
> moses/src/PCNTools.cpp: #include <cstdlib>
> moses/src/Util.cpp: #include <cstring>
> moses/src/Util.h: #include <cstdlib>
> moses/src/Word.h: #include <cstring>
> moses/src/WordsBitmap.h: #include <cstdlib>
>
> There's a ton of other warnings but upon detailed inspection, nothing
> that should break the program.
> ------------------------------------------------------------------------
>
> _______________________________________________
> Moses-support mailing list
> [email protected]
> http://mailman.mit.edu/mailman/listinfo/moses-support
>
_______________________________________________
Moses-support mailing list
[email protected]
http://mailman.mit.edu/mailman/listinfo/moses-support
