Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 110350 - Mirror-list seems to have changed, missing colons confuse mirrorselect
Summary: Mirror-list seems to have changed, missing colons confuse mirrorselect
Status: RESOLVED FIXED
Alias: None
Product: Websites
Classification: Unclassified
Component: Other (show other bugs)
Hardware: All All
: High normal
Assignee: Gentoo Infrastructure
URL: http://www.gentoo.org/main/en/mirrors...
Whiteboard:
Keywords:
: 110897 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-10-24 10:08 UTC by Markus Moll
Modified: 2005-11-01 08:50 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Moll 2005-10-24 10:08:14 UTC
The list of mirrors seems to have changed. I just realized that I cannot use    
mirrorselect any more. It only shows entries for Asia and "Other Mirrors".   
The problem seems to be related to the list however, since the parser in 
mirrorselect states: 
#these always have to have the ":" after them. tercel said so.   
                self.locations = ('North America:', 'South America:',   
'Europe:', \   
                                'Australia:', 'Asia:', 'Other Mirrors:')   
   
Now all but "Asia" and "Other Mirrors" do not have leading colons any more,   
and obviously they cannot be parsed. 

Reproducible: Always
Steps to Reproduce:
1. Download http://www.gentoo.org/main/en/mirrors.xml 
2.   
3.   
   
Actual Results:  
Colons were missing 

Expected Results:  
Colons should be there
Comment 1 Markus Moll 2005-10-24 10:25:19 UTC
Apparently there must have been more changes. 
When I add 'Germany' to the list of mirrors, I get all German mirrors. 
Apparently the country paragraphs also confuse mirrorselect. 
Comment 2 Colin Kingsley (RETIRED) gentoo-dev 2005-10-24 17:46:28 UTC
I will update the parser. Watch this bug for updates
Comment 3 David Fellows 2005-10-30 18:12:17 UTC
See my additonal comments re bug http://bugs.gentoo.org/show_bug.cgi?id=110897
Comment 4 Jeffrey Forman (RETIRED) gentoo-dev 2005-10-30 20:26:48 UTC
Colin,

Any update on this? I thought you were pushing out an update mirrorselect
Comment 5 David Fellows 2005-10-31 05:57:07 UTC
Gratuitous advice for Colin:
Before you hack your parser have a discussion with the folk that create
mirrors.xml and 
   a) establish exactly what you will parse
   b) create test cases (for them to run) that will ensure that they don't make 
      future changes that break your parsing.
Comment 6 Thomas van Kampen 2005-10-31 07:30:40 UTC
Perhaps a little suggestion (don't know if this is the right place, it probably is):

Make some kind of XML file with the mirrors in the form of (for example):
...
<location name="North America">
  <mirror title="Some mirror" type="FTP">ftp://ftp.some-mirror.tdl/</mirror>
  <mirror ... >
</location>
...

I have no real knowledge of XML, so this is just a little example. If it's done
that way it easier to parse for any interpreter applications or scripts. The
website itself can then also use this locally (PHP, perl, cgi? I don't know what
gentoo.org uses).

Ofcourse this would be a suggestion for the future because it involves some
thinking and work.
Comment 7 Colin Kingsley (RETIRED) gentoo-dev 2005-10-31 08:58:13 UTC
Thanks for the advice, but its all under controll:)

expect a new release in ~arch by tonight at midnight EST.
Comment 8 Colin Kingsley (RETIRED) gentoo-dev 2005-10-31 08:59:13 UTC
*** Bug 110897 has been marked as a duplicate of this bug. ***
Comment 9 Colin Kingsley (RETIRED) gentoo-dev 2005-10-31 18:51:51 UTC
New version commited to ~arch. Please test so we can get it stable asap, as it
is the only currently working version.
Comment 10 Markus Moll 2005-11-01 00:15:43 UTC
Works for me. 
But now there is another (minor) annoyance:  
mirrorselect now accidentally lists  
 - /doc/en/rsync.xml, and  
 - /doc/en/source_mirrors.xml  
as download mirrors. 
 
Anyway, thanks a lot. 
Comment 11 Thomas van Kampen 2005-11-01 01:42:32 UTC
Works very well now.

On top of Markus' comment, also http://ibiblio.org/pub/Linux/MIRRORS.html (under
Other Mirrors: ) is listed.
Comment 12 David Fellows 2005-11-01 07:07:42 UTC
Works for me (amd64). Not that I would expect it to be architecture dependent.

I agree with Markus' and Thomas' comments, but note that they only affect the -i
option.  

As an aside, it would be good to clarify the documentation of -i in the help to
make it clear that all it does is transcribe your selections to GENTOO_MIRRORS
with no warranty that the servers even exist. (At least that's my interpretation.)

Thanks.
Comment 13 Colin Kingsley (RETIRED) gentoo-dev 2005-11-01 08:50:50 UTC
reguarding #10 and #11, that is a separate problem. I'll be taking it up with
the maintainer of mirrors.xml