guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] gnu: Add xcompmgr


From: Paul van der Walt
Subject: Re: [PATCH] gnu: Add xcompmgr
Date: Fri, 23 Oct 2015 15:13:43 +0200

Hey Cyrill,

On 2015-10-23 at 16:45, quoth Cyrill Schenkel:
>> On 2015-10-23 at 11:46, quoth Paul van der Walt:
>>>> +                                (setenv "NOCONFIGURE" "t")
>>
>> Does that do what i think it does?  If so, why not just use something
>> like the following?
>>
>>   (modify-phases %standard-phases
>>     (delete 'configure)
>>     ...)
>
> It does what you think it does. If I did omit it, I'd need to patch the
> `autogen.sh` file so that it passes the neccessary parameters to the
> configure script and I'd also need to set the neccessary environment
> variables. It seems to me that setting the `NOCONFIGURE` environment
> variable is by far the simpler solution.

Ah, in fact, i misunderstood and thought it did something else :).  I
blame my ambiguous communication.  I thought it would simply make the
configure script not run, at all.  Of course, that makes no sense.  The
parameter seems to stop *autogen.sh* from running `configure`, which is
reasonable, since gnu-build-system will do so with the right parameters.

Your version seems right indeed, i was simply confused; sorry about
that!

Cheers,
p.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]