Compilation fails under certain circumstances in Xcode with stdio.h colliding with system include about marisa-trie

[GoogleCodeExporter]

Thank you for your report and solution!

I'll test the problem tonight.

Maybe, I'll try using #include "marisa/stdio.h" instead of #include "stdio.h" 
so as to avoid the problem, because "marisa/stdio.h" is already a part of 
libmarisa's API.

Original comment by [email protected] on 19 Aug 2013 at 1:40

[GoogleCodeExporter]

Great! Looking forward to the fix!

Original comment by [email protected] on 19 Aug 2013 at 8:06

[GoogleCodeExporter]

I tested compilation with Xcode, but I couldn't reproduce the problem.
Also, configure & make approach worked as well.

However, I'm not sure if my approach was right or not because I'm not familiar 
with Xcode.

First, I created a project.
Then, I copied lib/marisa.h and lib/marisa/ to the project directory.
After that, I added the source & header files to the project (with 
drag-and-drop).
And I could build a project successfully.

I would appreciate if you could tell me your steps to build your application.

By the way, I attached a special version that uses #include "marisa/stdio.h" 
instead of #include "stdio.h".
This version might solve your problem but it does not work with Visual C++.

Original comment by [email protected] on 20 Aug 2013 at 5:46

Attachments:

• marisa-0.2.5.tar.gz

[GoogleCodeExporter]

Thanks - we'll look at that! Our problem probably was caused by using 
'recursive' include in Xcode. We didn't include the marisa files 'explicitly' 
but included a directory with a few subdirs. each subdir is an open source 
project - one of them marisa - then it happened for us ...

Original comment by [email protected] on 20 Aug 2013 at 3:06

[GoogleCodeExporter]

I'm happy to hear that the problem is solved.

In the future, I'll use non-standard include file names in order to avoid this 
kind of problems.

Original comment by [email protected] on 21 Aug 2013 at 2:34

• Changed state: WontFix

[GoogleCodeExporter]

Hi - sorry to be a bore - but why is this fixed? wouldn't it be great to always 
include "marisa/stdio.h" instead of "stdio.h" and adjust the visual studio 
build? 

Many thanks
Thomas

Original comment by [email protected] on 21 Aug 2013 at 10:16

[GoogleCodeExporter]

I'm sorry about being so late to reply.

I confirmed that VC++ can build libmarisa by adding "../../lib" to 
AdditionalIncludeDirectories and I committed a fixed version (r147).

Thank you.

Original comment by [email protected] on 27 Aug 2013 at 12:16

• Changed state: Fixed