On Wed, Feb 11, 2015 at 06:05:13PM +0100, Lennart Poettering wrote: > On Mon, 09.02.15 21:41, Cristian Rodríguez (crrodrig...@opensuse.org) wrote: > > > execute.c only uses basename (the GNU version in <string.h>) > > Is this right even? > > This was added by Zbigniew in 2b6bf07dd23bb467099d213c97b3875c5e453491: > > "Get rid of our reimplementation of basename > > The only problem is that libgen.h #defines basename to point to > it's own broken implementation instead of the GNU one. This can be > fixed by #undefining basename." > > > So, what's this about? > > Somehow, given how stupid basename is defined, I am tempted to > introduce a call for this again, that is not stupid, and works for all > cases without ambiguities. > > Anyway, Zbigniew, could you comment on this, please? I think the patch is fine. As long as every libgen.h include is followed by #undef basename, things are OK.
Zbyszek _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel
