CVSROOT: cvswrappers - treat patches as binary files

Elan Ruusamäe glen at pld-linux.org
Mon Nov 3 21:02:33 CET 2008


On Monday 03 November 2008 21:32:41 Elan Ruusamäe wrote:
> On Monday 03 November 2008 12:38:31 Marcin Krol wrote:
> > > this will make cvsweb unable to provide diffs between versions,
> >
> > Diffs works just fine with this change on both old and new cvsweb.
>
> for non-binary files - yes (presumeably you looked older patch files)
> for binary (cvs admin -kb) - no
> cvswrappers affect newly added files only anyway...
>
> here's file which is binary for cvs:
> http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/test/kdeaddons-babelfish-google
>.patch
>
> no diffs, no view as text...

and asking again, how the F*K you're supposed to remove the -kb???


glen at wintersunset pld/SOURCES $ cvs admin -kkv php-fpm-zts.patch
RCS file: /cvsroot/SOURCES/php-fpm-zts.patch,v
cvs server: admin -k no longer affects future checkins.  Use update -k / commit -f instead.
done

glen at wintersunset pld/SOURCES $ cvsstat php-fpm-zts.patch|sh
 php-fpm-zts.patch    *Status: Up-to-date Local: 1.2 CVS: 1.2
       Sticky Options: b
glen at wintersunset pld/SOURCES $ cvs update -kkv php-fpm-zts.patch
U php-fpm-zts.patch

glen at wintersunset pld/SOURCES $ cvs ci -f -m '- remove -kb' php-fpm-zts.patch
cvs server: Up-to-date check failed for `php-fpm-zts.patch'
cvs [server aborted]: correct above errors first!

glen at wintersunset pld/SOURCES $

until no answer, reverting cvswrappers forcing binary!

-- 
glen


More information about the pld-devel-en mailing list