Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade 3.9.6 -> 3.9.7 #205

Closed
wants to merge 0 commits into from
Closed

Upgrade 3.9.6 -> 3.9.7 #205

wants to merge 0 commits into from

Conversation

iipeace
Copy link
Contributor

@iipeace iipeace commented Mar 8, 2020

Signed-off-by: Peace Lee iipeace5@gmail.com

@iipeace
Copy link
Contributor Author

iipeace commented Mar 8, 2020

@kraj
please review again :)

meta-oe/recipes-devtools/guider/guider_3.9.7.bb Outdated Show resolved Hide resolved
meta-oe/recipes-devtools/guider/guider_3.9.7.bb Outdated Show resolved Hide resolved
@kraj
Copy link
Contributor

kraj commented Mar 9, 2020

also commit msg should be

recipename: Short summary

Long summary
Sign-off-by

@iipeace
Copy link
Contributor Author

iipeace commented Mar 9, 2020

@kraj
please review again :)

@kraj
Copy link
Contributor

kraj commented Mar 9, 2020

thanks in master-next now

@kraj
Copy link
Contributor

kraj commented Mar 9, 2020

Fails to build without meta-python2 see [1]

@kraj kraj added the feedback waiting response from submitter label Mar 9, 2020
@iipeace
Copy link
Contributor Author

iipeace commented Mar 9, 2020

then should I add below code?

inherit ${@bb.utils.contains("BBFILE_COLLECTIONS", "meta-python2", "distutils", "", d)}inherit distutils

python() { if 'meta-python2' not in d.getVar('BBFILE_COLLECTIONS').split(): raise bb.parse.SkipRecipe('Requires meta-python2 to be present.')}

@kraj
Copy link
Contributor

kraj commented Mar 10, 2020

perhaps in the code replace python with python3 explicitly

@iipeace
Copy link
Contributor Author

iipeace commented Mar 24, 2020

@kraj,
how can I access to https://jenkins.ccp.xcal.tv/jenkins/view/YP/job/Yocto-world-arm/448/console ??
I can't see any register button in the page.

@kraj
Copy link
Contributor

kraj commented Mar 24, 2020

Thats not public repo and the logs for that build are gone. I should have pointed at errors.yp.org location. Let me see if I can cherry-pick it again and reporduce the error.

@kraj
Copy link
Contributor

kraj commented Mar 24, 2020

This patch is still wrong, it should still check for meta-python2 presence as such it will fail meta-oe build when meta-py2 is not present, we do not have dependency on meta-py2

@iipeace
Copy link
Contributor Author

iipeace commented Mar 24, 2020

okay, then could you share error logs?

@iipeace
Copy link
Contributor Author

iipeace commented Jun 8, 2020

@kraj
could you review again?

@iipeace
Copy link
Contributor Author

iipeace commented Jun 8, 2020

I changed my PR.

LIC_FILES_CHKSUM = "file://LICENSE;md5=2c1c00f9d3ed9e24fa69b932b7e7aff2"

PV = "3.9.7+git${SRCPV}"
PR = "r0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop PR = r0 is default.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

may I leave this line?
because I will use this variable later.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For what? Nobody is doing manual PR bumps anymore, there is PRservice for that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't know about it.
Thanks!

meta-oe/recipes-devtools/guider/guider_3.9.7.bb Outdated Show resolved Hide resolved
meta-oe/recipes-devtools/guider/guider_3.9.7.bb Outdated Show resolved Hide resolved
@iipeace iipeace force-pushed the master branch 2 times, most recently from 48e18b1 to 8cde7e0 Compare June 10, 2020 22:05
@kraj
Copy link
Contributor

kraj commented Jun 11, 2020

I have merged it now thanks for persisting :)

@kraj kraj closed this Jun 11, 2020
@shr-project
Copy link
Contributor

@iipeace This new version looks much better without the dependency on meta-python2, but rewritting the git history of master branch broke guider recipe e.g. in zeus branch of meta-openembedded where it now fails to fetch:

ERROR: guider-3.9.4+gitAUTOINC+b433f38056-r0 do_fetch: Fetcher failure: Unable to find revision b433f3805674ef4588c1a161986c74eeac6a48e7 in branch master even from upstream

before re-write

guider$ git branch -a --contains b433f3805674ef4588c1a161986c74eeac6a48e7
* master
  redundant

after:

guider$ git remote update
Fetching origin
remote: Enumerating objects: 423, done.
remote: Counting objects: 100% (423/423), done.
remote: Compressing objects: 100% (31/31), done.
remote: Total 5064 (delta 405), reused 409 (delta 391), pack-reused 4641
Receiving objects: 100% (5064/5064), 4.14 MiB | 5.98 MiB/s, done.
Resolving deltas: 100% (2500/2500), completed with 23 local objects.
From git://github.com/iipeace/guider
 + dc4f3ae...ab803ac master             -> master  (forced update)
 + bca0ee3...197d588 refs/pull/207/head -> refs/pull/207/head  (forced update)
 * [new ref]         refs/pull/209/head -> refs/pull/209/head
 * [new ref]         refs/pull/210/head -> refs/pull/210/head
 * [new ref]         refs/pull/214/head -> refs/pull/214/head
 * [new ref]         refs/pull/215/head -> refs/pull/215/head
 * [new ref]         refs/pull/217/head -> refs/pull/217/head
 * [new ref]         refs/pull/218/head -> refs/pull/218/head
 * [new ref]         refs/pull/220/head -> refs/pull/220/head
 * [new ref]         refs/pull/222/head -> refs/pull/222/head
 * [new ref]         refs/pull/223/head -> refs/pull/223/head
 * [new ref]         refs/pull/224/head -> refs/pull/224/head
 t [tag update]      versions/3.8.0     -> versions/3.8.0
 t [tag update]      versions/3.9.6     -> versions/3.9.6
 t [tag update]      versions/3.9.7     -> versions/3.9.7
guider$ git branch -a --contains b433f3805674ef4588c1a161986c74eeac6a48e7
  redundant

Please restore master branch as it was to fix released versions of meta-oe and implement new changes on top of that - and you really shouldn't be moving the tags..

@shr-project shr-project reopened this Jun 14, 2020
@iipeace
Copy link
Contributor Author

iipeace commented Jun 14, 2020

@shr-project,
It was my mistake to remove all useless files from git history by filter-branch command...;(
how can I recover tag commits?

@shr-project
Copy link
Contributor

@iipeace

I've created fork of your repo at:
https://github.com/shr-project/guider
and pushed the older version of master and tags I had in my local downloads directory from OE

To github.com:shr-project/guider.git
 + d3ad332...dc4f3ae master -> master (forced update)
 + f505d4c...a4220da versions/3.8.0 -> versions/3.8.0 (forced update)
 + 006aa86...29eb4c2 versions/3.9.6 -> versions/3.9.6 (forced update)
 + 993014a...0786c98 versions/3.9.7 -> versions/3.9.7 (forced update)

It doesn't have to be the latest version of master you had before re-writting the history - so just rebase whatever you have on top of this older revision - you should be also able to find your own old history in git reflog (which should show where you did the filter-branch), that would be more accurate and really the latest before re-write. It should be there, but depends on how long expire interval you're using or if you called git expire since the re-write.

For tags you can use something like:

git remote add shr git@github.com:shr-project/guider.git
git fetch --tags shr
# this should show that these 3 tags were updated
git push -f origin versions/3.8.0   versions/3.9.6 versions/3.9.7 
# assuming your origin is your github repo

@iipeace
Copy link
Contributor Author

iipeace commented Jun 15, 2020

@shr-project,
I rebased older branch you shared and "show-ref --tags" command shows as a below.
'''
694c2bf59d1ab199dccd5673c2c4c5777b73cdc5 refs/tags/versions/2.0.0
fb224a932cc3ae0fe734189ba8f3e5953fcaad66 refs/tags/versions/3.0.0
eb0388935ee262b4408f623ab73907a4cc827b06 refs/tags/versions/3.7.0
e38a93ef34e3faf9ca27c05132c80b85861ce06f refs/tags/versions/3.7.1
f505d4cee736afdadd9c0f53fc3f0d464ae2a915 refs/tags/versions/3.8.0
e76d3b2424294892815031bb46237d3aa43c8ef7 refs/tags/versions/3.8.7
fb0eb4a379936a70d46e38450e114d64309e8c57 refs/tags/versions/3.8.9
73ac11ba939a45056664ed9e1468cd312131df03 refs/tags/versions/3.9.0
2c1035fea192de2d609e91bb34bc0eb7f22a4ede refs/tags/versions/3.9.1
3c9755dc4b6f295552b1569b72cc2dae5ca4d5d5 refs/tags/versions/3.9.3
bd2997de13685c6fff2b6852c317a94bc9b7d5b7 refs/tags/versions/3.9.4
787dc60443f5f44085c09a1a9200d663ffe11ea8 refs/tags/versions/3.9.5
006aa86cba4b0ec427702845d1cf1976a2c30498 refs/tags/versions/3.9.6
993014a044f64d54bd0d181ee6ac04d689352e36 refs/tags/versions/3.9.7
'''

is this right?
then I am going to push them to upstream.

@shr-project
Copy link
Contributor

These 3.8.0, 3.9.6, 3.9.7 tags look like what is currently on your github, not the old versions matching with commits in old master branch. Did you forget to run "git fetch --tags shr"?

You can easily check with
git branch -a --contains refs/tags/versions/3.9.7
should show the restored master branch - and similarly for 3.8.0, 3.9.6 tags and SRCREVs used in builds.

From meta-oe perspective it these 3 SRCREVs b433f3805674ef4588c1a161986c74eeac6a48e7 (zeus and warrior) fef25c41efb9bde0614ea477d0b90bd9565ae0b4 (dunfell) and c87269d6d4669d12c99a2a7b9b07c39a5dd24ac6 (master). The first 2 exist only in the "original" master. The c87269d6d4669d12c99a2a7b9b07c39a5dd24ac6 currently in master only in the re-written master, so the recipe in meta-oe will need and SRCREV update again once you push the rebased branch.

@iipeace
Copy link
Contributor Author

iipeace commented Jun 15, 2020

@shr-project,
I forgot to git fetch --tags :)
this is about that.

'''
From https://github.com/shr-project/guider
t [tag update] versions/3.8.0 -> versions/3.8.0
t [tag update] versions/3.9.6 -> versions/3.9.6
t [tag update] versions/3.9.7 -> versions/3.9.7
peacelee@osdc:~/work/guider$ git show-ref --tags
694c2bf59d1ab199dccd5673c2c4c5777b73cdc5 refs/tags/versions/2.0.0
fb224a932cc3ae0fe734189ba8f3e5953fcaad66 refs/tags/versions/3.0.0
eb0388935ee262b4408f623ab73907a4cc827b06 refs/tags/versions/3.7.0
e38a93ef34e3faf9ca27c05132c80b85861ce06f refs/tags/versions/3.7.1
a4220da234c36506f7c5b69542b6a49b6d1e5796 refs/tags/versions/3.8.0
e76d3b2424294892815031bb46237d3aa43c8ef7 refs/tags/versions/3.8.7
fb0eb4a379936a70d46e38450e114d64309e8c57 refs/tags/versions/3.8.9
73ac11ba939a45056664ed9e1468cd312131df03 refs/tags/versions/3.9.0
2c1035fea192de2d609e91bb34bc0eb7f22a4ede refs/tags/versions/3.9.1
3c9755dc4b6f295552b1569b72cc2dae5ca4d5d5 refs/tags/versions/3.9.3
bd2997de13685c6fff2b6852c317a94bc9b7d5b7 refs/tags/versions/3.9.4
787dc60443f5f44085c09a1a9200d663ffe11ea8 refs/tags/versions/3.9.5
29eb4c26be50281b7d26de13dfa04d78a2651f4d refs/tags/versions/3.9.6
0786c98b08cda833ba9e388214ba9a04daf1c233 refs/tags/versions/3.9.7
'''

@shr-project
Copy link
Contributor

These tags match with what I wrote 3 comments above (#205 (comment)), so it's better because they match with the old ones. But it doesn't show anything about your new master branch.

@iipeace
Copy link
Contributor Author

iipeace commented Jun 15, 2020

@shr-project,
so should I push them to upstream master?
I will do that if it can solve build problems.

@shr-project
Copy link
Contributor

Yes, if you did it correctly (use "git branch --contains" to confirm as explained before) it will fix the build issues in warrior, zeus, dunfell branches. For master you will probably need to submit another SRCREV update to use corresponding SRCREV in the new branch.

@kraj
Copy link
Contributor

kraj commented Jun 15, 2020

Please push this as new pull request

@iipeace
Copy link
Contributor Author

iipeace commented Jun 15, 2020

@shr-project,
please apply for changes.

@kraj,
I will push this as new PR.

@shr-project
Copy link
Contributor

please apply for changes.

What?

kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Nov 1, 2022
License-Update: Copyright year updated to 2022.

Changelog:
==========
    Dropped support for legacy Pythons (2.7, 3.6 or older).
    Switched CI to GitHub Actions.
    Removed dependency on the py library (that was not properly specified as a dependency anyway).
    Fix skipping test in test_utils.py if appropriate VCS not available. Also fix typo. Contributed by Sam James in openembedded#211.
    Added support for pytest 7.2.0 by using pytest.hookimpl and pytest.hookspec to configure hooks. Contributed by Florian Bruhin in openembedded#224.
    Now no save is attempted if --benchmark-disable is used. Fixes openembedded#205. Contributed by Friedrich Delgado in openembedded#207.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Nov 2, 2022
License-Update: Copyright year updated to 2022.

Changelog:
==========
    Dropped support for legacy Pythons (2.7, 3.6 or older).
    Switched CI to GitHub Actions.
    Removed dependency on the py library (that was not properly specified as a dependency anyway).
    Fix skipping test in test_utils.py if appropriate VCS not available. Also fix typo. Contributed by Sam James in openembedded#211.
    Added support for pytest 7.2.0 by using pytest.hookimpl and pytest.hookspec to configure hooks. Contributed by Florian Bruhin in openembedded#224.
    Now no save is attempted if --benchmark-disable is used. Fixes openembedded#205. Contributed by Friedrich Delgado in openembedded#207.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Nov 7, 2022
License-Update: Copyright year updated to 2022.

Changelog:
==========
    Dropped support for legacy Pythons (2.7, 3.6 or older).
    Switched CI to GitHub Actions.
    Removed dependency on the py library (that was not properly specified as a dependency anyway).
    Fix skipping test in test_utils.py if appropriate VCS not available. Also fix typo. Contributed by Sam James in openembedded#211.
    Added support for pytest 7.2.0 by using pytest.hookimpl and pytest.hookspec to configure hooks. Contributed by Florian Bruhin in openembedded#224.
    Now no save is attempted if --benchmark-disable is used. Fixes openembedded#205. Contributed by Friedrich Delgado in openembedded#207.

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback waiting response from submitter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants