From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1XYHAp-0001GX-6W for mharc-grub-devel@gnu.org; Sun, 28 Sep 2014 12:18:11 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37527) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XYHAi-0001GB-ER for grub-devel@gnu.org; Sun, 28 Sep 2014 12:18:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XYHAc-0008Il-VY for grub-devel@gnu.org; Sun, 28 Sep 2014 12:18:04 -0400 Received: from mail-lb0-x22e.google.com ([2a00:1450:4010:c04::22e]:55037) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XYHAc-0008IV-No for grub-devel@gnu.org; Sun, 28 Sep 2014 12:17:58 -0400 Received: by mail-lb0-f174.google.com with SMTP id l4so16850300lbv.33 for ; Sun, 28 Sep 2014 09:17:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=TQS7xKGADV4seTQ9gVS2uEYJhXKl5k2L6j17kj4Y8V8=; b=thqPCE72/eYot/tKspCb8nAvm98h/Le0S4ij/lwURFc5ao0sFqNhJBy04wpGfU00fm 3Y8aqvUTkexYaH0VGHClAqauwqA200CiJL02W0CdgW9dt63d4PJvCvjkXc6yEueWOOAf HeoX7myKKmeBUanT07p3UjKbJbbKXSUMFe+h8iUQn9b1ZhNZrGm8kjvespT8o+QR+G88 w2u2EK5hCrtGnuMcppW60IXCYCckn+2WDJDY7wY/ADs+LserqmBm0M1ij9dkiRulhNO6 bkKfumbpLHt7j/chisL7kbNy38D21SMRk51k0xxQUD9yRjPq9hvxJm9OVWSgwrgD2g0E u69w== X-Received: by 10.112.52.68 with SMTP id r4mr32201834lbo.31.1411921072309; Sun, 28 Sep 2014 09:17:52 -0700 (PDT) Received: from opensuse.site (ppp94-29-94-89.pppoe.spdop.ru. [94.29.94.89]) by mx.google.com with ESMTPSA id s1sm3924303laj.12.2014.09.28.09.17.50 for (version=SSLv3 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 28 Sep 2014 09:17:51 -0700 (PDT) Date: Sun, 28 Sep 2014 20:17:49 +0400 From: Andrei Borzenkov To: The development of GNU GRUB Subject: Re: About the CLI of both grub-mkrescue versions Message-ID: <20140928201749.3b92c870@opensuse.site> In-Reply-To: <31041545521760249029@scdbackup.webframe.org> References: <31041545521760249029@scdbackup.webframe.org> X-Mailer: Claws Mail 3.9.2 (GTK+ 2.24.23; x86_64-suse-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2a00:1450:4010:c04::22e Cc: scdbackup@gmx.net X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Sep 2014 16:18:10 -0000 =D0=92 Tue, 23 Sep 2014 10:12:12 +0200 "Thomas Schmitt" =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > Hi, >=20 > i recently advertised grub-mkrescue as replacement of > script snippets for creating ISO images with legacy GRUB. >=20 > But it is not very appealing to the audience if i have > to mention the different interpretation of argument "--" > by the shell script of GRUB-2.00 and by the C program > of the git master branch. >=20 > The C program demands "--" before any custom options for > xorriso -as mkisofs are accepted. Omitting it lets argp_parser() > return ARGP_ERR_UNKNOWN rather than adding the mkisofs option > to the xorriso argument list. > The shell script forwards "--" to xorriso -as mkisofs, > where it ends the mkisofs emulation. Afterwards, the additional > mkisofs options are not understood by xorriso. >=20 > We discussed this a while ago in the thread following > http://lists.gnu.org/archive/html/grub-devel/2014-01/msg00074.html > There was no conclusion. >=20 >=20 > If this CLI change shall persist, then there needs to be > some indication for scripts, whether the "--" is mandatory or > harmful. > Further, the documentation of grub-mkrescue needs to be updated. Care to send a patch for both help output and documentation? > Including the help text in the C code, which currently describes > the behavior of the shell script. >=20 > In my personal view, it would be better to keep the behavior of > GRUB-2.00. E.g. by replacing in grub-mkrescue.c line 229 > return ARGP_ERR_UNKNOWN; > by a call of xorriso_push() ... i guess ... >=20 It won't work. Unknown options are detected before user parser is called; user parser never sees them. So either you reimplement argument processing or you have to bite the bullet and accept that behavior is now conforms to standard one ...=20 Or you can raise this issue on gnulib list, whether they can consider implementing this (passing unknown options through to user parser).