[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: jigdo-file: Does not report package rejections because checksum mismatch



On Fri, Dec 29, 2017 at 5:35 AM, Thomas Schmitt <scdbackup@gmx.net> wrote:
> I wrote:
>> > > >                   sed -e 's/^[hH][tT][tT][pP]:\/\///' \
>> > > >                        ... more -e for https, ftp, and file ...
>
> Philip Hands wrote:
>> > >   sed -e 's,^\(https\?\|ftp\|file\)://,,i'
>> > > ...
>> > >  "$imageTmp/${url#[[:alpha:]]*://}"
>
>> > Are these widely portable enough ?
>
>> [Philip Hands analyzed portability with positive result]
>
> I still think that the long explicit sed is clearer.

FWIW I agree.

And with respect to the new awk dependency mentioned earlier, it's a simple
expression and awk tempts me with that all the time because it's so much easier.
Just you know, be advised that on say Debian 8.9 it's gawk while on Ubuntu 17.10
it's mawk, etc. Debian 9.2 is still gawk.

> ....
> The effective throughput of roughly 1.5 to 2.5 MB/s is still much slower
> than wget's speed report of about 5.5 MB/s.
> I tried with 100 files per run of wget and "jigdo-file make-image".
> No significant difference to see. It's all about mirror server latency
> with each single file, i guess.
> us.cdimage.debian.org is a quick one.

>From which location? Germany?
I find it a bit slower than average from here inside the US, but I
have large, local
mirrors. And at long last, one of my own is budgeted in 2018. You betcha ;-)

> I wrote:
>> > > > fileMD5=`$jigdoFile md5 "$localPath" 2>/dev/null | awk '{print $1}'`
>
> Philip Hands wrote:
>> `$jigdoFile md5sum --report=quiet "$localPath" | sed 's/ .*$//'`
>
> I'll take this one.

Oh, OK. Yeah, better.

> Next development step will be to issue a correct "Aaargh" message and to
> tell at least some of the mismatching files in that message.
> Have a nice day :)

Vielen Dank Thomas.

> Thomas
>


Reply to: