[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version control repositories


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230512





------- Additional Comments From bjohnson@xxxxxxxxxxxx  2007-03-01 14:48 EST -------
> # sitelib for noarch packages, sitearch for others (remove the unneeded one)
%{!?python_sitelib: %define python_sitelib %(%{__python} -c "from 
> distutils.sysconfig import get_python_lib; print get_python_lib()")}
%{!?python_sitearch: %define python_sitearch %(%{__python} -c "from 
> distutils.sysconfig import get_python_lib; print get_python_lib(1)")}

You can remove the sitearch line if your package is a noarch.

> Requires:       httpd, mod_python, subversion >= 1.2, rcs, diffutils

You require both cvs (rcs) and subversion file even though someone could use one
or the other.  Is there a better way to do this?

You are also setting up the include file for cvsgraph, but not requiring cvsgraph.

GNU enscript is already available in Extras.  If you require and configure for
enscript you can have sytax highlighting.

The INSTALL files says that you need subversion python bindings - it appears
that those are in the main subversion package?

> %doc CHANGES README INSTALL TODO
Normally we don't include INSTALL files, except in this case it has additional
instruction on how to setup the mysql commit database (if someone wants to do
that at a later time), so it is useful.

The COMMITTERS file lists the contributors to the project, so it should be
included in %doc.

Are ./www and ./viewvc.org useful docs?

In viewvc.conf:
>     AddHandler python-program .py
>     PythonHandler handler

will require mod_python (which you do).  This is only one way of running viewvc.
 Is there any particular reason that you've chosen this method?  Is this the
best way?

In viewvc.conf:
> #    PythonDebug On
remove :)


> Requires:       httpd, mod_python, subversion >= 1.2, rcs, diffutils

If you require mod_python, you don't need httpd because it will be found by
dependency of mod_python on httpd.

I'll have to dig into the substitutions you're doing, but that will take me a
little longer.

Here are the rpmlint complaints for you package:
E: viewvc script-without-shebang
/usr/lib/python2.4/site-packages/viewvc/bin/mod_python/query.py
E: viewvc script-without-shebang
/usr/lib/python2.4/site-packages/viewvc/bin/mod_python/handler.py
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/vclib/ccvs/blame.py 0644
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/py2html.py 0644
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/compat_difflib.py 0644
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/compat_ndiff.py 0644
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/query.py 0644
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/ezt.py 0644
E: viewvc htaccess-file
/usr/lib/python2.4/site-packages/viewvc/bin/mod_python/.htaccess
E: viewvc script-without-shebang
/usr/lib/python2.4/site-packages/viewvc/bin/mod_python/.htaccess
E: viewvc script-without-shebang
/usr/lib/python2.4/site-packages/viewvc/bin/mod_python/viewvc.py
E: viewvc non-executable-script
/usr/lib/python2.4/site-packages/viewvc/lib/blame.py 0644
W: viewvc setup-not-quiet
W: viewvc mixed-use-of-spaces-and-tabs (spaces: line 5, tab: line 41)


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]