Skip to content

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

Closed
karthago1 wants to merge 1 commit intoopenembedded:masterfrom
karthago1:bugfix-dev-package-not-installed
Closed

sdbus-c++-libsystemd: bugfix dev package is not installed#567
karthago1 wants to merge 1 commit intoopenembedded:masterfrom
karthago1:bugfix-dev-package-not-installed

Conversation

@karthago1
Copy link
Copy Markdown

@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
Copy Markdown
Contributor

kraj commented Apr 21, 2022

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

@karthago1
Copy link
Copy Markdown
Author

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

@shr-project
Copy link
Copy Markdown
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
Copy Markdown
Author

thank you for the explanation.

@kraj
Copy link
Copy Markdown
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