#612 closed enhancement (fixed)
Support for multiple <reference>s
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | nALFS - Back End (XML Handlers) | Version: | SVN |
Severity: | minor | Keywords: | |
Cc: |
Description
When fetching file from <reference> fails, we should check for another one.
Change History (11)
comment:1 by , 21 years ago
qa_contact: | alfs-discuss@linuxfromscratch.org → alfs-log@linuxfromscratch.org |
---|
comment:2 by , 21 years ago
Owner: | changed from | to
---|---|
Version: | CVS → 1.1.7 |
comment:3 by , 21 years ago
comment:4 by , 21 years ago
Owner: | changed from | to
---|---|
Priority: | lowest → normal |
comment:5 by , 21 years ago
dependson: | → 671 |
---|
comment:6 by , 21 years ago
Status: | new → assigned |
---|
comment:8 by , 21 years ago
Keywords: | VERIFIED added |
---|---|
Version: | 1.1.7 → CVS |
I would add some new functions in there, it started to turn into spaghetti code a bit (continue, break, goto -- all in one).
But it's not a big deal of course -- it works, which is important. :-)
comment:9 by , 21 years ago
Yeah, my first plan was to create get_urls() and pass it the name of the multi-instance element that holds the URLs. Problem with that is that <download> and <unpack> construct their URLs differently (<reference> is a complete URL, where <url> is only a prefix), so that would get_urls() would need a callback to a function to create the final URL... too much pain for little gain. Maybe just break out the code into a separate function in each handler, but not try to consolidate them.
When it comes time to add <reference> support to <patch> as I want to do, I'll revisit this topic.
comment:10 by , 21 years ago
Keywords: | VERIFIED removed |
---|
comment:11 by , 20 years ago
Version: | CVS → SVN |
---|
Will combine code from new-download and new-unpack into library this weekend, which will resolve this issue. Please assign to me.