[PATCH/RFC] viewvc: Add a note that CVS is read-only
Mike McGrath
mmcgrath at redhat.com
Thu Aug 12 15:38:47 UTC 2010
+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 at lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/infrastructure
>
More information about the infrastructure
mailing list