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

meta-oe: gpsd depends on meta-python unconditionally #871

Closed
lumag opened this issue Sep 12, 2024 · 12 comments
Closed

meta-oe: gpsd depends on meta-python unconditionally #871

lumag opened this issue Sep 12, 2024 · 12 comments

Comments

@lumag
Copy link
Contributor

lumag commented Sep 12, 2024

Adding just meta-oe results in the following error from bitbake:

NOTE: Resolving any missing task queue dependencies
ERROR: Nothing RPROVIDES 'python3-pyserial' (but ../meta-openembedded/meta-oe/recipes-navigation/gpsd/gpsd_3.25.bb RDEPENDS on or otherwise requires it)
NOTE: Runtime target 'python3-pyserial' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['python3-pyserial']
NOTE: Runtime target 'gpsd' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['gpsd', 'python3-pyserial']
...

Please make this part conditionally depend on the meta-python being enabled.

@quaresmajose
Copy link
Contributor

quaresmajose commented Sep 12, 2024

maybe we can make it conditional with a PACKAGECONFIG, disabled by default

PACKAGECONFIG[ubxtool] = ",,python3-pyserial"

introduced in 1266c91

@lumag
Copy link
Contributor Author

lumag commented Sep 13, 2024

Well, there is also a second issue: if there is a CI for meta-oe, it needs to be updated to catch such issues. I unfortunately don't know enough details of the CI setup to approach this problem.

@quaresmajose
Copy link
Contributor

@quaresmajose
Copy link
Contributor

Well, there is also a second issue: if there is a CI for meta-oe, it needs to be updated to catch such issues. I unfortunately don't know enough details of the CI setup to approach this problem.

I believe the yocto-check-layer can catch this type of things.

@quaresmajose
Copy link
Contributor

fixed eb731aa

@lumag
Copy link
Contributor Author

lumag commented Sep 27, 2024

It is not. With just meta-oe added and with the default configuration I'm still getting:

ERROR: Nothing RPROVIDES 'python3-pyserial' (but /home/lumag/Projects/RPB/build-rpb/conf/../../layers/meta-openembedded/meta-oe/recipes-navigation/gpsd/gpsd_3.25.bb RDEPENDS on or otherwise requires it)
NOTE: Runtime target 'python3-pyserial' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['python3-pyserial']
ERROR: Required build target 'gpsd' has no buildable providers.
Missing or unbuildable dependency chain was: ['gpsd', 'python3-pyserial']

@quaresmajose
Copy link
Contributor

well! in reality I didn't remove the problem.

 RDEPENDS:python3-pygps = " \
   python3-core \
   python3-io \
-  python3-pyserial \
   python3-threading \
   python3-terminal \
   gpsd \
   python3-json"

Thanks for notifying me
I will have to come back to this again

@lumag
Copy link
Contributor Author

lumag commented Sep 27, 2024

Please also check that RDEPENDS get addded back if pyserial is enabled

quaresmajose referenced this issue Sep 27, 2024
The python3-pyserial dependency was introduced in [1].
It is provided by the meta-python layer and so make it conditionally.

Fixes:

| NOTE: Resolving any missing task queue dependencies
| ERROR: Nothing RPROVIDES 'python3-pyserial' (but ../meta-openembedded/meta-oe/recipes-navigation/gpsd/gpsd_3.25.bb RDEPENDS on or otherwise requires it)
| NOTE: Runtime target 'python3-pyserial' is unbuildable, removing...
| Missing or unbuildable dependency chain was: ['python3-pyserial']
| NOTE: Runtime target 'gpsd' is unbuildable, removing...
| Missing or unbuildable dependency chain was: ['gpsd', 'python3-pyserial']

[1] https://git.openembedded.org/meta-openembedded/commit/?id=1266c912afa0abf118eaa5d152a0641c87665fbd

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
@quaresmajose
Copy link
Contributor

v2 merged 09563d1

@lumag
Copy link
Contributor Author

lumag commented Oct 1, 2024

I am sorry for the delay, I will be able to run tests next week.

@lumag
Copy link
Contributor Author

lumag commented Oct 6, 2024

Seems to be working now, thank you!

@lumag lumag closed this as completed Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants