[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

ViewCVS perpetually busted with Subversion repositories



On Sat, Aug 23, 2003 at 10:08:12AM +0200, Jérôme Marant wrote:
> You may find some clues about how to fix viewcvs on the XFS SVN server there:
> http://mailman.lyra.org/pipermail/viewcvs-dev/2003-July/001079.html
> 
> It looks like it has been fixed on svn.debian.org.

Thank you, but I'm afraid I don't have the time to maintain my own fork
of ViewCVS.

In my opinion:
        * Debian's ViewCVS package should be hijacked, or the maintainer
          should get more serious about dealing with the problems his
          packaging of a CVS snapshot causes;
        * A new upload should be made to unstable, adding an epoch and
          reverting to the last version sanctioned by the upstream
          developers; this way we won't release a snapshot in sarge
        * A viewcvs(-snapshot?) package can be maintained in Debian
          experimental if anyone has the patience to deal with it
        * The package's debconfage needs to be fixed or removed:
          1) removed, if ViewCVS has a directory browser that can choose
             between repositories (i.e., if ViewCVS can look at
             /var/lib/cvs and/or /var/lib/svn when they don't directly
             ccontain repos, the debconfage can be nuked entirely)
          2) fixed; scan /var/lib/{cvs,svn} at configuration time and
             use the first repository found as the default_root by
             default (letting the user change it if we're in an
             interactive frontend, of course)
        * The viewcvs CGI script needs to run with its umask set to 0002
          so it doesn't fuck up Subversion repositories.  I've attached
          a patch for this.

-- 
G. Branden Robinson                |     It just seems to me that you are
Debian GNU/Linux                   |     willfully entering an arse-kicking
branden@debian.org                 |     contest with a monstrous entity
http://people.debian.org/~branden/ |     that has sixteen legs and no arse.
--- viewcvs.cgi	2003-07-03 21:40:42.000000000 -0500
+++ viewcvs.cgi.PRECIOUS	2003-06-24 00:59:57.000000000 -0500
@@ -38,6 +38,7 @@
 # Adjust sys.path to include our library directory
 #
 
+import os
 import sys
 
 if LIBRARY_DIR:
@@ -55,4 +56,5 @@
 import sapi
 import viewcvs
 
+os.umask(0002)
 viewcvs.main(sapi.CgiServer())

Attachment: pgpfDtXrlPlFs.pgp
Description: PGP signature


Reply to: