From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mail.openembedded.org (Postfix) with ESMTP id 8096071A4B for ; Thu, 3 Nov 2016 19:29:28 +0000 (UTC) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP; 03 Nov 2016 12:29:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,440,1473145200"; d="scan'208";a="1063576256" Received: from bkarakos-mobl.ger.corp.intel.com (HELO peggleto-mobl.ger.corp.intel.com) ([10.255.187.65]) by fmsmga001.fm.intel.com with ESMTP; 03 Nov 2016 12:29:27 -0700 From: Paul Eggleton To: Alexander Kanavin Date: Fri, 04 Nov 2016 08:29:26 +1300 Message-ID: <1480827.LzmdYstA2d@peggleto-mobl.ger.corp.intel.com> Organization: Intel Corporation User-Agent: KMail/4.14.10 (Linux/4.7.10-100.fc23.x86_64; KDE/4.14.20; x86_64; ; ) In-Reply-To: <6511599a-ddb1-77b0-f42e-69afeaa60cae@linux.intel.com> References: <6511599a-ddb1-77b0-f42e-69afeaa60cae@linux.intel.com> MIME-Version: 1.0 Cc: Patches and discussions about the oe-core layer Subject: Re: using devtool for rebasing patches X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Nov 2016 19:29:28 -0000 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Hi Alex, On Thu, 03 Nov 2016 18:01:27 Alexander Kanavin wrote: > it would be very useful if devtool offered support for rebasing source > patches when they fail to apply, but currently there does not seem to be > any such feature: > > ak@kanavin-desktop:~/development/poky/build$ devtool modify libksba > Parsing recipes..done. > NOTE: Fetching libksba... > NOTE: Unpacking... > NOTE: Patching... > ERROR: Applying 'ksba-add-pkgconfig-support.patch' failed: > checking file Makefile.am > Hunk #1 FAILED at 21. > 1 out of 1 hunk FAILED > checking file configure.ac > Hunk #1 succeeded at 414 (offset 14 lines). > checking file ksba.pc.in > checking file src/ksba.m4 > ERROR: Function failed: patch_do_patch > ak@kanavin-desktop:~/development/poky/build$ > > What devtool could do here is apply the patches that can be applied and > then partially apply the patches that fail. Then give the developer some > kind of UI for dealing with the rejected hunks (which typically means > finding the place in the target file where the changes should go and > then manually editing them in, while looking at the original patch). > > Without this feature devtool is half as useful as it could be > particularly for recipe version updates. "devtool upgrade" is designed specifically to handle the upgrade case and tries to rebase the old patches onto the new version, putting you into resolution mode with "git am" if it fails to apply a patch. I appreciate that you may now be working through sorting out all of the patches to reduce the fuzziness when applying, and that won't help in that scenario, but it would definitely be worth considering for upgrades. I'm open to implementing better support for handling patch application failure for devtool modify - in fact there is a bug open for just that: https://bugzilla.yoctoproject.org/show_bug.cgi?id=8325 Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre