Re: patchset-8 release (Re: [unionfs][patch] improvements of the unionfs - Problem Report, kern/91010)

Cy Schubert ha scritto:
> In message <43EBA250.1080307_at_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))
> 
> 

Ok, it compiled. New iso is at:

http://torrent.freesbie.org/FreeSBIE-unionfs-i386-20060210.iso.torrent

It seems better, I can log in and even startx. Further testing is coming :)

Thanks,
Dario

-- 
Dario Freni (saturnero_at_freesbie.org)
FreeSBIE developer (http://www.freesbie.org)
GPG Public key at http://www.saturnero.net/saturnero.asc