931100 - Remove already-redirected en-US pages from SVN

Status: RESOLVED FIXED

(www.mozilla.org :: Legacy PHP system, defect)

Description

[Steven Garrity [:sgarrity]]

There are some PHP pages and supporting assets (img/css/js) remaining in the mozilla.org SVN repo for pages that are already redirect to Bedrock. I'm working on a list of such files and will use this bug to track their removal.

Comment 1

[Steven Garrity [:sgarrity]]

A rough list to start for myself:
http://www.mozilla.com/en-US/firefox/geolocation
http://www.mozilla.org/en-US/firefox/happy/
http://www.mozilla.org/en-US/firefox/organizations/
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/firefox/new/index.html
http://www.mozilla.com/en-US/firefox/performance
http://www.mozilla.com/en-US/firefox/releases/index.html
http://www.mozilla.com/en-US/firefox/security
http://www.mozilla.org/en-US/firefox/technology/
http://www.mozilla.com/en-US/firefox/try
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/firefox/underthehood/
http://www.mozilla.com/en-US/firefoxos/1.0.1

Comment 2

[Steven Garrity [:sgarrity]]

Attachment: Patch removing en-US files already unused in SVN

Comment 3

[Paul [:pmac] McLanahan]

Comment on attachment 824796 [details] [diff] [review]
Patch removing en-US files already unused in SVN

Review of attachment 824796 [details] [diff] [review]:
-----------------------------------------------------------------

Looks okay to me.

Comment 4

[Steven Garrity [:sgarrity]]

I've also got a handful of empty dirs in SVN I'd like to remove in this patch. Since SVN doesn't handle empty DIRs well (or I don't), please see this list: https://gist.github.com/sgarrity/7238747

Comment 5

[Paul [:pmac] McLanahan]

(In reply to Steven Garrity [:sgarrity] from comment #4)
> I've also got a handful of empty dirs in SVN I'd like to remove in this
> patch. Since SVN doesn't handle empty DIRs well (or I don't), please see
> this list: https://gist.github.com/sgarrity/7238747

That also looks good. Thanks for doing this!

Comment 6

[Steven Garrity [:sgarrity]]

Trunk in 121747. Merged to stage in r121748. Merged to production in r121749.

Comment 7

[Kohei Yoshino [:kohei]]

Looks like /firefox/organizations/all.html is still required. A regression has been reported as Bug 933055.

Comment 8

[Paul [:pmac] McLanahan]

BAH! Missed that one. Sorry everyone!