[desktop-discuss] Re: Latest MPlayer

[Doug Scott]

> > The glib patch I mentioned only defines
> > G_GNUC_INTERNAL to __hidden for Sun Studio.
> > See /usr/lib/glib-2.0/include/glibconfig.h which
> is
> > included by /usr/include/glib-2.0/glib/gtypes.h.
> > 
> > yippi, who wrote the glib patch, might know if
> there
> > is a way to avoid having to patch other modules
> (to
> > move G_GNUC_INTERNAL to the start of a line), but
> I'm
> > sure he would have told me if there was.
> 
> I just did a fresh install of build 42, and
> glibconfig.h
> unconditionally defines G_GNUC_INTERNAL to
> __hidden:
> 
> #define beep_params __hidden
> 
> Apparently, nothing that links to glib can be built 
> using GCC, which makes life extremely problematic.
> 
> In addition, glib/gmessages.h makes use of this
> #define without including glibconfig.h.  So open
> source
> packages which fail to include <glibconfig.h> first
> fail to compile (the one I was trying simply did an
> include of glib-object.h).
> 
> This all came from an attempt to build inkscape,
> which IMHO should be shipped wherever the
> GIMP is shipped.  I had it working on build 40
> by linking against blastwave packages, but the
> current JDS components don't seem very
> cooperative.
> 
> - Eric

Yes, I have had to copy the system header file into the build directory and modify the #define G_GNUC_INTERNAL to get gcc to work. The system header file needs an #ifdef to add the correct hidden attribute for the compiler. Also it looks like the GCC syntax is included in the latest Sun Studio Express SPRO CC. http://developers.sun.com/prodtech/cc/downloads/express_readme.html

Doug
 
 
This message posted from opensolaris.org