This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 4/4] compile: Add 'set compile-gcc'


> Date: Thu, 23 Apr 2015 23:09:16 +0200
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> Cc: gdb-patches@sourceware.org, pmuldoon@redhat.com
> 
> > > +Besides the compiler driver @value{GDBN} needs also shared library
> > > +@file{libcc1.so}.  It is searched in default shared library search path
> > > +(overridable with usual environment variable @code{LD_LIBRARY_PATH}),
> > > +unrelated to @code{PATH} or @code{set compile-gcc} settings.
> > 
> > This is Unix logic, btw.  On Windows, libcc1.dll will either be in the
> > same directory where the GCC driver is, or on PATH, or in libexec.  So
> > maybe you should qualify the above by saying this is on Posix hosts;
> > we could then amend it when this is supported on Windows (assuming it
> > isn't for now).
> 
> I do not think it would work on MS-Windows host as gcc/libcc1/ uses:
> 	# If any of these functions are missing, simply don't bother building
> 	# this plugin.
> 	GCC_ENABLE_PLUGINS
> 	AC_CHECK_FUNC(socketpair, , enable_plugin=no)
> 	AC_CHECK_FUNC(select, , enable_plugin=no)
> 	AC_CHECK_FUNC(fork, , enable_plugin=no)
> 	AM_CONDITIONAL(ENABLE_PLUGIN, test $enable_plugin = yes)

You underestimate the hacker power ;-)  Trust me, soon enough this
will work on Windows, and these conditions will then change.  So I
still think we should qualify the above logic by saying that it is
specific to Posix systems.

Thanks.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]