All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dietmar Maurer <dietmar@proxmox.com>
To: Thomas Schmitt <scdbackup@gmx.net>, grub-devel@gnu.org
Subject: Re: grub-mkrescue problems in argp_parse
Date: Tue, 14 Apr 2015 09:23:26 +0200 (CEST)	[thread overview]
Message-ID: <1020823566.14.1428996206628.JavaMail.open-xchange@ronja.mits.lan> (raw)
In-Reply-To: <19949562642841787556@scdbackup.webframe.org>

> Dietmar Maurer wrote:
> > > After reverting the following commit everything works again:
> > > http://git.savannah.gnu.org/cgit/grub.git/commit/?id=cf47a2fba5852014bc59959c5e357e8313933414
> 
> Andrei Borzenkov wrote:
> > It chokes on -V which is "display version and exit" in argp.
> 
> Alternative -as mkisofs option:  -volid 'PVE'
> 
> With old grub-mkrescue(.in) of 1.99, the version option was -v,
> which would collide with the -as mkisofs option which has as alias
> the word "-verbose".

OK, that works - thanks!

> > I still
> > believe this patch was mistake; it leaves no possibility to
> > disambiguate between grub and external options.
> 
> So we are back at the reasons why i proposed to introduce
> a new tool  grub-mkiso.c  with clear distinction of arguments
> for grub-mkiso and xorriso, and to have an artlessly implemented
> but high-fidelity compatibility mode for emulating old
> grub-mkrescue(.in).

It is quite easy to detect if there is a '--' in the argument list.
So you could disable magic code if you find '--'?

> My proposal of such an artless shell-to-C translation is posted as
> 
>   http://lists.gnu.org/archive/html/grub-devel/2014-10/msg00000.html
> 
> (Dietmar's command line would be for new grub-mkiso then.)
> 
> 
> ----------------------------------------------------------------
> Some xorriso musings:
> 
> Dietmar Maurer wrote:
> > > # grub-mkrescue -o pve-cd.iso data-gz -- -z -r -V 'PVE' -c boot/boot.cat
> > > -isohybrid-gpt-basdat -partition_offset 16 -c boot/boot.ca
> 
> This seems to be mangled a bit. Especially the double
> option -c. Were there more options ?

This was just a mistake.

> -isohybrid-gpt-basdat might collide with xorriso options
> used by grub-mkrescue. (It is intended for ISOLINUX/GRUB2
> BIOS/EFI hybrids. E.g. debian-7.7.0-amd64-netinst.iso.)
> 
> -partition_offset 16 is not really compliant with UEFI specs,
> which prescribe a single MBR partition starting at block 1,
> if GPT is present.
> The whole ISOLINUX/GRUB2 isohybrid world does not care about
> that prescription. But grub-mkrescue does.

OK, I will try without those options.
 
> To save some intermediate disk space:
> The generic xorriso command
>   -set_filter_r --zisofs ...paths.in.iso... --
> can do the mkzftree(1) compression on the fly, if libz is
> linked in.

Many thanks for your help.

- Dietmar



  reply	other threads:[~2015-04-14  7:24 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <2037922098.114.1428942601173.JavaMail.open-xchange@ronja.mits.lan>
2015-04-13 17:06 ` grub-mkrescue problems in argp_parse Andrei Borzenkov
2015-04-13 17:16   ` Dietmar Maurer
2015-04-13 17:49     ` Andrei Borzenkov
2015-04-13 19:10       ` Thomas Schmitt
2015-04-14  7:23         ` Dietmar Maurer [this message]
2015-04-14  9:16           ` Thomas Schmitt
2015-04-14  9:58             ` Dietmar Maurer
2015-04-14 10:07             ` Dietmar Maurer
2015-04-14 10:34               ` Thomas Schmitt
2015-04-24 15:54       ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-04-24 16:31         ` Thomas Schmitt
2015-04-29 14:05           ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-04-29 16:42             ` Andrei Borzenkov
2015-04-29 16:57             ` Thomas Schmitt
2015-05-07 14:40               ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-05-07 17:19                 ` Andrei Borzenkov
2015-05-07 18:41                   ` Thomas Schmitt
2015-05-07 18:19                 ` Thomas Schmitt
2015-04-24 17:14         ` Andrei Borzenkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1020823566.14.1428996206628.JavaMail.open-xchange@ronja.mits.lan \
    --to=dietmar@proxmox.com \
    --cc=grub-devel@gnu.org \
    --cc=scdbackup@gmx.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.