Re: [OE-core] [PATCH 0/4] devtool fixes
On Tue, Jul 12, 2022 at 04:59:38PM +0100, Richard Purdie wrote: > On Mon, 2022-07-11 at 21:06 -0700, Paul Eggleton wrote: > > A few fixes for devtool, mostly relating to recipes that fetch from git > > where S points to a subdirectory of the repo. > > > > Note: I wasn't 100% sure if it was appropriate to be referencing WORKDIR > > in patch.py; let me know if I should try to rework that. > > It is probably ok. The code won't pick up on the variable usage from a > hash perspective there but it would be referenced elsewhere anyway. > > There was a failure which is probably from this series though: > > https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3817/steps/14/logs/stdio > > > (I've not confirmed it was the series, I'm just guessing) Hmm, yes, it probably is. I did run the tests but somehow I didn't see this failure. I'll investigate further. Paul -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167940): https://lists.openembedded.org/g/openembedded-core/message/167940 Mute This Topic: https://lists.openembedded.org/mt/92326549/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/4] devtool fixes
On Mon, 2022-07-11 at 21:06 -0700, Paul Eggleton wrote: > A few fixes for devtool, mostly relating to recipes that fetch from git > where S points to a subdirectory of the repo. > > Note: I wasn't 100% sure if it was appropriate to be referencing WORKDIR > in patch.py; let me know if I should try to rework that. It is probably ok. The code won't pick up on the variable usage from a hash perspective there but it would be referenced elsewhere anyway. There was a failure which is probably from this series though: https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3817/steps/14/logs/stdio (I've not confirmed it was the series, I'm just guessing) Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167938): https://lists.openembedded.org/g/openembedded-core/message/167938 Mute This Topic: https://lists.openembedded.org/mt/92326549/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/4] devtool fixes
Thanks Paul, this was one of the most annoying shortcomings of devtool. Alex On Tue, 12 Jul 2022 at 06:07, Paul Eggleton wrote: > > A few fixes for devtool, mostly relating to recipes that fetch from git > where S points to a subdirectory of the repo. > > Note: I wasn't 100% sure if it was appropriate to be referencing WORKDIR > in patch.py; let me know if I should try to rework that. > > > The following changes since commit db28cd0e1540e44db963108430205c8c0c817774: > > gperf: Switch to upstream patch (2022-07-09 20:58:31 +0100) > > are available in the git repository at: > > git://git.openembedded.org/openembedded-core-contrib paule/devtool37-oe > > http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/devtool37-oe > > Paul Eggleton (4): > patch: handle if S points to a subdirectory of a git repo > devtool: finish: handle patching when S points to subdir of a git repo > devtool: ignore pn- overrides when determining SRC_URI overrides > oe-selftest: devtool: test modify git recipe building from a subdir > > meta/lib/oe/patch.py| 8 +++--- > meta/lib/oe/recipeutils.py | 9 -- > meta/lib/oeqa/selftest/cases/devtool.py | 51 > + > scripts/lib/devtool/standard.py | 29 ++- > 4 files changed, 84 insertions(+), 13 deletions(-) > > -- > 1.8.3.1 > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167915): https://lists.openembedded.org/g/openembedded-core/message/167915 Mute This Topic: https://lists.openembedded.org/mt/92326549/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 0/4] devtool fixes
A few fixes for devtool, mostly relating to recipes that fetch from git where S points to a subdirectory of the repo. Note: I wasn't 100% sure if it was appropriate to be referencing WORKDIR in patch.py; let me know if I should try to rework that. The following changes since commit db28cd0e1540e44db963108430205c8c0c817774: gperf: Switch to upstream patch (2022-07-09 20:58:31 +0100) are available in the git repository at: git://git.openembedded.org/openembedded-core-contrib paule/devtool37-oe http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/devtool37-oe Paul Eggleton (4): patch: handle if S points to a subdirectory of a git repo devtool: finish: handle patching when S points to subdir of a git repo devtool: ignore pn- overrides when determining SRC_URI overrides oe-selftest: devtool: test modify git recipe building from a subdir meta/lib/oe/patch.py| 8 +++--- meta/lib/oe/recipeutils.py | 9 -- meta/lib/oeqa/selftest/cases/devtool.py | 51 + scripts/lib/devtool/standard.py | 29 ++- 4 files changed, 84 insertions(+), 13 deletions(-) -- 1.8.3.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#167889): https://lists.openembedded.org/g/openembedded-core/message/167889 Mute This Topic: https://lists.openembedded.org/mt/92326549/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH 0/4] devtool fixes
Three bugfixes for devtool plus one refactoring patch to improve a related function in the recipeutils module. The following changes since commit 059db140885bad379534e6ec713f3ceb4e18faea: adt-installer: use DEPLOY_DIR in ANT_DEPLOY expansion (2015-08-16 17:28:14 +0100) are available in the git repository at: git://git.openembedded.org/openembedded-core-contrib paule/devtool-fixes5 http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/devtool-fixes5 Paul Eggleton (4): devtool: extract: prevent externalsrc from interfering with extraction devtool: extract: remove patches subdirectory when S == WORKDIR devtool: fix handling of BBCLASSEXTENDed recipes lib/oe/recipeutils: avoid parsing in get_var_files() meta/lib/oe/recipeutils.py| 3 +-- meta/lib/oeqa/selftest/devtool.py | 30 ++ scripts/devtool | 12 ++-- scripts/lib/devtool/standard.py | 29 + 4 files changed, 54 insertions(+), 20 deletions(-) -- 2.1.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core