Re: [PATCH 2/2] build: Move environment scripts to 'build-aux' directory.

[Alex Kost]

Mathieu Lirzin (2016-04-03 20:18 +0300) wrote:

> Alex Kost <address@hidden> writes:
>
>> Mathieu Lirzin (2016-04-03 19:18 +0300) wrote:
>>
>>> * pre-inst-env.in: Move to ...
>>> * build-aux/pre-inst-env.in: ... here.
>>> * test-env.in: Move to ...
>>> * build-aux/test-env.in: ... here.
>>> * configure.ac (AC_CONFIG_FILES): Adapt to this.
>>> ---
>>>  build-aux/pre-inst-env.in |  71 +++++++++++++++++++++++++
>>>  build-aux/test-env.in     | 131 
>>> ++++++++++++++++++++++++++++++++++++++++++++++
>>>  configure.ac              |   5 +-
>>>  pre-inst-env.in           |  71 -------------------------
>>>  test-env.in               | 131 
>>> ----------------------------------------------
>>>  5 files changed, 205 insertions(+), 204 deletions(-)
>>>  create mode 100644 build-aux/pre-inst-env.in
>>>  create mode 100644 build-aux/test-env.in
>>>  delete mode 100644 pre-inst-env.in
>>>  delete mode 100644 test-env.in
>>
>> Hm, this is a radical change :-) I'm afraid moving "pre-inst-env" may
>> break workflow of many people.  Anyway, if this change will be done,
>> "guix.texi" should also be adjusted as "pre-inst-env" is mentioned in
>> the manual multiple times.
>
> Sorry my patch summary line was confusing.  The generated files
> "pre-inst-env" and "test-env" aren't moving to build-aux. This would be
> be really inconvenient in the case of "pre-inst-env".  The patch
> concerns only the source files "pre-inst-env.in" and "test-env.in".  ;)

Oops, my bad, sorry for the noise :-)

-- 
Alex