#4 Sort Mirrors by Location

closed
leem
5
2007-11-18
2007-06-08
No

For Instalinux, certain distros are available from mirrors all over the world. The list in the pull-down menu is sorted by URL, and it would make more sense for it to be sorted by Location.

Discussion

  • leem

    leem - 2007-06-09

    Logged In: YES
    user_id=1316443
    Originator: NO

    Yes, today the list is loaded into a %hash and returned sorted by hostname, not location. A poor choice on my part in retrospect. It'll require a tweak to LinuxCOE.pm and coe_bootimage, but should be pretty harmless. The DBM version vs. MySQL of osvend (which is typically only used by small sites) has no location field. What would be the desired behavior there?

     
  • Chris Slater

    Chris Slater - 2007-06-09

    Logged In: YES
    user_id=1322142
    Originator: YES

    I found this in my current version of coe_bootimage.cgi

    > # foreach my $waystation (sort(@waystations)) { ## Original
    > foreach my $waystation (@waystations) {

    I think the fix is to put an order by in coedb.pm and then change this line...

     
  • leem

    leem - 2007-06-28
    • assigned_to: nobody --> lmayes
     
  • leem

    leem - 2007-06-28
    • status: open --> pending
     
  • leem

    leem - 2007-06-28

    Logged In: YES
    user_id=1316443
    Originator: NO

    Fixed in COEdb.pm.in version 1.5, let me know if that works for you.

    Lee

     
  • Chris Slater

    Chris Slater - 2007-06-28
    • status: pending --> open
     
  • Chris Slater

    Chris Slater - 2007-06-28

    Logged In: YES
    user_id=1322142
    Originator: YES

    Hmmm... I pulled down the latest from CVS. I can see the "order by" in the sql statement, but I'm still getting them sorted by hostname.

     
  • Chris Slater

    Chris Slater - 2007-06-28

    Logged In: YES
    user_id=1322142
    Originator: YES

    In coe_bootimage around line 948:

    foreach my $waystation (sort(@waystations)) {

    If I change it to this, it works:

    foreach my $waystation (@waystations) {

     
  • leem

    leem - 2007-06-28

    Logged In: YES
    user_id=1316443
    Originator: NO

    You are absolutely correct. I did my tests by altering location but they all had the same hostname (just duping rows in database). oops! coe_bootimage.in changed to stop the secondary sort by hostname.

    /cvsroot/linuxcoe/SystemDesigner/cgi-bin/coe_bootimage.in,v <-- coe_bootimage.in
    new revision: 1.56; previous revision: 1.55

    Thanks for the feedback.

    Lee

     
  • leem

    leem - 2007-11-03
    • status: open --> pending
     
  • SourceForge Robot

    Logged In: YES
    user_id=1312539
    Originator: NO

    This Tracker item was closed automatically by the system. It was
    previously set to a Pending status, and the original submitter
    did not respond within 14 days (the time period specified by
    the administrator of this Tracker).

     
  • SourceForge Robot

    • status: pending --> closed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks