Bug#985853: debian-installer: Whitespace before a commented line in preseed file causes line to be parsed

2021-04-11 Thread Samuel Thibault
Control: tags -1 + pending

Hello,

Holger Wansing, le sam. 10 avril 2021 12:32:57 +0200, a ecrit:
> Charles Curley  wrote (Thu, 25 Mar 2021 
> 12:06:32 -0600):
> > On Thu, 25 Mar 2021 10:49:06 -0400
> > lsore...@csclub.uwaterloo.ca (Lennart Sorensen) wrote:
> > 
> > > Well none of the examples ever have spaces before # for comments.
> > > The documentation page you linked to doesn't even mention comments at
> > > all. I would agree that perhaps it should.  I have certainly
> > > encountered file types before where comments had to have # at the
> > > start of the line.
> > 
> > May I suggest inserting the following as the last bullet point item at
> > the top of "B.3. Creating a preconfiguration file" the following:
> > 
> > A comment consist of a line which *starts* with a sharp character ("#")
> > and extends the length of that line.
> > 
> > https://www.debian.org/releases/stable/i386/apbs03.en.html
> > 
> > The emphasis on the word "starts" should probably be italics, or, in
> > HTML, .
> 
> Re-assigning to installation-guide

Commited, thanks!

Samuel



Bug#985853: debian-installer: Whitespace before a commented line in preseed file causes line to be parsed

2021-04-10 Thread Holger Wansing
Control: reassign -1 src:installation-guide

Charles Curley  wrote (Thu, 25 Mar 2021 
12:06:32 -0600):
> On Thu, 25 Mar 2021 10:49:06 -0400
> lsore...@csclub.uwaterloo.ca (Lennart Sorensen) wrote:
> 
> > Well none of the examples ever have spaces before # for comments.
> > The documentation page you linked to doesn't even mention comments at
> > all. I would agree that perhaps it should.  I have certainly
> > encountered file types before where comments had to have # at the
> > start of the line.
> 
> May I suggest inserting the following as the last bullet point item at
> the top of "B.3. Creating a preconfiguration file" the following:
> 
> A comment consist of a line which *starts* with a sharp character ("#")
> and extends the length of that line.
> 
> https://www.debian.org/releases/stable/i386/apbs03.en.html
> 
> The emphasis on the word "starts" should probably be italics, or, in
> HTML, .

Re-assigning to installation-guide


Holger


-- 
Holger Wansing 
PGP-Fingerprint: 496A C6E8 1442 4B34 8508  3529 59F1 87CA 156E B076



Bug#985853: debian-installer: Whitespace before a commented line in preseed file causes line to be parsed

2021-03-25 Thread Charles Curley
On Thu, 25 Mar 2021 10:49:06 -0400
lsore...@csclub.uwaterloo.ca (Lennart Sorensen) wrote:

> Well none of the examples ever have spaces before # for comments.
> The documentation page you linked to doesn't even mention comments at
> all. I would agree that perhaps it should.  I have certainly
> encountered file types before where comments had to have # at the
> start of the line.

May I suggest inserting the following as the last bullet point item at
the top of "B.3. Creating a preconfiguration file" the following:

A comment consist of a line which *starts* with a sharp character ("#")
and extends the length of that line.

https://www.debian.org/releases/stable/i386/apbs03.en.html

The emphasis on the word "starts" should probably be italics, or, in
HTML, .

-- 
Does anybody read signatures any more?

https://charlescurley.com
https://charlescurley.com/blog/



Bug#985853: debian-installer: Whitespace before a commented line in preseed file causes line to be parsed

2021-03-25 Thread Lennart Sorensen
On Thu, Mar 25, 2021 at 09:45:17AM +1030, Andrew McDonnell wrote:
> Package: debian-installer
> Version: 20190702+deb10u8
> Severity: important
> Tags: d-i
> 
> In a preseed file I accidentally had a space before a comment character, which
> caused my preseed to fail in unexpected ways. I could not find anythying that
> stood out in the documentation (e.g.
> https://www.debian.org/releases/buster/amd64/apbs04.en.html or
> https://www.debian.org/releases/stable/amd64/apbs03.en.html) stating that this
> would occur.
> 
> The specific example in my case looked like this:
> 
> #_preseed_V1
> d-i debian-installer/locale string en_AU
> d-i keyboard-configuration/xkb-keymap select us
> d-i keymap select us
> ... etc ...
> # Example of fetching a script to run
>  #d-i preseed/run string http://10.1.2.3/my-script.sh
> 
> 
> My install was hanging and when I entered a console and looked in the syslog,
> it was attempting to access that script for which the IP address does not 
> exist
> on my network. I finally started to understand the problem when I did this, 
> the
> latter finally triggered a parse error in the installer console:
> 
>  #d-iWHATpreseed/run string http://10.1.2.3/my-script.sh
> 
> 
>  #d-iWHATpreseed/runISstringHAPPENINGhttp://10.1.2.3/my-script.sh
> 
> at this point I saw the white space, removed it and the problem went away.
> 
> (I am also unsure whether "d-iWHAT" is also a bug or just some default 
> applying
> if the item owner is not found)
> 
> So I guess that either
> - whitespace is disallowed before a comment character and this should be added
> to https://www.debian.org/releases/stable/amd64/apbs03.en.html - it mentioned
> whitespace between fields but not at the start of a line
> - this is a bug

Well none of the examples ever have spaces before # for comments.
The documentation page you linked to doesn't even mention comments at all.
I would agree that perhaps it should.  I have certainly encountered file
types before where comments had to have # at the start of the line.

-- 
Len Sorensen



Bug#985853: debian-installer: Whitespace before a commented line in preseed file causes line to be parsed

2021-03-24 Thread Andrew McDonnell
Package: debian-installer
Version: 20190702+deb10u8
Severity: important
Tags: d-i

In a preseed file I accidentally had a space before a comment character, which
caused my preseed to fail in unexpected ways. I could not find anythying that
stood out in the documentation (e.g.
https://www.debian.org/releases/buster/amd64/apbs04.en.html or
https://www.debian.org/releases/stable/amd64/apbs03.en.html) stating that this
would occur.

The specific example in my case looked like this:

#_preseed_V1
d-i debian-installer/locale string en_AU
d-i keyboard-configuration/xkb-keymap select us
d-i keymap select us
... etc ...
# Example of fetching a script to run
 #d-i preseed/run string http://10.1.2.3/my-script.sh


My install was hanging and when I entered a console and looked in the syslog,
it was attempting to access that script for which the IP address does not exist
on my network. I finally started to understand the problem when I did this, the
latter finally triggered a parse error in the installer console:

 #d-iWHATpreseed/run string http://10.1.2.3/my-script.sh


 #d-iWHATpreseed/runISstringHAPPENINGhttp://10.1.2.3/my-script.sh

at this point I saw the white space, removed it and the problem went away.

(I am also unsure whether "d-iWHAT" is also a bug or just some default applying
if the item owner is not found)

So I guess that either
- whitespace is disallowed before a comment character and this should be added
to https://www.debian.org/releases/stable/amd64/apbs03.en.html - it mentioned
whitespace between fields but not at the start of a line
- this is a bug



-- System Information:
Debian Release: bullseye/sid
  APT prefers focal-updates
  APT policy: (500, 'focal-updates'), (500, 'focal-security'), (500, 'focal')
Architecture: amd64 (x86_64)

Kernel: Linux 5.8.0-36-lowlatency (SMP w/8 CPU cores; PREEMPT)
Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE
Locale: LANG=en_AU.UTF-8, LC_CTYPE=en_AU.UTF-8 (charmap=UTF-8), LANGUAGE= 
(charmap=UTF-8)
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled