Skip to content

Conversation

@ppryga-nordic
Copy link
Contributor

Configuration of the sample included CONFIG_BT_ISO=y.
The CONFIG_BT_ISO is defined as hidden option in
subsys/bluetooth/Kconfig. That makes build of the sample
to fail. It blocks PRs in the CI if sample is build.

The commit fixes the issue by change of CONFIG_BT_ISO
to CONFIG_BT_ISO_UNICAST.

Fixes build issue in PR: #31875

Signed-off-by: Piotr Pryga [email protected]

Configuration of the sample included CONFIG_BT_ISO=y.
The CONFIG_BT_ISO is defined as hidden option in
subsys/bluetooth/Kconfig. That makes build of the sample
to fail. It blocks PRs in the CI if sample is build.

The commit fixes the issue by change of CONFIG_BT_ISO
to CONFIG_BT_ISO_UNICAST.

Signed-off-by: Piotr Pryga <[email protected]>
@github-actions github-actions bot added the area: Samples Samples label Jul 2, 2021
@ppryga-nordic ppryga-nordic requested review from Thalley and cvinayak July 2, 2021 21:24
ppryga-nordic referenced this pull request Jul 3, 2021
Pass enh ack data injection arguments to the nRF5 radio driver.

Signed-off-by: Lukasz Maciejonczyk <[email protected]>
Thalley
Thalley previously approved these changes Jul 5, 2021
@Thalley
Copy link
Contributor

Thalley commented Jul 6, 2021

The PR that this fixes has been reverted and a new one has been created including this fix: #36752 so this can be closed.

@Thalley Thalley dismissed their stale review July 6, 2021 06:52

PR no longer needed

@ppryga-nordic
Copy link
Contributor Author

Closing due to revert of PR #31875.

@MaureenHelm MaureenHelm deleted the github-samples-fix-kconfig-iso-connected-benchmark branch July 6, 2021 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants