"Fabian Franz" <[EMAIL PROTECTED]> writes:

> Package: debmirror
> Severity: normal
> Followup-For: Bug #389894
>
> I think this should be fixed instead of workarounded.
>
> --- debmirror.old       2007-01-16 19:14:05.000000000 +0100
> +++ debmirror   2007-01-16 19:15:33.000000000 +0100
> @@ -1389,6 +1389,8 @@
>        }
>      } else {
>        $bytes_gotten += $file_lists_size{"$tempdir/$subdir/$file.gz"};
> +      system_redirect_io("gzip -d", "$tempdir/$subdir/$file.gz", 
> "$tempdir/$subdir/$file");
> +      system_redirect_io("bzip2", "$tempdir/$subdir/$file", 
> "$tempdir/$subdir/$file.bz2");

Wrong place. You don't know yet if you need to update the Packages and
Packages.bz2 files. Since that takes quite a while and happens all the
time (every time there is no update, which is a lot for stable or
repeat runs) it would be a big waste.

>      }
>    } elsif ($ignore_release) {
>      say("Ignoring missing Release file for $subdir/$file.gz");
>
> The check could of course also first check that Packages fails the check, but 
> this is up to the author.
>
> Besides. It is a good idea to not waste bandwidth on debian servers with big 
> packages files if this case occurs.
>
> cu
>
> Fabian

I have a better patch for this pending with some other changes that
might have to wait till after etch. I have to ask a release manager.

MfG
        Goswin


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to