+1 from me. Though this may need to be coordinated with notting's patch
:)
-Mike
On Thu, 12 Aug 2010, Todd Zullinger wrote:
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
_______________________________________________
infrastructure mailing list
infrastructure(a)lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/infrastructure