GNU bug report logs - #12793
build: configure should warn about 'root' earlier (

Previous Next

Package: coreutils;

Reported by: Michael Felt <mamfelt <at> gmail.com>

Date: Sat, 3 Nov 2012 14:40:01 UTC

Severity: wishlist

Done: Assaf Gordon <assafgordon <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #14 received at 12793 <at> debbugs.gnu.org (full text, mbox):

From: Eric Blake <eblake <at> redhat.com>
To: Pádraig Brady <P <at> draigBrady.com>
Cc: Michael Felt <mamfelt <at> gmail.com>, 12793 <at> debbugs.gnu.org,
	bug-gnulib <bug-gnulib <at> gnu.org>
Subject: Re: bug#12793: FORCE_UNSAFE_CONFIGURE=1 - not really a bug,
	but friendliness issue
Date: Mon, 05 Nov 2012 08:12:52 -0700
[Message part 1 (text/plain, inline)]
[adding gnulib]

On 11/03/2012 10:34 AM, Pádraig Brady wrote:
> On 11/03/2012 02:36 PM, Michael Felt wrote:
>> Would be much nicer if ./configure complained at the beginning about
>> being
>> root, rather than at the end.
> 
> This comes from a specific test in gnulib:
> http://git.sv.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=5c53ef81
> rather than a coreutils policy decision.
> 
> I suppose that check could be hoisted to
> one that is done up front?

We can't directly host the entire test earlier - the test as written
can't detect failure under root until after the C compiler is known
(since it first tests for $? = 99 before checking the environment
variable).  Hoisting the entire test into GL_EARLY means it would come
before the compiler checks, which won't work.

But perhaps we could isolate just the environment variable test to come
first, without regards to the compiler test.  I'll play with that idea.

-- 
Eric Blake   eblake <at> redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

[signature.asc (application/pgp-signature, attachment)]

This bug report was last modified 6 years and 281 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.