From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Enge Subject: Re: patches question Date: Thu, 23 Jun 2016 22:27:53 +0200 Message-ID: <20160623202753.GB2529@solar> References: <20160623104107.GA2505@shadowwalker> <20160623132347.GA9193@shadowwalker> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55156) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGBEH-0001t7-R6 for guix-devel@gnu.org; Thu, 23 Jun 2016 16:28:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bGBEE-0006A8-LY for guix-devel@gnu.org; Thu, 23 Jun 2016 16:28:01 -0400 Received: from mailrelay1.public.one.com ([91.198.169.124]:11601) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGBEE-00069X-89 for guix-devel@gnu.org; Thu, 23 Jun 2016 16:27:58 -0400 Content-Disposition: inline In-Reply-To: <20160623132347.GA9193@shadowwalker> 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: guix-devel@gnu.org On Thu, Jun 23, 2016 at 01:23:47PM +0000, ng0 wrote: > Firefox wip, untested, not even run, just filling in fixes > which very likely have broken syntax looks like the later > inserted file. In this file there is a 140+ lines phase > which aims at doing what the following inserted patch > does. It is still growing (2 files left to patch). > As 'maybe' icecat can make use of the harfbuzz graphite > phase in this case it makes more sense to drop it > into gnu/packages/patches/ .. right? I must admit I have difficulties understanding what you are trying to do (after I finally managed to parse the first sentence above, in which it is rather unclear who or what is the subject of the different verbs). It is clear that the complicated scheme "substitute*" in your package definition are inferior to a real patch; if all they do is just reimplement finely chiselled patches, they do not make sense. The example I had in mind for uses of "substitute*" was when a string could be replaced by another one everywhere in files, for instance "/bin/sh" by "sh" or these kinds of things; then "substitute*" will still work, even if a few lines are swapped in the file or the string occurs once more or less in a later version. A patch would fail in that situation. It looks like you are trying to use the system harfbuzz instead of a bundled copy? Or the system graphite? If that is the case, maybe it would be optimal to propose a patch upstream? The icecat package definition contains a few comments pointing to bug reports at bugzilla.mozilla.org. Andreas