search

OpenDDS / meta-opendds

Open Embedded Layer for the OpenDDS package
Other
8 stars 4 forks source link

update to latest yocto release + version bump of opendds #34

Closed vermaete closed 10 months ago

vermaete commented 11 months ago

The description how to contribute in the readme.md file could be updated?

Patches
=======

Please submit any patches against the meta-opendds layer via the pull-request
mechanism on github.

Maintainer: Keith Derrick <keith.derrick@lge.com>
vermaete commented 11 months ago

@mitza-oci Did another small commit to set the version in the readme correct

vermaete commented 10 months ago

Hi,

It should be the bug tracker of the recipe. So OpenDDS, as it is. https://docs.yoctoproject.org/singleindex.html#term-BUGTRACKER

I assume the info the user would like to find about the meta layer itself is in the readme.md file. Br and my best wishes for 2024

On Tue, Jan 2, 2024 at 3:47 PM Adam Mitz @.***> wrote:

@.**** commented on this pull request.

In recipes-connectivity/opendds/opendds.inc https://github.com/OpenDDS/meta-opendds/pull/34#discussion_r1439516061:

@@ -1,5 +1,6 @@ SUMMARY = "OpenDDS is an open source C++ implementation of the Object Management Group (OMG) Data Distribution Service (DDS)" HOMEPAGE = "https://opendds.org" +BUGTRACKER = "https://github.com/OpenDDS/OpenDDS/issues"

Should this be OpenDDS/meta-opendds? -- if users have problems they should probably start here

— Reply to this email directly, view it on GitHub https://github.com/OpenDDS/meta-opendds/pull/34#pullrequestreview-1800536242, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA5IFKXAIXKAWJ2DCUQXM3LYMQMYJAVCNFSM6AAAAABBFK3QF6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMBQGUZTMMRUGI . You are receiving this because you authored the thread.Message ID: @.***>

-- Jan Vermaete “Success is a self-correcting phenomenom.” -- Gary Hamel

vermaete commented 10 months ago

hi @mitza-oci

With my last commit the master, latest release and LTS release of Yocto should work. For me, this concludes the the PR. Could you merge it?

Next on my list is:

mitza-oci commented 10 months ago

hi @mitza-oci

With my last commit the master, latest release and LTS release of Yocto should work. For me, this concludes the the PR. Could you merge it?

Looks good to me. @plabanca would you please review? If you don't have time we can probably merge anyway.