All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Thomas Schmitt" <scdbackup@gmx.net>
To: grub-devel@gnu.org
Subject: Re: How to prepare an ISO 9660 CD for booting via GRUB ?
Date: Sun, 11 Apr 2010 09:46:23 +0200	[thread overview]
Message-ID: <106354008331269@192.168.2.69> (raw)
In-Reply-To: <4BC10638.5070907@gmail.com>

Hi,

many thanks for testing xorriso in new
situations. 


> 1) It tries to build even if no iconv is present

The tests in configure.ac resp. acinclude.m4
are too lax, obviously. They aim for finding
out whether one needs -liconv or not. But they
don't assure that iconv() works.
I will have to make experiments with a
non-existing dummy function, as it seems
impossible to disable iconv() on Linux.


> 2) I needed to explicitly add LIBS=-liconv to configure depstite
> configure properly detecting the need of -liconv. In some commands
> actually -liconv was double: one from configure and another from LIBS
> but somewhere it's not propagated properly

This sounds a bit like a bug in autotools.

But in the log you sent, i only find them in
pairs. Obviously one from LIBS= and one from
  dnl GNU iconv has no function iconv() but libiconv() and a macro iconv()
  AC_CHECK_LIB(iconv, libiconv, , )

Are you sure that the lack of -liconv was not
due to libiconv missing initially ?


> 3) A ton of warnings.

The implementation of the isspace() family seems
not to be liked by the compiler. Shrug.

I found legitimate warnings about putting out
uid_t and gid_t by printf formatter "%d".
(Question is whether (int) is a suitable cast
 in this case.)

Then there is the intentional warning that no
MMC system adapter is available for operating
optical drives directly.
  libburn/sg.c:42: warning: unused variable INTENTIONAL_COMPILER_WARNING
  libburn/sg.c:39: warning: unused variable This_libburn_will_not_be_able_to_operate_on_real_CD_drives

This does not hamper the GRUB use case. One will
just need cdrecord for burning the CD, rather
than letting xorriso do:
  xorriso -as cdrecord -v dev=... blank=as_needed my_grub.iso


------------------------------------------------

So mainly the LIBS=-liconv remains as build
obstacle.

I have changed the iconv detection code in
configure.ac meanwhile (to care for the "const"
ambiguity on FreeBSD). A new tarball
  http://www.gnu.org/software/xorriso/xorriso-0.5.3.tar.gz
is uploaded.

  cat xorriso/xorriso_timestamp.h
should say
  #define Xorriso_timestamP "2010.04.10.180103"
or later.

Could you please try it on cygwin without LIBS=
(but libiconv installed) and send me the log of
the failing compile run ?

If nothing else helps then i plan to set
  LIBS="$LIBS -liconv"
in a special cygwin case in acinclude.m4.
(Do you know a reliable test pattern for cygwin ?
 Like *-*-linux* for Linux.)

For now the double -liconv options give me hope
that it could work without that.


Have a nice day :)

Thomas




  reply	other threads:[~2010-04-11  7:47 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-01 13:49 How to prepare an ISO 9660 CD for booting via GRUB ? Thomas Schmitt
2010-04-01 16:59 ` Frombenny
2010-04-01 17:59   ` Thomas Schmitt
2010-04-01 19:28     ` Drake Donahue
2010-04-01 20:43       ` Thomas Schmitt
2010-04-01 20:52         ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-01 21:47         ` Drake Donahue
2010-04-01 18:22 ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-01 20:25   ` Thomas Schmitt
2010-04-01 20:44     ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-01 22:05       ` Thomas Schmitt
2010-04-01 22:43         ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-02  9:32           ` Thomas Schmitt
2010-04-02 11:51             ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-02 17:10               ` Thomas Schmitt
2010-04-02 17:24                 ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-03  8:03                   ` Thomas Schmitt
2010-04-03 14:15                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-06 11:26                       ` Thomas Schmitt
2010-04-07 12:47                         ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-07 14:11                           ` Thomas Schmitt
2010-04-09 20:53                             ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-09 22:28                               ` Thomas Schmitt
2010-04-09 22:53                                 ` Seth Goldberg
2010-04-10  7:37                                   ` Thomas Schmitt
2010-04-10 11:09                                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-10 11:57                                       ` Thomas Schmitt
2010-04-10 12:27                                         ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-10 12:57                                           ` Thomas Schmitt
2010-04-10 15:22                                             ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-10 23:14                                 ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-11  7:46                                   ` Thomas Schmitt [this message]
2010-04-11 12:29                                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-11 12:56                                       ` Thomas Schmitt
2010-04-11 17:32                                         ` Colin Watson
2010-04-17 14:55                                         ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-17 17:52                                           ` Thomas Schmitt
2010-04-17 18:04                                             ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-17 19:39                                               ` Thomas Schmitt
2010-04-17 20:30                                                 ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-18 11:26                                                   ` Thomas Schmitt
2010-04-24 19:54                                                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-04-18 12:41                                                   ` Thomas Schmitt

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=106354008331269@192.168.2.69 \
    --to=scdbackup@gmx.net \
    --cc=grub-devel@gnu.org \
    /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.