Bug#896071: debootstrap fails to retrive Release file over https

2018-05-20 Thread Philipp Kern
On 5/20/18 12:30 PM, Hideki Yamane wrote: > On Sun, 20 May 2018 10:14:13 +0200 > Philipp Kern wrote: >> So the way it works with your patch is that local variables are >> inherited by called functions (but not the caller). So from and dest >> from just_get() are visible in

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-20 Thread Hideki Yamane
On Sun, 20 May 2018 10:14:13 +0200 Philipp Kern wrote: > So the way it works with your patch is that local variables are > inherited by called functions (but not the caller). So from and dest > from just_get() are visible in wgetprogress() and hence the construction > of $@

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-20 Thread Philipp Kern
On 5/20/18 1:24 AM, Hideki Yamane wrote: > On Sat, 19 May 2018 20:18:17 +0200 > Philipp Kern wrote: >> You local'ed from and dest and now don't pass it anymore to >> wgetprogress. How does this work? > It is passed to wget via $@ So the way it works with your patch is that

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-19 Thread Hideki Yamane
On Sat, 19 May 2018 20:18:17 +0200 Philipp Kern wrote: > You local'ed from and dest and now don't pass it anymore to > wgetprogress. How does this work? It is passed to wget via $@ -- Regards, Hideki Yamane henrich @ debian.org/iijmio-mail.jp

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-19 Thread Philipp Kern
On 19.05.2018 07:14, Hideki Yamane wrote: > On Mon, 14 May 2018 00:48:53 +0200 > Philipp Kern wrote: >> any new about incorporating Raphael's suggestion? There's still a grave >> bug opened against debootstrap right now (on a version that is in testing). > > Sorry for late

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-18 Thread Hideki Yamane
On Mon, 14 May 2018 00:48:53 +0200 Philipp Kern wrote: > any new about incorporating Raphael's suggestion? There's still a grave > bug opened against debootstrap right now (on a version that is in testing). Sorry for late reply, revert previous fix and modified functions as

Bug#896071: debootstrap fails to retrive Release file over https

2018-05-13 Thread Philipp Kern
Hideki, On 4/24/18 3:29 PM, Raphael Hertzog wrote: > On Mon, 23 Apr 2018, Hideki Yamane wrote: >> On Sun, 22 Apr 2018 09:40:54 +1000 >> David Margerison wrote: "$@" is extracted as '' and wget tries to fetch it and fails, then returns 1. >>> >>> Regarding

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-24 Thread Raphael Hertzog
On Mon, 23 Apr 2018, Hideki Yamane wrote: > Hi, > > On Sun, 22 Apr 2018 09:40:54 +1000 > David Margerison wrote: > > > "$@" is extracted as '' and wget tries to fetch it and fails, > > > then returns 1. > > > > Regarding the proposed fix, in general using $@

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-24 Thread Hideki Yamane
On Tue, 24 Apr 2018 14:07:30 +0200 Raphael Hertzog wrote: > Honestly, I'd prefer not to diverge from Debian and use plain http as well > so that we limit the risk of regression related to https support. > > In particular since our main http.kali.org host redirects to many

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-24 Thread Raphael Hertzog
Hi, On Tue, 24 Apr 2018, Hideki Yamane wrote: > I'll revert below your commit since this regression fix also solve it. > > > commit c1c20ed48e83fe9d4f3512c524f7734d4e1469ac > > Author: Raphaël Hertzog > > Date: Thu Apr 12 12:18:29 2018 +0200 > > > > Do not use HTTPS

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-24 Thread Hideki Yamane
Hi, I'll revert below your commit since this regression fix also solve it. > commit c1c20ed48e83fe9d4f3512c524f7734d4e1469ac > Author: Raphaël Hertzog > Date: Thu Apr 12 12:18:29 2018 +0200 > > Do not use HTTPS for Kali bootstrap script Please let me know if you

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-24 Thread Hideki Yamane
On Tue, 24 Apr 2018 00:52:10 +1000 David Margerison wrote: > needs to be changed to something like this >wgetprogress $a $b $c "$url" Unfortunately, changed to it like above but caused same error. diff --git a/functions b/functions index 1e41862..d54b07f 100644

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-23 Thread Hideki Yamane
Hi, On Sun, 22 Apr 2018 09:40:54 +1000 David Margerison wrote: > > "$@" is extracted as '' and wget tries to fetch it and fails, > > then returns 1. > > Regarding the proposed fix, in general using $@ without quotes is fragile. Most of the case, quotes is

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-21 Thread David Margerison
On 21 April 2018 at 21:32, Hideki Yamane wrote: > > Thanks for the report, and here's a proposal fix. > > [ ! "$VERBOSE" ] && NVSWITCH="-nv" > local ret=0 > if [ "$USE_DEBIANINSTALLER_INTERACTION" ] && [ "$PROGRESS_NEXT" ]; > then > -

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-21 Thread Hideki Yamane
control: tags -1 +confirmed control: tags -1 +patch On Thu, 19 Apr 2018 09:56:30 +0200 Alf Gaida wrote: > there was a change in behaviour with the latest upload: Thanks for the report, and here's a proposal fix. "$@" is extracted as '' and wget tries to fetch it and

Processed: Re: Bug#896071: debootstrap fails to retrive Release file over https

2018-04-21 Thread Debian Bug Tracking System
Processing control commands: > tags -1 +confirmed Bug #896071 [debootstrap] debootstrap fails to retrive Release file over https Added tag(s) confirmed. > tags -1 +patch Bug #896071 [debootstrap] debootstrap fails to retrive Release file over https Added tag(s) patch. -- 896071:

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-20 Thread Cyril Brulebois
Hi, Ben Hutchings (2018-04-19): > On Thu, 2018-04-19 at 09:56 +0200, Alf Gaida wrote: > > Package: debootstrap > > Version: 1.0.97 > > Severity: grave > > > > Dear Maintainer, > > > > there was a change in behaviour with the latest upload: > > > > 1 agaida@newdev ~ %

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-19 Thread Alf Gaida
No, sorry, no journal entries, no coredumps - nothing signature.asc Description: OpenPGP digital signature

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-19 Thread Ben Hutchings
On Thu, 2018-04-19 at 09:56 +0200, Alf Gaida wrote: > Package: debootstrap > Version: 1.0.97 > Severity: grave > > Dear Maintainer, > > there was a change in behaviour with the latest upload: > > 1 agaida@newdev ~ % sudo debootstrap --arch=amd64 --no-check-gpg >

Bug#896071: debootstrap fails to retrive Release file over https

2018-04-19 Thread Alf Gaida
Package: debootstrap Version: 1.0.97 Severity: grave Dear Maintainer, there was a change in behaviour with the latest upload: 1 agaida@newdev ~ % sudo debootstrap --arch=amd64 --no-check-gpg --include=apt-utils,bzip2,gnupg2,dirmngr,xz-utils,perl --variant=minbase sid ./fuck