Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 205247

Summary:

Add metalink download

Product:

Community

Reporter:

Bram <bneijt>

Component:

Website

Assignee:

phoenix.ui <phoenix.ui-inbox>

Status:

RESOLVED WONTFIX

QA Contact:

Severity:

enhancement

Priority:

CC:

anthonybryan

Version:

unspecified

Target Milestone:

---

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Additional template, should be in www/downloads/content/	none

Description Bram

2007-10-02 18:03:47 EDT

Created attachment 79595 [details]
Additional template, should be in www/downloads/content/

Metalinks are XML download files, which allow clients to use the mirror closest to them or switch when one if two bussy.

The Metalink format is close to the XML list already available, so it should be trivial to implement a working version of this and add it to the download section.

I've attached a patch (diff on download.php and an extra file) to allow for this. This file has however not been tested as I don't have the database information or database structure at this moment.

I consider both attachments Public Domain, although they do inherit their base structure from code by Dennis Roy (as mentioned in the header), which did not have any licensing information attached. This means that anybody willing to take up this code should look into or be Dennis Roy ;-)

I could only add one patch and couldn't add the file to CVS to let that create the diff, so the following diff for www/downloads/download.php is inline:
diff -r1.84 download.php
473a474,477
>                       if($_format == "metalink") {
>                               $app->sendXMLHeader();
>                               include("content/en_mir_list_metalink.php");
>                       }

(The if was added just after $_format == "xml".)

Comment 1 Denis Roy

2007-10-04 10:07:27 EDT

Thanks for the patch. We'll have a look at this.

> I could only add one patch and couldn't add the file to CVS to let that create
> the diff

You should use Eclipse.. The team/patch/diff tools are most excellent  ;-)

Comment 2 Ant Bryan

2007-11-16 19:44:44 EST

(In reply to comment #1)
> Thanks for the patch. We'll have a look at this.
> 
> > I could only add one patch and couldn't add the file to CVS to let that create
> > the diff
> 
> You should use Eclipse.. The team/patch/diff tools are most excellent  ;-)

Just curious, what's the status on this? 

Would it help if the submitter used Eclipse? :)

Comment 3 Denis Roy

2008-04-22 15:28:53 EDT

There doesn't seem to be much interest in this, so I'll close as WONTFIX.  Thanks for the patch, though.