From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: [PATCH] gnu: Add p7zip. Date: Sun, 04 Sep 2016 16:31:48 +0200 Message-ID: <87zinn91nv.fsf@elephly.net> References: <87a8gwxa9c.fsf@openmailbox.org> <87y44f4plm.fsf@mdc-berlin.de> <20ae08af155acdc7edd02b68afb9e66d@openmailbox.org> <20160808195835.GF18650@jasmine> <871t1y74fe.fsf@mdc-berlin.de> <87d1lheore.fsf@mdc-berlin.de> <87oa51puzr.fsf@openmailbox.org> <878tw5el71.fsf@elephly.net> <87k2floj6v.fsf@openmailbox.org> <87r39hysdu.fsf@elephly.net> <87r39ge4pe.fsf@openmailbox.org> <87inumijn2.fsf@openmailbox.org> <87wpiwacrm.fsf@elephly.net> <874m5zipdq.fsf@openmailbox.org> <87mvjra2yq.fsf@elephly.net> <87zinrh24h.fsf@openmailbox.org> 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]:41913) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bgYSn-00047P-JA for guix-devel@gnu.org; Sun, 04 Sep 2016 10:32:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bgYSj-00070k-Ct for guix-devel@gnu.org; Sun, 04 Sep 2016 10:32:00 -0400 Received: from sender163-mail.zoho.com ([74.201.84.163]:24313) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bgYSj-00070d-4V for guix-devel@gnu.org; Sun, 04 Sep 2016 10:31:57 -0400 In-reply-to: <87zinrh24h.fsf@openmailbox.org> 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: Kei Kebreau Cc: guix-devel@gnu.org Hi Kei, > Ricardo Wurmus writes: > >> Kei Kebreau writes: >> >>> Ricardo Wurmus writes: >>> >>>> Kei Kebreau writes: >>>> >>>>> Whoops, last patch was a bit messy and stacked on the previous one. This >>>>> patch should be better! >>>> >>>> I just wanted to push a slightly modified version of this (attached) but >>>> I cannot actually build the package. The patch to remove unused code >>>> does not apply due to different line endings. >>>> >>>> Could you please take a look at this again and make sure that the patch >>>> to the sources applies? >>>> >>>> ~~ Ricardo >>>> >>> I just tried it on my machine, and everything applied and built >>> correctly from a clean Guix tree. I don't exactly know how to proceed >>> From here. Perhaps a third party can try to build from the patch? >> >> The problem might be with the inline patch in the email. Don’t know. >> >> ~~ Ricardo > > I'll try sending it again, for lack of a better plan. If all else fails > I can post it on paste.lisp.org. I fetched the unpatched sources, initialised a fresh git repo, then partially applied the patch, then fixed the errors, then regenerated the patch and compared it to your patch. Turns out that all of the lines in the two failed hunks end on 0x0d0a, but in your patch this doesn’t happen, hence the error messages. I pushed the amended commit to master as adbd7faf412297e48931ba4f7ad9a062d852ab8a. ~~ Ricardo