From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roel Janssen Subject: Re: [PATCH] Fix compiling on CentOS 7. Date: Sun, 28 Aug 2016 16:13:57 +0200 Message-ID: <87r399dlqy.fsf@gnu.org> References: <87d1kvpazv.fsf@gnu.org> <20160827065756.kblwxq3gmg63eug2@venom> <87wpj2a9m9.fsf@gmail.com> <87lgziy4qn.fsf@gnu.org> <87bn0diy89.fsf@gnu.org> <20160828081102.grkh52hivbhcf5be@venom> <20160828093444.GH26988@macbook42.flashner.co.il> <20160828095336.uh4mjcdxskkqzjdp@venom> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:40733) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1be0pq-00051r-7C for guix-devel@gnu.org; Sun, 28 Aug 2016 10:13:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1be0pl-00037t-7L for guix-devel@gnu.org; Sun, 28 Aug 2016 10:13:18 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:50536) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1be0pl-00037Y-3Y for guix-devel@gnu.org; Sun, 28 Aug 2016 10:13:13 -0400 In-reply-to: <20160828095336.uh4mjcdxskkqzjdp@venom> 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" To: =?utf-8?B?VG9tw6HFoSDEjGVjaA==?= Cc: guix-devel@gnu.org Tomáš Čech writes: > On Sun, Aug 28, 2016 at 12:34:44PM +0300, Efraim Flashner wrote: >>On Sun, Aug 28, 2016 at 10:11:02AM +0200, Tomáš Čech wrote: >>> On Sun, Aug 28, 2016 at 01:33:10AM +0200, Roel Janssen wrote: >>> > >>> > Florian Paul Schmidt writes: >>> > >>> > > On 08/27/2016 10:51 AM, Roel Janssen wrote: >>> > > >>> > > > I don't see a way to apply this patch "on the CentOS side" except for >>> > > > waiting (another two or three years) until they upgrade Automake. >>> > > >>> > > When packaging guix for CentOS, just add a patch to the source package >>> > > definition.. >>> > >>> > I do not intend to create an RPM package for CentOS, I am merely trying >>> > to do the following: >>> > git clone git://git.sv.gnu.org/guix.git >>> > cd guix >>> > ./bootstrap >>> > ./configure >>> > make >>> >>> I see, I haven't thought about that... >>> >>> But, in that case, wouldn't be creating guix RPM package for CentOS even >>> more beneficial alternative (making it easy for all CentOS7 users...)? >>> >>> Best regards, >>> >>> S_W >> >>That would require keeping it up-to-date. > > Actually I think that this particular patch could be replaced by > script snippet (untested) in spec: > > for dir in doc emacs gnu nix; do > sed "s@%D%@${dir}@g" "${dir}/local.mk" > done > > Furthermore, it requires usually fix once per release - when user run > `guix pull', it becomes irrelevant what is user running (but daemon > changes). But what is the point of scripting the patch procedure if we can just prevent the need for it? Typos that weren't there anyway? Moving directories around that we're not planning, and we could adjust for with the above three lines _when the change is relevant_? We can lower the version requirement for Automake by this simple patch! And doing so is relevant to getting things to work as expected on CentOS 7. > I must say I'm impressed by the effort to fully maintain CentOS 7 to > 2020. If you don't want to go package way (which I think is > prefferable) I'm not against the reverting patch anymore. Thanks. Mathieu, could you revert your patch, or do you want me to look at it? Kind regards, Roel Janssen