Skip to content

Conversation

@kapi-no
Copy link
Contributor

@kapi-no kapi-no commented Oct 24, 2025

Ref: NCSDK-17015

Ref: NCSDK-17015

Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
@kapi-no kapi-no requested a review from a team as a code owner October 24, 2025 08:44
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 24, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 24, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: 37138b2599653857d03262d048b6eda36c45bf30

more details

sdk-nrf:

PR head: 37138b2599653857d03262d048b6eda36c45bf30
merge base: b119e5e0b13cefc81a4c7b5b9c820bb29530a2d8
target head (main): 7264b114c02ebab736a93d7545647f9369d73942
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1)
scripts
│  ├── hid_configurator
│  │  │ requirements.txt

Outputs:

Toolchain

Version: cfa6b06338
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:cfa6b06338_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 49
  • ✅ Integration tests
    • ✅ desktop52_verification
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@kapi-no kapi-no added the DNM label Oct 24, 2025
@@ -1,3 +1,3 @@
# pyhidapi
hid>=1.0.3, <1.0.5
hid==1.0.8
Copy link
Contributor

Choose a reason for hiding this comment

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

I would desist from specifying the library version (remove the upper version limit). It seems that the issue was caused by incompatibilities between Python hid package version and version of the underlying libraries. This eventually triggered: undefined symbol: hid_get_input_report.

Requiring newer version here might cause problems for users that need to use older version of underlying libraries (e.g. older OS release). Requiring older version might lead to missing some updates (theoretically including security fixes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants