Ticket #198 (Fixed)Sat Feb 21 19:16:54 UTC 2009
CVS revision viewer doesn't list modified files for check-ins against branches, only against trunk
| Reported by: | Jeffrey Lee (213) | Severity: | Enhancement |
| Part: | Web site: RCVSweb and CVSweb (CVS browsing) | Release: | 2nd public site release |
| Milestone: | 2nd public site release completed | Status | Fixed |
Details by Jeffrey Lee (213):
The revision viewer at http://www.riscosopen.org/viewer/revisions only lists modified files if they were checked in to the trunk. Check-ins against branches will show the correct log message, but there will be no modified/added/etc. files listed.
Changelog:
Modified by Ben Avison (25) Sun, February 22 2009 - 14:30:36 GMT
Are the missing files there if you look inside the “Attic” subdirectory?
I think maybe the hideattic=0 option should maybe be the default, since this feature of CVS is not intuitive for non-CVS gurus.
Modified by Jeffrey Lee (213) Wed, March 11 2009 - 20:58:34 GMT
I think you’ve misunderstood. If you look at http://www.riscosopen.org/viewer/revisions/logs… you’ll see that the log viewer provides links to the modified files, but if you look at http://www.riscosopen.org/viewer/revisions/logs… you’ll see that it doesn’t. The first one was a check in against the trunk, the second was against the Cortex branch.
Modified by Andrew Hodgkinson (6) Thu, April 23 2009 - 17:10:52 GMT
- Milestone changed from Unspecified to 2nd public site release completed
- Severity changed from Normal to Enhancement
- Release changed from Unspecified to 2nd public site release
This appears to be a limitation of the cvslog2web script which runs when commits are made in CVS. It generates the HTML summaries from these. The Ruby On Rails wrapper application is only used to show these HTML summaries inside the wider Rails-based site framework.
The author was receptive to a patch I submitted back in 2007 after making changes to the script to support the ROOL site. If you report the problem using the address given at the bottom of the page listed below, you may well get a fix:
http://ethan.tira-thompson.com/cvslog2web/
Leaving this ticket open for now, but marking it as an enhancement request rather than a fault (since that seems to best describe the omission from cvslog2web).
Modified by Andrew Hodgkinson (6) Thu, April 23 2009 - 17:46:53 GMT
As a footnote, the version of cvslog2web used on the site has been updated from 1.11 to 1.16.
Modified by Jeffrey Lee (213) Sun, March 14 2010 - 19:17:31 GMT
Somehow I managed to miss your last two comments until now. Next time I check in to a branch I’ll have to remember to check if the bug’s still there, and if so report it to the cvslog2web guys.
Modified by Jeffrey Lee (213) Sat, March 20 2010 - 02:03:57 GMT
- Status changed from Open to Fixed
Good news – the new version of cvslog2web shows branched files just fine.
