From mboxrd@z Thu Jan  1 00:00:00 1970
From: Alex Kost <alezost@gmail.com>
Subject: Re: [PATCH 2/2] build: Move environment scripts to 'build-aux'
	directory.
Date: Mon, 04 Apr 2016 00:56:00 +0300
Message-ID: <87shz21hn3.fsf@gmail.com>
References: <1459700326-30016-1-git-send-email-mthl@gnu.org>
	<1459700326-30016-3-git-send-email-mthl@gnu.org>
	<871t6m39o2.fsf@gmail.com> <87d1q61uhq.fsf@gnu.org>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:50910)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <alezost@gmail.com>) id 1amq06-0003X9-KB
	for guix-devel@gnu.org; Sun, 03 Apr 2016 17:56:07 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <alezost@gmail.com>) id 1amq03-0001nV-E6
	for guix-devel@gnu.org; Sun, 03 Apr 2016 17:56:06 -0400
In-Reply-To: <87d1q61uhq.fsf@gnu.org> (Mathieu Lirzin's message of "Sun, 03
	Apr 2016 19:18:25 +0200")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: Mathieu Lirzin <mthl@gnu.org>
Cc: guix-devel@gnu.org

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

> Alex Kost <alezost@gmail.com> 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