Skip to content

serial: Fix empty package and use shared lib instead of static lib.#809

Closed
wesleyel wants to merge 3 commits intoopenembedded:masterfrom
wesleyel:patch-serial
Closed

serial: Fix empty package and use shared lib instead of static lib.#809
wesleyel wants to merge 3 commits intoopenembedded:masterfrom
wesleyel:patch-serial

Conversation

@wesleyel
Copy link
Contributor

The older recipe of serial built a static lib libserial.a.

Which was contained in FILES_${PN}-dev and FILES_${PN} is empty.

This will make bitbake unhapply and raise Nothing provides serial needed by other-recipe problem.

To fix it we need add CMake set_target_properties function.

see also:

  1. https://docs.yoctoproject.org/pipermail/yocto/2015-January/022921.html
  2. https://stackoverflow.com/questions/59091938/yocto-oe-recipe-with-cmake-install-a-shared-library-so

Copy link
Contributor

@kraj kraj left a comment

Choose a reason for hiding this comment

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

Can you submit this patch upstream ? and also set Upstream-Status in the patch accordingly.

@wesleyel
Copy link
Contributor Author

@kraj Thanks for your reminder. I have appended Upstream-Status label.

I noticed that the maintainer of the serial project has not had any activity in the past 10 months. Maybe in this case we can merge it directly into meta-oe.

@kraj
Copy link
Contributor

kraj commented Apr 15, 2024

@wesleyel
Copy link
Contributor Author

https://errors.yoctoproject.org/Errors/Details/763022/

Sorry. I have not test ptest feature on my project.

Can you give me some hint for fix unimplemented-ptest Error?

I noticed that we have CATKIN_ENABLE_TESTING macro to enable test binary building. But I don't know how to make them integrating with ptest.

https://github.com/wjwwood/serial/blob/main/tests/CMakeLists.txt

@kraj
Copy link
Contributor

kraj commented Apr 30, 2024

https://errors.yoctoproject.org/Errors/Details/763022/

Sorry. I have not test ptest feature on my project.

Can you give me some hint for fix unimplemented-ptest Error?

I noticed that we have CATKIN_ENABLE_TESTING macro to enable test binary building. But I don't know how to make them integrating with ptest.

https://github.com/wjwwood/serial/blob/main/tests/CMakeLists.txt

Enable ptest via adding it to local.conf

DISTRO_FEATURES:append = " ptest"

then try building it again.

@kraj
Copy link
Contributor

kraj commented May 4, 2024

fixed with 224eb26

@kraj kraj closed this May 4, 2024
@wesleyel
Copy link
Contributor Author

wesleyel commented May 7, 2024

Thanks!

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.

2 participants