+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@xxxxxxxxxxxxxxxxxxxxxxx > https://admin.fedoraproject.org/mailman/listinfo/infrastructure > _______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/infrastructure