Subject: Re: Since we're on the subject, my pet PRs
To: Dave Huang <khym@azeotrope.org>
From: D'Arcy J.M. Cain <darcy@NetBSD.org>
List: tech-userlevel
Date: 03/15/2003 06:45:50
On Saturday 15 March 2003 06:14, Dave Huang wrote:
> On Sat, Mar 15, 2003 at 05:14:02AM -0500, D'Arcy J.M. Cain wrote:
> > On Saturday 15 March 2003 03:23, Dave Huang wrote:
> > > bin/5419: from May 8 1998... it was actually looked at in 2000, but
> > >  nothing since then.
> >
> > I have closed both of these.
>
> I notice you re-opened 5419... I tried recompiling find with the latest

Yah, I misread the PR and thought that the code was supposed to be added, not 
removed.

> sources from anoncvs, and I still see the problem. The #ifdef block
> causes find to skip any whiteouts returned by fts_read(), so I really
> have no idea why it's in there. It came from the merge of 4.4BSD-Lite2,
> but I don't know what it was doing in Lite2 either.

I tried testing it and union mounts don't seem to work correctly at all for 
me.  I will continue trying to test this.


> > Can you tell me the file or the URL where you saw this please.
>
> Around line 374 of distrib/notes/common/postinstall

Found it.  I made a change but when I rebuild the docs strange things happen.  
The pkg_add line looks like this:

PKG_PATH=ftp://ftp.netbsd.org/pub/NetBSD/packages/1.6P//home/darcy/work/minix/All

OK, so M in my environment is set to /home/darcy/work/minix (don't ask :-) so 
I understand where that comes from but I don't know why it uses it.  
Undefining M doesn't look exactly right either.  I will submit the change 
though and let someone else regen.

Here is the change I am making.  Does it seem right to you?

Index: postinstall
===================================================================
RCS file: /cvsroot/src/distrib/notes/common/postinstall,v
retrieving revision 1.48
diff -u -r1.48 postinstall
--- postinstall 2002/09/28 23:35:41     1.48
+++ postinstall 2003/03/15 11:45:06
@@ -371,7 +371,7 @@
 usually in the
 .Pa \*V/\*M/All
 subdir.
-You can install them with the following commands:
+You can install them with the following commands under sh(1):
 .Pp
 .Bd -unfilled
 .Ic # PKG_PATH=ftp://ftp.netbsd.org/pub/NetBSD/packages/\*V/\*M/All
@@ -379,6 +379,13 @@
 .Ic # pkg_add -v tcsh
 .Ic # pkg_add -v apache
 .Ic # pkg_add -v perl
+\&...
+.Ed
+.Pp
+If you are using csh(1) then replace the first two lines with the following:
+.Pp
+.Bd -unfilled
+.Ic # setenv PKG_PATH ftp://ftp.netbsd.org/pub/NetBSD/packages/\*V/\*M/All
 \&...
 .Ed
 .Pp

-- 
D'Arcy J.M. Cain <darcy@netbsd.org>
http://www.NetBSD.org/