Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Aug 2005 16:30:36 -1000
From:      juli mallett <jmallett@FreeBSD.org>
To:        Tom Rhodes <trhodes@FreeBSD.org>
Cc:        juli mallett <jmallett@FreeBSD.org>, cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/lib/libufs Makefile
Message-ID:  <20050831023036.GA25335@toxic.magnesium.net>
In-Reply-To: <20050830200219.534e6d72@localhost>
References:  <200508251022.j7PAMWFO030767@repoman.freebsd.org> <20050825195416.GA65980@toxic.magnesium.net> <20050826062242.734ffae4@localhost> <20050828221142.GA71466@toxic.magnesium.net> <20050830200219.534e6d72@localhost>

next in thread | previous in thread | raw e-mail | index | archive | help
* Tom Rhodes <trhodes@FreeBSD.org> [ Date: 2005-08-30 ]
	[ w.r.t. Re: cvs commit: src/lib/libufs Makefile ]
> On Sun, 28 Aug 2005 12:11:42 -1000
> juli mallett <jmallett@FreeBSD.org> wrote:
> 
> > * Tom Rhodes <trhodes@FreeBSD.org> [ Date: 2005-08-26 ]
> > 	[ w.r.t. Re: cvs commit: src/lib/libufs Makefile ]
> > > On Thu, 25 Aug 2005 09:54:16 -1000
> > > juli mallett <jmallett@FreeBSD.org> wrote:
> > > 
> > > > * Tom Rhodes <trhodes@FreeBSD.org> [ Date: 2005-08-25 ]
> > > > 	[ w.r.t. cvs commit: src/lib/libufs Makefile ]
> > > > > trhodes     2005-08-25 10:22:30 UTC
> > > > > 
> > > > >   FreeBSD src repository
> > > > > 
> > > > >   Modified files:
> > > > >     lib/libufs           Makefile 
> > > > >   Log:
> > > > >   Hook getino.3 up to the build and link it to putino.3.
> > > > >   
> > > > >   PR:     83820
> > > > 
> > > > This is wrong.  As the PR states it was purposely not hooked up.
> > > > inode.c doesn't have functions which look like the interfaces
> > > > you've just published, and there isn't even a putino(3) in
> > > > there.  If the cause of the confusion is the manpage being listed
> > > > in Xr's or something, then comment out the Xr's.  This is very
> > > > obviously wrong, if you try to use what's in the manpage I
> > > > committed.  Probably it shoudln't be in CVS at all.  The reason I
> > > > put it there at the time is that it was ready and inode.c was
> > > > about ready to hit CVS, but that got dropped.  Now, if you want
> > > > to help me remember what P4 tree that inode.c was in, and pick up
> > > > whereever I left off, that'd be awesome.
> > > 
> > > Let me understand real quick, it was purposely not hooked up because
> > > it was the only file not updated?  I mean, I can back this out,
> > > no problem; however, I fail to see why nothing is going on with
> > > this code.  No clue on the branch, sorry.
> > 
> > It was purposely not hooked up because the code that it documents
> > isn't in CVS.
> > 
> 
> I know now why it wasn't hooked up to the build.  What I'm asking
> is:
> 
> 1: Is this currently maintained?  and
> 2: What utilities are currently using this?

I don't remember what uses getino() as implemented in libufs.  Maybe
nothing, in which case it can just be removed.  In any event, the
interface that is there is bad and shouldn't be documented.  That said,
it isn't currently maintained very well, so the odds of me remembering
where the new code is and what's wrong with it are low, so in any
event, the right thing to do is for the manpage to not be installed,
and to not be in CVS.

Thanx,
juli.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050831023036.GA25335>