482079 - owners.html update needed to get pushed through svn instead of cvs (soon)

Status: RESOLVED INCOMPLETE

(Webtools Graveyard :: Despot, defect)

Description

[Samuel Sidler (old account; do not CC)]

We're working on switching mozilla.org from CVS to SVN and will need to make a Despot change that allows it to push to SVN instead of CVS.

Reed will have more details as we know them.

Comment 1

[David Boswell]

Updating summary since mirrors.html is updated by despotdaemon too.

Comment 2

[Samuel Sidler (old account; do not CC)]

Updating summary... I spun off mirrors.html to bug 510255 since it's a different script/software package.

Comment 3

[Samuel Sidler (old account; do not CC)]

Dave: Is this something you can work on in the next week or should I try my hand at Perl?

Comment 4

[Samuel Sidler (old account; do not CC)]

Attachment: owners.templ diff v1

To help Dave, I've created a patch for the template file, based on what's in SVN now.

Comment 5

[Samuel Sidler (old account; do not CC)]

Attachment: owners.templ diff v1.1

I hate diff.

Comment 6

[David Boswell]

Would now also be a good time to change where the owners.html file lives?  In
the new site structure owners.html fit better in the About directory along with other information about roles and responsibilities instead of in the top-level directory where it is now.

If this page can be checked into SVN at www.mozilla.org/about/owners.html
I'll set up a redirect so the old link will continue working.  

Does anyone know of any reasons not to move the file?

Comment 7

[David Boswell]

For timing, we're currently planning on going live with the site on Tuesday, August 25th.  If it won't be possible to make this change by then, please let us know.

Comment 8

[David Boswell]

justdave, do you have an estimate for when this will be ready?  We've decided not to block the site relaunch on this (scheduled for tonight) so it's possible we'll just have a version of the page that's slightly out of date for a short while.

Comment 9

[Reed Loden [:reed]]

Attachment: use svn instead of cvs - v1 (untested)

I cheated a little and repurposed some variables meant for CVS stuff so that they would work for SVN as well. I also hardcoded "owners.html". Both could be rewritten to be more generic if such a change isn't wanted. However, I'm reasonably sure only Mozilla uses Despot, so such a change wouldn't really hurt anybody. ;)

With this patch, the `repositories` table's row for /www needs to be updated with the following changes:
* `cvsroot` column needs to be changed to "svn+ssh://svn.mozilla.org/projects/www.mozilla.org/trunk"
* `ownerspath` column needs to be changed to "about/"

This assumes despotdaemon has a proper svn account that uses ssh and has ~/.ssh/config configured correctly. Some minor modifications could be made to the patch to make it use password authentication over https:// instead, if needed.

Comment 10

[Dave Miller [:justdave]]

Comment on attachment 396650 [details] [diff] [review]
use svn instead of cvs - v1 (untested)

>         $query = $::db->prepare("SELECT cvsroot FROM repositories WHERE id = ?");
>         $query->execute($ownersrepository);
>-        $ENV{CVSROOT} = ($query->fetchrow_array())[0];
>+        $svnroot = ($query->fetchrow_array())[0];

That's going to return ":ext:[e-mail]:/cvsroot" which I don't think is what our $svnroot is. :)

Comment 11

[Dave Miller [:justdave]]

Attachment: combined patch v2 (tested)

OK, got this all sorted, attached is the revised and tested patch which is now live in production.

Comment 12

[Reed Loden [:reed]]

Comment on attachment 396672 [details] [diff] [review]
combined patch v2 (tested)

>-            $regexp =~ s:\*(.):[^/]*\1:g;
>+            $regexp =~ s:\*(.):[^/]*$1:g;

Unrelated change that was just sitting on production, I guess?

>+        $svnroot = "svn+ssh://despotdaemon\@svn.mozilla.org/projects/mozilla.org/trunk";

Put this in a config variable in the config file?

>+        $ownerspath = "about/";

Instead of overwriting $ownerspath, can you just update the database?

Comment 13

[Takanori MATSUURA]

This bug lies at rest in the graveyard.