vim-jp / ctags

Use https://ctags.io instead (This was fork of http://ctags.sourceforge.net/)
GNU General Public License v2.0
26 stars 7 forks source link

Update options.c so it can compile on Sun #19

Closed ericcurtin closed 9 years ago

ericcurtin commented 10 years ago

Hi Guys,

I'm a heavy user of ctags and I love it thanks for all your hard work. I wanted to compile the latest version on ctags on a Sun/Solaris server at work, I don't have root access and asking IT to do it is a pain. So I thought compiling it myself was a better option. I compiled it pretty quickly thanks for making it so easy!

But I also have bad news the following doesn't compile on Sun so I removed them from my build because I did not want their functionality:

asm.c eiffel.c flex.c fortran.c jscript.c ocaml.c python.c sql.c tex.c vhdl.c

But I did make a code change to options.c as it seemed like a pretty important file to compile. This is the error you receive if you try to compile without this code change:

"options.c", line 661: Error: Cannot use const char* to initialize char*.

I made my decision based on this:

http://stackoverflow.com/a/833114

I have no knowledge of the codebase so I thought this change would have the most minimal effect. I will leave you decide if you want to ifdef around it so that this change only occurs during the Sun compile.

Regards,

Eric Curtin

mattn commented 10 years ago

Thanks. But I don't understand what your said. You pointed some *.c in above but you had fixed problem by modifying one source file not header file. And also this is C code not C++ code. So you must not use class template. You should use C cast operator (char*) simply.

k-takata commented 10 years ago

Perhaps, you use a C++ compiler instead of a C compiler. Please make sure that you use a C compiler.

mattn commented 9 years ago

Long time has elapsed