On Wed, 11.02.15 18:10, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote:
> 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. Can you elaborate on the background of this? Why the #undef precisely? The man page says nothing about this? Lennart -- Lennart Poettering, Red Hat _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel