All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bruce Dubbs <bruce.dubbs@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: autogen.sh warnings
Date: Wed, 09 Dec 2009 17:28:21 -0600	[thread overview]
Message-ID: <4B203295.2020102@gmail.com> (raw)
In-Reply-To: <1260398055.2881.14.camel@fz.local>

Felix Zielcke wrote:
> Am Mittwoch, den 09.12.2009, 16:19 -0600 schrieb Bruce Dubbs:
>> config.guess, config.sub, missing, mkinstalldirs, and install-sh are 
>> only copied from /usr/share/automake-<version> as a part of automake. 
>> AFAICT, they are not used in GRUB.  I'm pretty sure they are the same
>> on 
>> all architectures, but I could be wrong about that. 
> 
> config.guess and config.sub are used by configure.

Called yes, and it will give errors if not present, but is the output 
used?  Looking at configure, I don't think so.

> mkinstalldirs is used by the generated Makefile.

It looks like a part of the generated install and dist targets.  Looking 
at the code of mkinstalldirs, it fundamentally does a mkdir -p.  It can 
change the permissions of the final directory, but that is not used by 
the Makefile.  It does do some cleanup for NextStep and OpenStep and 
makes sure that GNU mkdir is used on Sun.   The odds this will change in 
the future are very small.

> Looks like configure checks if install-sh is there but except of this it
> isn't used.

> missing isn't used because we don't use automake and maintainer mode.

Looks that way to me too.

   -- Bruce

P.S.  Some may ask why I am putting so much into the warnings generated 
by the build process.  There are a few reasons.  First, I feel that 
persistent warnings tend to make developers ignore ALL warnings and that 
makes it very easy to miss something relevant.  Second, it gives new 
developers a feeling that the original developers paid attention to 
details and will give confidence that the rest of the package is 
correct.  Third, it scratches an itch for me.  :)



  reply	other threads:[~2009-12-09 23:28 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-07 17:38 autogen.sh warnings Bruce Dubbs
2009-12-07 19:21 ` Colin Watson
2009-12-07 20:28   ` Bruce Dubbs
2009-12-07 22:12     ` Vladimir 'φ-coder/phcoder' Serbinenko
2009-12-07 23:07       ` Bruce Dubbs
2009-12-07 22:40     ` Colin Watson
2009-12-07 23:26       ` Bruce Dubbs
2009-12-09 21:52   ` Robert Millan
2009-12-09 22:19     ` Bruce Dubbs
2009-12-09 22:34       ` Felix Zielcke
2009-12-09 23:28         ` Bruce Dubbs [this message]
2009-12-10  0:25           ` Felix Zielcke
2009-12-10  6:05             ` Bruce Dubbs
2009-12-10 10:15               ` Felix Zielcke
2009-12-10 16:20                 ` Bruce Dubbs
2009-12-24 21:13                   ` Robert Millan
2009-12-29  3:50                     ` Bruce Dubbs
2010-01-01 11:33                       ` Robert Millan
2010-01-01 16:57                         ` Bruce Dubbs
2010-01-03 16:25                           ` Robert Millan
2009-12-10  0:49           ` Robert Millan
2009-12-09 22:56       ` Robert Millan
2009-12-07 19:50 ` Vladimir 'φ-coder/phcoder' Serbinenko
2009-12-07 20:51   ` Bruce Dubbs
2009-12-07 22:16     ` Vladimir 'φ-coder/phcoder' Serbinenko
2009-12-07 22:49     ` Bruce Dubbs
2009-12-07 23:54       ` Colin Watson
2009-12-08  0:01         ` Vladimir 'φ-coder/phcoder' Serbinenko
2009-12-08  0:13           ` Bruce Dubbs
2010-01-07 19:14             ` Robert Millan
2009-12-08  0:16           ` Colin Watson
2009-12-08  0:34             ` Building system (Re: autogen.sh warnings) Vladimir 'φ-coder/phcoder' Serbinenko
2009-12-08  3:40               ` Bruce Dubbs
2009-12-08 10:20         ` autogen.sh warnings Felix Zielcke

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=4B203295.2020102@gmail.com \
    --to=bruce.dubbs@gmail.com \
    --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.