From owner-freebsd-ports-bugs@freebsd.org Sun Nov 22 08:27:36 2015 Return-Path: Delivered-To: freebsd-ports-bugs@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 84A5FA35DF7 for ; Sun, 22 Nov 2015 08:27:36 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4E3C91F07 for ; Sun, 22 Nov 2015 08:27:36 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id tAM8Ra2l058418 for ; Sun, 22 Nov 2015 08:27:36 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 204725] Mk/bsd.port.mk: makepatch inserts context junk after r401709 Date: Sun, 22 Nov 2015 08:27:36 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Ports Framework X-Bugzilla-Version: Latest X-Bugzilla-Keywords: needs-qa, patch, regression X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: marino@FreeBSD.org X-Bugzilla-Status: New X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: marino@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: maintainer-feedback? X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 22 Nov 2015 08:27:36 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=204725 --- Comment #7 from John Marino --- (In reply to Jan Beich from comment #6) I don't understand most of your response. The "local: 2: bad variable name" was the primary problem. It basically aborted the operation. However, I'm still on FreeBSD 9.2 on this test machine. "If makepatch cannot avoid screwing up multi-file patches ..." It clearly states both in the header and the commit message that if file X is modified by patch file A, B, and C, that it will combine all the changes and which patch it is appled to will vary. This is not "a screwup". If anything, the port maintainer could be blamed for allowing the situation, but sometimes it can't be helped if applying external patches. In such a case, a single patch may be temporarily moved before running "make makepatch" so the multipatch is generated the same way, and the single patch generated by hand. The assumption is the patches being regenerated do not apply without fuzz. ".. and any hunks applied before are *lost* because .orig file(s) now points to a patched version(s)" I'm not sure what you are talking about here. There are two ways to use makepatch. 1) regenerate (clean/extract/do-patch/makepatch) 2) update patches (clean/extract/do-patch/ /makepatch There are no lost hunks. Unless you're still talking about a file that is patched multiple times (which again, should be avoided) "One way to fix this issue is to employ a VCS to record changes then export them back." I'm not interested in introducing a VCS requirement, nor do I see how that would even help. The "it can't restore multiple partial changes" issue is a rare use case that has a limited workaround and it's a publicized limitation. To tell somebody not to use a tool for a limitation that occurs around %0.1 of the ports ... how does that help anybody? However, I can't tell from your response. Does the attached patch cause "makepatch" to work as designed? -- You are receiving this mail because: You are on the CC list for the bug.