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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+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


[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux