Skip to content

base: bluez5: enable using libedit#1044

Merged
MrCry0 merged 2 commits intofoundriesio:mainfrom
MrCry0:main-bluez-libedit
Feb 16, 2023
Merged

base: bluez5: enable using libedit#1044
MrCry0 merged 2 commits intofoundriesio:mainfrom
MrCry0:main-bluez-libedit

Conversation

@MrCry0
Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 commented Feb 11, 2023

Support using libedit for client package feature of bluez5 if GPLv3 packages are disabled.

Signed-off-by: Oleksandr Suvorov [email protected]

@MrCry0
Copy link
Copy Markdown
Contributor Author

MrCry0 commented Feb 11, 2023

@vanmaegima Honister-based backport of this solution is coming.

@MrCry0 MrCry0 force-pushed the main-bluez-libedit branch from 23a1afe to 01583d2 Compare February 13, 2023 09:14
Comment thread meta-lmp-base/recipes-connectivity/bluez5/bluez5_5.65.bbappend Outdated
Comment thread meta-lmp-base/recipes-connectivity/bluez5/bluez5_5.65.bbappend Outdated
Comment thread meta-lmp-base/recipes-connectivity/bluez5/bluez5_5.65.bbappend Outdated
@ricardosalveti
Copy link
Copy Markdown
Member

Nice work, patch small enough that can be maintained easily here.

Would be nice to confirm with @vanmaegima that the non-interactive usage is indeed ok for the customer.

@MrCry0 MrCry0 force-pushed the main-bluez-libedit branch 2 times, most recently from 8174bb8 to 961dccd Compare February 15, 2023 20:44
Copy link
Copy Markdown
Contributor

@igoropaniuk igoropaniuk left a comment

Choose a reason for hiding this comment

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

LGTM

@MrCry0 MrCry0 marked this pull request as draft February 16, 2023 08:50
@MrCry0 MrCry0 marked this pull request as ready for review February 16, 2023 12:46
Copy link
Copy Markdown
Member

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

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

LGTM with little suggestion that can be implemented after

Comment thread meta-lmp-base/recipes-connectivity/bluez5/bluez5_5.65.bbappend Outdated
Support automatic choose between readline/libedit for client and mesh
package features of bluez5.

Upstream-Status: Pending
Signed-off-by: Oleksandr Suvorov <[email protected]>
Add libedit config to bluez5 to be able to build a client feature
of bluez5 (bluetoothctl tool).

Signed-off-by: Oleksandr Suvorov <[email protected]>
@MrCry0 MrCry0 force-pushed the main-bluez-libedit branch from 961dccd to dfe0cc9 Compare February 16, 2023 15:56
@MrCry0 MrCry0 merged commit fc3a4bb into foundriesio:main Feb 16, 2023
@MrCry0 MrCry0 deleted the main-bluez-libedit branch March 27, 2023 13:54
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.

5 participants