From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mathieu Lirzin Subject: Re: [PATCH 2/2] build: Move environment scripts to 'build-aux' directory. Date: Sun, 03 Apr 2016 19:18:25 +0200 Message-ID: <87d1q61uhq.fsf@gnu.org> References: <1459700326-30016-1-git-send-email-mthl@gnu.org> <1459700326-30016-3-git-send-email-mthl@gnu.org> <871t6m39o2.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:60366) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1amlfW-0003P1-83 for guix-devel@gnu.org; Sun, 03 Apr 2016 13:18:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1amlfR-00056F-EV for guix-devel@gnu.org; Sun, 03 Apr 2016 13:18:34 -0400 In-Reply-To: <871t6m39o2.fsf@gmail.com> (Alex Kost's message of "Sun, 03 Apr 2016 20:05:17 +0300") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-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: Alex Kost Cc: guix-devel@gnu.org Alex Kost 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". ;) -- Mathieu Lirzin