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

sdbus-c++-libsystemd: bugfix dev package is not installed #567

Conversation

karthago1
Copy link

@karthago1 karthago1 commented Apr 21, 2022

since sdbus-c++-libsystemd outputs only static libraries, it will
not be installed inside the SDK.

add RDEPENDS:${PN}-dev = "" to remove the dependency to the empty package
when installing the sdbus-c++-libsystemd-dev into the SDK

without this change the SDK is missing sdbus-c++-dev and sdbus-c++-libsystemd-dev

Signed-off-by: Ben Fekih, Hichem hichem.benfekih@ifm.com

@kraj
Copy link
Contributor

kraj commented Apr 21, 2022

can you try using RDEPENDS:${PN}-dev = "" instead ?

@karthago1
Copy link
Author

@kraj yes it works also, is there any reason to use RDEPENDS:${PN}-dev = "" over ALLOW_EMPTY:${PN} = "1" ?

@shr-project
Copy link
Contributor

As explained many times in various places ALLOW_EMPTY just adds completely useless empty package into the image. And creates confusion when someone "installs" the package on target just to discover that the package manager downloaded and installed useless thing which was created just to keep default runtime dependencies in recipes happy.

since sdbus-c++-libsystemd outputs only static libraries, it will
not be installed inside the SDK.

add RDEPENDS:${PN}-dev = "" to remove the dependency to the empty package
when installing the sdbus-c++-libsystemd-dev into the SDK

without this change the SDK is missing sdbus-c++-dev and sdbus-c++-libsystemd-dev

Signed-off-by: Ben Fekih, Hichem <hichem.benfekih@ifm.com>
@karthago1 karthago1 force-pushed the bugfix-dev-package-not-installed branch from 70d4fdc to b44104c Compare April 22, 2022 06:16
@karthago1
Copy link
Author

thank you for the explanation.

@kraj
Copy link
Contributor

kraj commented Apr 25, 2022

applied with ec6d073

@kraj kraj closed this Apr 25, 2022
bluca pushed a commit to bluca/meta-openembedded that referenced this pull request May 23, 2022
Changelog:
url for these releases
<https://github.com/pyscaffold/pyscaffold/issues/####>
=========
Ensure build configuration produces sdist as it is needed by conda, openembedded#570
Documentation improvements in openembedded#567

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Trevor Gamblin <trevor.gamblin@windriver.com>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Dec 8, 2022
Changelog:
=========
  Update vendored double-conversion to 3.2.1 (openembedded#570)
  Fix len integer overflow issue (openembedded#567)

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 Dec 9, 2022
Changelog:
=========
  Update vendored double-conversion to 3.2.1 (openembedded#570)
  Fix len integer overflow issue (openembedded#567)

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 Dec 9, 2022
Changelog:
=========
  Update vendored double-conversion to 3.2.1 (openembedded#570)
  Fix len integer overflow issue (openembedded#567)

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants