Re: svn commit: r1896485 - /subversion/site/staging/download.html
On Wed, Jan 5, 2022 at 1:03 PM Daniel Sahlberg wrote: > > Den ons 5 jan. 2022 kl 16:22 skrev Nathan Hartman : >> So, I think it might be a good idea to revert this change and let >> Infra manage how they supply those arrays; if it seems as though >> they're providing an incorrect listing, it should be fixed on their >> end because that will affect all projects, not just Subversion. > > > Done, in r1896725. Thanks! >> The removal of "You may consult the complete list of mirrors" link >> should *not* be reverted. That page is now a 404. > > > That was a separate commit (for easy reverting :-) ). > > I think we should now merge staging => publish, everything looks good to me. > Do you want to check and do the honors? Sure. I will check it one more time in the morning and then do it... Cheers, Nathan
Re: svn commit: r1896485 - /subversion/site/staging/download.html
Den ons 5 jan. 2022 kl 16:22 skrev Nathan Hartman : > Interesting. On publish, previously the "Other mirrors" list was > showing me dlcdn.a.o for both the main and backup. > > As of right now, I am getting two options on publish: > https://dlcdn.apache.org/ > https://downloads.apache.org/ (backup) > > While on staging, I am only getting > https://dlcdn.apache.org/ > > So, I think it might be a good idea to revert this change and let > Infra manage how they supply those arrays; if it seems as though > they're providing an incorrect listing, it should be fixed on their > end because that will affect all projects, not just Subversion. > Done, in r1896725. > The removal of "You may consult the complete list of mirrors" link > should *not* be reverted. That page is now a 404. > That was a separate commit (for easy reverting :-) ). I think we should now merge staging => publish, everything looks good to me. Do you want to check and do the honors? /Daniel
svn commit: r1896725 - /subversion/site/staging/download.html
Author: dsahlberg Date: Wed Jan 5 17:54:47 2022 New Revision: 1896725 URL: http://svn.apache.org/viewvc?rev=1896725&view=rev Log: In site/staging: Revert r1896485. * download.html EZT lua doesn't support [else], and [http] and [backup] are arrays so no use comparing. Modified: subversion/site/staging/download.html Modified: subversion/site/staging/download.html URL: http://svn.apache.org/viewvc/subversion/site/staging/download.html?rev=1896725&r1=1896724&r2=1896725&view=diff == --- subversion/site/staging/download.html (original) +++ subversion/site/staging/download.html Wed Jan 5 17:54:47 2022 @@ -73,9 +73,9 @@ Other mirrors: [if-any ftp] [for ftp][ftp][end] [end] - [if-any backup][is backup http][else][# Only show backup if different from http ] + [if-any backup] [for backup][backup] (backup)[end] - [end][end] + [end]
Re: svn commit: r1896485 - /subversion/site/staging/download.html
On Wed, Jan 5, 2022 at 7:45 AM Daniel Sahlberg wrote: > > Den tis 28 dec. 2021 kl 21:14 skrev : >> >> Author: dsahlberg >> Date: Tue Dec 28 20:14:17 2021 >> New Revision: 1896485 >> >> URL: http://svn.apache.org/viewvc?rev=1896485&view=rev >> Log: >> In site/staging: The ASF download script currently return the same site >> for both "http" and "backup" (after the migration to dlcdn.a.o). >> >> * download.html: Avoid displaying the same URL twice in the >> >> Modified: >> subversion/site/staging/download.html >> >> Modified: subversion/site/staging/download.html >> URL: >> http://svn.apache.org/viewvc/subversion/site/staging/download.html?rev=1896485&r1=1896484&r2=1896485&view=diff >> == >> --- subversion/site/staging/download.html (original) >> +++ subversion/site/staging/download.html Tue Dec 28 20:14:17 2021 >> @@ -73,9 +73,9 @@ Other mirrors: >>[if-any ftp] >> [for ftp]> selected="selected"[end]>[ftp][end] >>[end] >> - [if-any backup] >> + [if-any backup][is backup http][else][# Only show backup if different >> from http ] >> [for backup]> selected="selected"[end]>[backup] (backup)[end] >> - [end] >> + [end][end] >> >> >> > > > The reasoning behind this change is described in the log message but I'm not > happy about it for a few reasons: > * [http] and [backup] are arrays, so comparing these are probably not a good > idea. I can't come up with anything reasonably simple though. > * The lua version of EZT doesn't seem to support [else] at all. The fact that > it works for me is probably purely accidental. > > I have a strong feeling that the download page will not work in case > different URLs are returned from the "closer.lua" script. Please check if you > are getting different lists in [http] and [backup] from > https://subversion-staging.apache.org/download.cgi?as_json=1 (I'm getting > https://dlcdn.apache.org/ on both of these). In that case check if the > looks reasonable at > https://subversion-staging.apache.org/download.cgi. Also compare with our > published download page at https://subversion.apache.org/download.cgi. > > I'm inclined to revert this change and live with the fact that the download > page can look slightly silly (displaying dlcdn.a.o as both the main and the > backup mirror) until ASF Infra check on closer.lua (or someone else find a > more clever way of writing the EZT comparison code). Interesting. On publish, previously the "Other mirrors" list was showing me dlcdn.a.o for both the main and backup. As of right now, I am getting two options on publish: https://dlcdn.apache.org/ https://downloads.apache.org/ (backup) While on staging, I am only getting https://dlcdn.apache.org/ So, I think it might be a good idea to revert this change and let Infra manage how they supply those arrays; if it seems as though they're providing an incorrect listing, it should be fixed on their end because that will affect all projects, not just Subversion. The removal of "You may consult the complete list of mirrors" link should *not* be reverted. That page is now a 404. Cheers, Nathan
Re: svn commit: r1896485 - /subversion/site/staging/download.html
Den tis 28 dec. 2021 kl 21:14 skrev : > Author: dsahlberg > Date: Tue Dec 28 20:14:17 2021 > New Revision: 1896485 > > URL: http://svn.apache.org/viewvc?rev=1896485&view=rev > Log: > In site/staging: The ASF download script currently return the same site > for both "http" and "backup" (after the migration to dlcdn.a.o). > > * download.html: Avoid displaying the same URL twice in the > > Modified: > subversion/site/staging/download.html > > Modified: subversion/site/staging/download.html > URL: > http://svn.apache.org/viewvc/subversion/site/staging/download.html?rev=1896485&r1=1896484&r2=1896485&view=diff > > == > --- subversion/site/staging/download.html (original) > +++ subversion/site/staging/download.html Tue Dec 28 20:14:17 2021 > @@ -73,9 +73,9 @@ Other mirrors: >[if-any ftp] > [for ftp] selected="selected"[end]>[ftp][end] >[end] > - [if-any backup] > + [if-any backup][is backup http][else][# Only show backup if different > from http ] > [for backup] selected="selected"[end]>[backup] (backup)[end] > - [end] > + [end][end] > > > > The reasoning behind this change is described in the log message but I'm not happy about it for a few reasons: * [http] and [backup] are arrays, so comparing these are probably not a good idea. I can't come up with anything reasonably simple though. * The lua version of EZT doesn't seem to support [else] at all. The fact that it works for me is probably purely accidental. I have a strong feeling that the download page will not work in case different URLs are returned from the "closer.lua" script. Please check if you are getting different lists in [http] and [backup] from https://subversion-staging.apache.org/download.cgi?as_json=1 (I'm getting https://dlcdn.apache.org/ on both of these). In that case check if the looks reasonable at https://subversion-staging.apache.org/download.cgi. Also compare with our published download page at https://subversion.apache.org/download.cgi. I'm inclined to revert this change and live with the fact that the download page can look slightly silly (displaying dlcdn.a.o as both the main and the backup mirror) until ASF Infra check on closer.lua (or someone else find a more clever way of writing the EZT comparison code). Kind regards, Daniel
svn commit: r1896702 - /subversion/site/staging/mailing-lists.html
Author: dsahlberg
Date: Wed Jan 5 11:31:35 2022
New Revision: 1896702
URL: http://svn.apache.org/viewvc?rev=1896702&view=rev
Log:
In site/staging:
* mailing-lists.html
First attempt at making a lists.apache.org compatible search form.
Modified:
subversion/site/staging/mailing-lists.html
Modified: subversion/site/staging/mailing-lists.html
URL:
http://svn.apache.org/viewvc/subversion/site/staging/mailing-lists.html?rev=1896702&r1=1896701&r2=1896702&view=diff
==
--- subversion/site/staging/mailing-lists.html (original)
+++ subversion/site/staging/mailing-lists.html Wed Jan 5 11:31:35 2022
@@ -13,7 +13,7 @@
-
+
@@ -34,32 +34,24 @@
If you cannot find the answer in the Subversion book,
- please search the archives:
-
- https://www.google.com/custom";>
+ please search the archives (the search form links to
+ https://lists.apache.org";>lists.apache.org, a site requiring
+ JavaScript. If your browser doesn't support JavaScript, please check
+ another archive linked below!):
+
+ https://lists.apache.org/list"; id="f">
-
-
-
-
-
-
-
-
-
-
-
+
+ https://lists.apache.org/list?*@subversion.apache.org:gte=0d:'+document.getElementById('q').value;
return false;">
-
+
