Skip to content

mcumgr: fix build with clang and bump#914

Merged
ricardosalveti merged 3 commits intofoundriesio:mainfrom
quaresmajose:mcumgr
Nov 14, 2022
Merged

mcumgr: fix build with clang and bump#914
ricardosalveti merged 3 commits intofoundriesio:mainfrom
quaresmajose:mcumgr

Conversation

@quaresmajose
Copy link
Copy Markdown
Member

@quaresmajose quaresmajose commented Nov 14, 2022

  • base: mcumgr: bump 5c56bd
Changelog:
- chore: upgrade deps
- Merge pull request #26 from AudioStreamingPlatform/master
- Added LICENSE file
  • base: mcumgr: relay only on go that uses goarch underline
grep -rn goarch openembedded-core/meta/classes/go.bbclass
openembedded-core/meta/classes/go.bbclass:1:inherit goarch
  • base: non-clangable: add mcumgr
llvm-strip fails with:

ERROR: mcumgr-v0.0.1+git-r0 do_package: Fatal errors occurred in subprocesses:
Command '['x86_64-lmp-linux-llvm-strip', '--remove-section=.comment', '--remove-section=.note', '/srv/oe/build/tmp-lmp/work/corei7-64-lmp-linux/mcumgr/v0.0.1+git-r0/package/usr/bin/mcumgr']' returned non-zero exit status 1.
Subprocess output:x86_64-lmp-linux-llvm-strip: error: SHT_STRTAB string table section [index 9] is non-null terminated
ERROR: Task (/srv/oe/build/conf/../../layers/meta-lmp/meta-lmp-base/recipes-devtools/mcumgr/mcumgr_git.bb:do_package) failed with exit code '1'

Signed-off-by: Jose Quaresma [email protected]

Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

llvm-strip fails with:

| ERROR: mcumgr-v0.0.1+git-r0 do_package: Fatal errors occurred in subprocesses:
| Command '['x86_64-lmp-linux-llvm-strip', '--remove-section=.comment', '--remove-section=.note', '/srv/oe/build/tmp-lmp/work/corei7-64-lmp-linux/mcumgr/v0.0.1+git-r0/package/usr/bin/mcumgr']' returned non-zero exit status 1.
| Subprocess output:x86_64-lmp-linux-llvm-strip: error: SHT_STRTAB string table section [index 9] is non-null terminated
| ERROR: Task (/srv/oe/build/conf/../../layers/meta-lmp/meta-lmp-base/recipes-devtools/mcumgr/mcumgr_git.bb:do_package) failed with exit code '1'

Signed-off-by: Jose Quaresma <[email protected]>
| grep -rn goarch openembedded-core/meta/classes/go.bbclass
| openembedded-core/meta/classes/go.bbclass:1:inherit goarch

Signed-off-by: Jose Quaresma <[email protected]>
Changelog:
- chore: upgrade deps
- Merge pull request foundriesio#26 from AudioStreamingPlatform/master
- Added LICENSE file

Signed-off-by: Jose Quaresma <[email protected]>
@quaresmajose quaresmajose changed the title base: non-clangable: add mcumgr mcumgr: fix build with clang and bump Nov 14, 2022
Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti ricardosalveti merged commit b9dd994 into foundriesio:main Nov 14, 2022
@quaresmajose quaresmajose deleted the mcumgr branch November 14, 2022 19:15
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.

4 participants