[PULL] button mapping fix and unconstify patches

Peter Hutterer peter.hutterer at who-t.net
Wed Feb 12 16:37:34 PST 2014


On Wed, Feb 12, 2014 at 05:43:04PM -0500, Gaetan Nadon wrote:
> On 14-02-12 12:57 PM, Keith Packard wrote:
> > This function is supposed to be automatically replaced with os/strlcpy.c
> > when not present in your C library.
> >
> > Something appears to be amiss here; I don't have strlcpy *or* strlcat in
> > my glibc, and so configure correctly adds os/strlcpy.c and os/strlcat.c
> > to the build.
> >
> I cannot reproduce the problem anymore. I retraced my commands from the
> terminal, run make CC="gcc -D_FORTIFY_SOURCE=2" again, this time it works.

just an issue with incomplete builds, happens to me when I rebase or switch
branches. make clean in os/ usually does the job, otherwise
autoreconf/configure definitely does.

Cheers,
   Peter

> This shows it is already defined:
> 
>     nadon at memsize:~/xorg/src/app/xclock$ gcc -dM -E - < /dev/null | grep
>     FORTIFY
>     #define _FORTIFY_SOURCE 2
> 
> There does not seem to be any harm in defining for all builds. If it is
> not supported, it will be ignored.
> 
> Let me know, and I can create a patch to add this in util-macros. It can
> be conditionally added using AC_CHECK_DECLS. If it is already define, it
> won't be added.
> 



More information about the xorg-devel mailing list