Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 09 Feb 2006 20:23:08 -0800
From:      Cy Schubert <Cy.Schubert@komquats.com>
To:        Dario Freni <saturnero@freesbie.org>
Cc:        hiroo.ono@gmail.com, ozawa@ongs.co.jp, freebsd-hackers@freebsd.org, Daichi GOTO <daichi@freebsd.org>, ota@j.email.ne.jp, freebsd-current@freebsd.org, Alexander@Leidinger.net
Subject:   Re: patchset-8 release (Re: [unionfs][patch] improvements of the  unionfs - Problem Report, kern/91010)
Message-ID:  <200602100423.k1A4N8MG080084@cwsys.cwsent.com>
In-Reply-To: Message from Dario Freni <saturnero@freesbie.org>  of "Thu, 09 Feb 2006 21:13:04 %2B0100." <43EBA250.1080307@freesbie.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
In message <43EBA250.1080307@freesbie.org>, Dario Freni writes:
> Daichi GOTO ha scritto:
> > I have updated the patchset-8 of unionfs.
> > 
> > Patchset-8:
> >    For 7-current
> >      http://people.freebsd.org/~daichi/unionfs/unionfs-p8.diff
> > 
> >    For 6.x
> >      http://people.freebsd.org/~daichi/unionfs/unionfs6-p8.diff
> > 
> >    Changes in unionfs-p8.diff
> >      - Fixed the issue that user whom has access permission
> >        cannot change the directory because he cannot create
> >        its shadow directory. As a result of this fixed, now
> >        unionfs uses root permission creating shadow directory
> >        temporarily.
> > 
> > The document of those unionfs patches is pretty improved by Hiroo
> > ONO-san.
> > 
> >   http://people.freebsd.org/~daichi/unionfs/  (English)
> >   http://people.freebsd.org/~daichi/unionfs/index-ja.html  (Japanese)
> > 
> > Please try -p8 Dario. We are thinking that you cat get it with -p8 :)
> 
> It doesn't compile on 6.x :/ (using unionfs6-p8.diff on a fresh RELENG_6)
> 
> cc -c -O2 -pipe -fno-strict-aliasing  -Wall -Wredundant-decls 
> -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes 
> -Wpointer-arith -Winline -Wcast-qual  -fformat-extensions -std=c99 
> -nostdinc -I-  -I. -I/usr/src/sys -I/usr/src/sys/contrib/altq 
> -I/usr/src/sys/contrib/ipfilter -I/usr/src/sys/contrib/pf 
> -I/usr/src/sys/contrib/dev/ath -I/usr/src/sys/contrib/dev/ath/freebsd 
> -I/usr/src/sys/contrib/ngatm -I/usr/src/sys/dev/twa -D_KERNEL 
> -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common 
> -finline-limit=8000 --param inline-unit-growth=100 --param 
> large-function-growth=1000  -mno-align-long-strings 
> -mpreferred-stack-boundary=2  -mno-mmx -mno-3dnow -mno-sse -mno-sse2 
> -ffreestanding -Werror  /usr/src/sys/ufs/ufs/ufs_lookup.c
> /usr/src/sys/ufs/ufs/ufs_lookup.c: In function `ufs_direnter':
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: `vdp' undeclared (first 
> use in this function)
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: (Each undeclared 
> identifier is reported only once
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: for each function it 
> appears in.)
> *** Error code 1
> 
> Stop in /usr/obj.unionfs-i386/usr/src/sys/FREESBIE.
> *** Error code 1
> 
> Stop in /usr/src.
> *** Error code 1
> 
> Stop in /usr/src.

Replace the line in the patch that says:

+               if (OFSFMT(vdp))

with:

+               if (OFSFMT(dvp))



Cheers,
Cy Schubert <Cy.Schubert@komquats.com>
Web:  http://www.komquats.com and http://www.bcbodybuilder.com
FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
BC Government:  <Cy.Schubert@gov.bc.ca>

    "Lift long enough and I believe arrogance is replaced by
    humility and fear by courage and selfishness by generosity
    and rudeness by compassion and caring."
        -- Dave Draper






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