Skip to content
This repository was archived by the owner on Dec 8, 2023. It is now read-only.

Fixing bridge module build warnings on ARM linux #43

Merged
merged 3 commits into from
Nov 20, 2020

Conversation

dipannita08
Copy link
Contributor

@dipannita08 dipannita08 commented Nov 18, 2020

Problem

  • Bridge module build warnings: unsafe header/lib path in cross compilation:
[ 99%] Building C object src/pnpbridge/module/CMakeFiles/pnpbridge_module.dir/__/common/pnp_device_client.c.o
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot'
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot/inc'
[ 99%] Building C object src/pnpbridge/module/CMakeFiles/pnpbridge_module.dir/__/common/pnp_dps.c.o
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot'
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot/inc'
[ 99%] Building C object src/pnpbridge/module/CMakeFiles/pnpbridge_module.dir/__/common/pnp_protocol.c.o
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot'
arm-none-linux-gnueabi-gcc: WARNING: unsafe header/library path used in cross-compilation: '-I/usr/include/azureiot/inc'

Fix

  • Remove redundant dependency of azureiot include paths from module cmake
  • Remove module dependency on uuid library

Additionally

  • Clean up adapter cmakes
  • Log only errors on telemetry callback in the BLE adapter

Validation

  • Build bridge as module targeting linux amd
  • Validate edge module support on linux amd
  • Build bridge as module targeting linux arm

@dipannita08 dipannita08 changed the title Fixing Bridge Module Cmake Warning Fixing bridge module build warnings on ARM linux Nov 20, 2020
@dipannita08 dipannita08 merged commit 9964f7f into master Nov 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants