[PATCH/RFC] viewvc: Add a note that CVS is read-only

Thu Aug 12 15:33:17 UTC 2010

While we redirect cvs.fedoraproject.org to pkgs.fedoraproject.org,
specific URL's are not redirected (and probably should not be).  Adding
a large note to the viewvc header template should make it more obvious
that CVS is not being updated anymore.
---

This is an RFC, because there is potential room for improvement here.
While the default cvsroot for viewvc is pkgs, there are still other
cvsroots accessible via viewvc and I'm not sure if all of them are
read-only.  I doubt they are.

The question is whether having the note in the header for all cvsroots
is a bad thing and if anyone knows the viewvc ezt template system well
enough to make it only show up when rootname == pkgs.  It seems like
it may be possible to do this, but I don't know the ezt syntax to make
that happen.  Help would be most welcome, if the general idea here is
sound and someone else is bored or curious enough to give it a try.

 modules/viewvc/files/viewvc/header.ezt |   24 ++++++++++++++++++++++++
 modules/viewvc/manifests/init.pp       |    6 ++++++
 2 files changed, 30 insertions(+), 0 deletions(-)
 create mode 100644 modules/viewvc/files/viewvc/header.ezt

diff --git a/modules/viewvc/files/viewvc/header.ezt b/modules/viewvc/files/viewvc/header.ezt
new file mode 100644
index 0000000..96bd5a2
--- /dev/null
+++ b/modules/viewvc/files/viewvc/header.ezt
@@ -0,0 +1,24 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
+    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<!-- ViewVC :: http://www.viewvc.org/ -->
+<head>
+  <title>[if-any rootname][[][rootname]][else]ViewVC[end] [page_title]</title>
+  <meta name="generator" content="ViewVC [vsn]" />
+  <link rel="shortcut icon" href="[docroot]/images/favicon.ico" />
+  <link rel="stylesheet" href="[docroot]/styles.css" type="text/css" />
+  [if-any rss_href]<link rel="alternate" type="application/rss+xml" title="RSS [[][rootname]][where]" href="[rss_href]" />[end]
+</head>
+<body>
+<h2>NOTE: CVS is now read-only.  See <a href="http://pkgs.fedoraproject.org/gitweb/">gitweb</a> for current changes.</h2>
+<div class="vc_navheader">
+<table><tr>
+<td><strong>[if-any roots_href]<a href="[roots_href]"><span class="pathdiv">/</span></a>[else]<span class="pathdiv">/</span>[end][if-any nav_path][for nav_path][if-any nav_path.href]<a href="[nav_path.href]">[end][if-index nav_path first][[][nav_path.name]][else][nav_path.name][end][if-any nav_path.href]</a>[end][if-index nav_path last][else]<span class="pathdiv">/</span>[end][end][end]</strong></td>
+<td style="text-align: right;">[if-any username]Logged in as: <strong>[username]</strong>[end]</td>
+</tr></table>
+</div>
+
+<div style="float: right; padding: 5px;"><a href="http://www.viewvc.org/" title="ViewVC Home"><img src="[docroot]/images/viewvc-logo.png" alt="ViewVC logotype" width="240" height="70" /></a></div>
+<h1>[page_title]</h1>
+
+
diff --git a/modules/viewvc/manifests/init.pp b/modules/viewvc/manifests/init.pp
index 08617ea..426458d 100644
--- a/modules/viewvc/manifests/init.pp
+++ b/modules/viewvc/manifests/init.pp
@@ -16,4 +16,10 @@ class viewvc::web {
         mode => 0644,
         require => Package['viewvc']
     }
+
+    file { '/usr/share/viewvc/templates/include/header.ezt':
+        source => 'puppet:///viewvc/viewvc/header.ezt',
+        mode => 0644,
+        require => Package['viewvc']
+    }
 }
-- 
1.7.2.1

-- 
Todd        OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I never forget a face, but in your case I'll be glad to make an
exception.
    -- Groucho Marx

