Closed Bug 647898 Opened 9 years ago Closed 9 years ago
Links from Sea
Monkey Nightly Builds page are broken
As reported in the MozillaZine forums a couple of days ago, the nightly-build links from the web site are broken and cause a "not found" error. http://www.seamonkey-project.org/dev/nightly The reason appears to be that they try to find 2.1b3pre in comm-central-trunk.
Either make it seamonkey/nightly/latest-comm-2.0/seamonkey-2.1b3pre.* or seamonkey/nightly/latest-comm-central-trunk/seamonkey-2.2a1pre.* instead, where the comm-central/mozilla-2.0 builds may be the preferred ones until SeaMonkey 2.1 is closing in on release and comm-central/mozilla-central gets the attention again.
The problem there is that's I'm not sure that linking to 2.2a1pre builds is what we want there, but linking to comm-2.0 would need a larger change in the website template to do it correctly. Linking to 2.2a1pre would just be a change of the variable in the config.tmpl file (I think that's what it's called).
(In reply to comment #2) > The problem there is that's I'm not sure that linking to 2.2a1pre builds is > what we want there, Agreed, the last comm-central/mozilla-central nightly was March 31, so that's spotty (and likely buggy) at best. > but linking to comm-2.0 would need a larger change in the > website template to do it correctly. If you look at it from the positive side, this may be a good opportunity to make the template more general to be covered in future similar situations. ;-)
Summary: Links from SeaMonkey Nightly Builds are broken → Links from SeaMonkey Nightly Builds page are broken
Looking at seamonkeyproject-org/src/dev/nightly.en.html, the path constructs as [% develdownloadbase %]/seamonkey-[% develversion %].en-US.*, thus another level for the branch would be needed with a similar variable in config.tmpl, [% develdownloadbase %]-[% develbranch %]/seamonkey-[% develversion %].en-US.* develdownloadbase = http://.../latest-comm develbranch = 2.0 develversion = 2.1b3pre This might be the most flexible solution.
I don't see why it cannot be this simple. What did I miss?
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #525412 - Flags: review?(kairo)
>-[% develdownloadbase = 'http://ftp.mozilla.org/pub/mozilla.org/seamonkey/nightly/latest-comm-central-trunk' %] >+[% develdownloadbase = 'http://ftp.mozilla.org/pub/mozilla.org/seamonkey/nightly/latest-comm-2.0' %] Yes, I was thinking about this "too simple solution" after posting comment #4, but wasn't sure if it's used anywhere else too - apparently it isn't. :-)
Comment on attachment 525412 [details] [diff] [review] patch [Checkin: comment 9] Oh, cool, I _did_ build in a variable for this. Cool, thanks for looking it up. :)
Attachment #525412 - Flags: review?(kairo) → review+
(oh, and thanks for spotting the mac thing as well)
Comment on attachment 525412 [details] [diff] [review] patch [Checkin: comment 9] Checking in lib/config.tmpl; /www/seamonkeyproject-org/lib/config.tmpl,v <-- config.tmpl new revision: 1.54; previous revision: 1.53 done Checking in src/dev/nightly.en.html; /www/seamonkeyproject-org/src/dev/nightly.en.html,v <-- nightly.en.html new revision: 1.5; previous revision: 1.4 done
Attachment #525412 - Attachment description: patch → patch [Checkin: comment 9]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.