Re: __progname in base

2015-12-16 Thread Ted Unangst
Theo Buehler wrote: > ping. > ok > On Tue, Dec 08, 2015 at 07:15:39PM +0100, Theo Buehler wrote: > > On Sat, Nov 07, 2015 at 12:20:42PM +0100, Tobias Stoeckmann wrote: > > > Based on Todd's patch for at and cron, I did a grep through our base > > > tree to see if there are more occurrences of

Re: __progname in base

2015-12-08 Thread Theo Buehler
On Sat, Nov 07, 2015 at 12:20:42PM +0100, Tobias Stoeckmann wrote: > Based on Todd's patch for at and cron, I did a grep through our base > tree to see if there are more occurrences of self-made __progname > handling. A few more of those: Index: caesar/caesar.c

Re: __progname in base

2015-12-05 Thread Ted Unangst
Tobias Stoeckmann wrote: > Opinions, thoughts? looks good, but you've got some mostly unrelated changes in here. this should be separate, but ok for the rest. > > Index: sbin/newfs_ext2fs/newfs_ext2fs.c > > === > > RCS file:

Re: __progname in base

2015-12-05 Thread Tobias Stoeckmann
On Sat, Dec 05, 2015 at 03:29:06AM -0500, Ted Unangst wrote: > looks good, but you've got some mostly unrelated changes in here. this should > be separate, but ok for the rest. It started with a "check argv" code review and ended up with __progname adjustments, so I agree here and removed the

Re: __progname in base

2015-12-04 Thread Michael McConville
Tobias Stoeckmann wrote: > Opinions, thoughts? > > > [...] > > > > Index: sbin/newfs/newfs.c > > === > > RCS file: /cvs/src/sbin/newfs/newfs.c,v > > retrieving revision 1.100 > > diff -u -p -u -p -r1.100 newfs.c > > ---

Re: __progname in base

2015-11-07 Thread Joerg Sonnenberger
On Sat, Nov 07, 2015 at 12:20:42PM +0100, Tobias Stoeckmann wrote: > Index: bin/mt/mt.c > === > RCS file: /cvs/src/bin/mt/mt.c,v > retrieving revision 1.36 > diff -u -p -u -p -r1.36 mt.c > --- bin/mt/mt.c 12 Nov 2013 04:36:02

Re: __progname in base

2015-11-07 Thread Joerg Sonnenberger
On Sat, Nov 07, 2015 at 03:32:11PM +0100, Joerg Sonnenberger wrote: > As Ingo recently reminded me, OpenBSD actually did add getprogname() at > some point, which needs need a actually manual forward definition. Too early for writing. It does *not* need such an ugly manual extern declaration.