Hi,

please make sure this works with the following entries in sources.list:

deb http://host/path suite component
deb http://host/path dir/
deb [ arch=amd64 ] http://host/path suite component
deb [ arch=amd64 ] http://host/path dir/
deb [ arch=armel ] http://host/path suite component
deb [ arch=armel ] http://host/path dir/
deb [foo=bar;arch=amd64,armel;key=0x2763452] http://host/path suite component
deb [foo=bar;arch=amd64,armel;key=0x2763452] http://host/path dir/

Assuming it runs on amd64 it shouldn't download from the arch=armel entries.


Further in Line 277 I would suggest:

-    $apt = new IO::File("LC_ALL=C apt-cache --all-versions show $package |")
+    $apt = new IO::File("LC_ALL=C apt-cache --all-versions show 
$package=$version |")

Line 293 should match for ^$ or end of input. Description might not always be
the last entry.

I can also think of two ways to make this function more robust:

1) Parse apt-get install [--reinstall] --print-uris $package
2) aptitude download $package

Last please Recommend or Suggest apt (and aptitude if you use that).

MfG
        Goswin



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to