noble icon indicating copy to clipboard operation
noble copied to clipboard

Binding options

Open bitcloud opened this issue 4 years ago • 16 comments

transfer from https://github.com/noble/noble/pull/651

To use two different devices in one setup you need to configure the binding in your code and not with ENV-Vars.

I am using one device for connecting while the other keeps on collecting advertisements.

bitcloud avatar Sep 15 '20 12:09 bitcloud

please rebase

rzr avatar Sep 18 '20 08:09 rzr

this PR needs to be rebased, or you can submit smaller changes

rzr avatar Oct 05 '20 11:10 rzr

I startet it already, but it needs more effort then I originally thought and I currently have very little spare time at hand. So this is gonna take some time. If someone else wants to jump in, feel free.

On 5. Oct 2020, at 13:49, Philippe Coval [email protected] wrote:

this PR needs to be rebased, or you can submit smaller changes

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/abandonware/noble/pull/121#issuecomment-703580459, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACYW6OJ6WV6NXLTMU25TTDSJGXDBANCNFSM4RNCOJ3Q.

bitcloud avatar Oct 05 '20 15:10 bitcloud

Would it make sense that we close this PR, you can split this changeset into several smaller PR ?

rzr avatar Oct 15 '20 10:10 rzr

Hi @bitcloud , I took your PR on my own dev env, and I only 14 changes, not 119 as yours. I think you have a problem with CR/LF (or any other) on git.

Please see https://github.com/abandonware/noble/pull/155 I didn't test / try code, simply take it back (a fix a typo cancle to cancel).

atrovato avatar Dec 22 '20 07:12 atrovato

may you try to split your changes in smaller PRs

rzr avatar Dec 22 '20 13:12 rzr

can this change be part of next release ?

rzr avatar May 10 '21 12:05 rzr

@rzr this change (at least as it is described in the post) was merged in #155 and released in v1.9.2-12.

The diff for this PR is quite large, and contains other, unrelated changes. @atrovato has pulled out out other features (see #174) and the rest is left in #185.

I believe this PR can be rejected/closed. Disclaimer: I'm just a user of this library (thanks for keeping it maintained!)

airdrummingfool avatar May 10 '21 18:05 airdrummingfool

Ok please rebase to see if there are any leftover

rzr avatar Dec 16 '21 08:12 rzr

ping

rzr avatar May 18 '22 13:05 rzr

This branch has conflicts that must be resolved

rzr avatar Sep 06 '22 10:09 rzr

release is planned soon do you want this change to be part of it ?

rzr avatar Sep 24 '22 21:09 rzr

ping

rzr avatar Feb 05 '24 22:02 rzr

Is anyone interested to take over this change ?

rzr avatar Apr 11 '24 14:04 rzr

@bitcloud Ping!

itay-grudev avatar May 08 '24 14:05 itay-grudev

I assume there is no interest, feel free to cherrypick change an open a new PR

rzr avatar Aug 05 '24 14:08 rzr