Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build Failed #21255

Closed
yinhejianke opened this issue Jul 27, 2022 · 9 comments · Fixed by #22461
Closed

build Failed #21255

yinhejianke opened this issue Jul 27, 2022 · 9 comments · Fixed by #22461
Assignees
Labels
stale Stale issue or PR

Comments

@yinhejianke
Copy link

Problem

<what's wrong or missing, please include any applicable:

  • expected behavior
  • actual behavior
  • steps to reproduce
  • system configuration
  • screenshots, images, logs, etc.

ninja: Entering directory `./out/debug'
[1733/3907] c++ linux_x64_gcc/obj/src/lib/support/tests/libSupportTests.TestStateMachine.cpp.o
FAILED: linux_x64_gcc/obj/src/lib/support/tests/libSupportTests.TestStateMachine.cpp.o
g++ -MMD -MF linux_x64_gcc/obj/src/lib/support/tests/libSupportTests.TestStateMachine.cpp.o.d -Wconversion -O0 -g2 -fno-common -ffunction-sections -fdata-sections -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables -fPIC -Wall -Werror -Wextra -Wshadow -Wunreachable-code -Wvla -Wformat -Wformat-nonliteral -Wformat-security -Wno-deprecated-declarations -Wno-unknown-warning-option -Wno-maybe-uninitialized -Wno-missing-field-initializers -Wno-unused-parameter -Wno-psabi -Wno-cast-function-type -fdiagnostics-color -fno-strict-aliasing -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/sysprof-4 -I/usr/include/gio-unix-2.0 -I/usr/include/libmount -I/usr/include/blkid -Wno-implicit-fallthrough -Wno-implicit-fallthrough -std=gnu++14 -fno-rtti -Wnon-virtual-dtor -DCHIP_HAVE_CONFIG_H=1 -I../../src/include -I../../src -Ilinux_x64_gcc/gen/include -I../../zzz_generated/app-common -I../../config/standalone -I../../third_party/nlassert/repo/include -I../../third_party/nlio/repo/include -I../../third_party/nlfaultinjection/repo/include -I../../third_party/inipp/repo/inipp -I../../third_party/nlunit-test/repo/src -I../../third_party/jsoncpp/repo/include -c ../../src/lib/support/tests/TestStateMachine.cpp -o linux_x64_gcc/obj/src/lib/support/tests/libSupportTests.TestStateMachine.cpp.o
../../src/lib/support/tests/TestStateMachine.cpp: In constructor ‘{anonymous}::SimpleStateMachine::SimpleStateMachine()’:
../../src/lib/support/tests/TestStateMachine.cpp:158:41: error: member ‘{anonymous}::SimpleStateMachine::mStateMachine’ is used uninitialized [-Werror=uninitialized]
158 | SimpleStateMachine() : mTransitions(mStateMachine), mStateMachine(mTransitions) {}
| ^~~~~~~~~~~~~
At global scope:
cc1plus: note: unrecognized command-line option ‘-Wno-unknown-warning-option’ may have been intended to silence earlier diagnostics
cc1plus: all warnings being treated as errors
[1746/3907] c++ linux_x64_gcc/obj/examples/chip-tool/chip-tool.main.cpp.o
ninja: build stopped: subcommand failed.

Proposed Solution

<suggested fix, suggested enhancement>

@yinhejianke
Copy link
Author

Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-linux-gnu/12.1.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: /build/gcc/src/gcc/configure --enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++,d --enable-bootstrap --prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/ --with-build-config=bootstrap-lto --with-linker-hash-style=gnu --with-system-zlib --enable-__cxa_atexit --enable-cet=auto --enable-checking=release --enable-clocale=gnu --enable-default-pie --enable-default-ssp --enable-gnu-indirect-function --enable-gnu-unique-object --enable-libstdcxx-backtrace --enable-link-serialization=1 --enable-linker-build-id --enable-lto --enable-multilib --enable-plugin --enable-shared --enable-threads=posix --disable-libssp --disable-libstdcxx-pch --disable-werror
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 12.1.0 (GCC)

@yinhejianke
Copy link
Author

DISTRIB_ID="Arch"
DISTRIB_RELEASE="rolling"
DISTRIB_DESCRIPTION="Arch Linux"
NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://bugs.archlinux.org/"
LOGO=archlinux-logo

@yinhejianke
Copy link
Author

image

@bzbarsky-apple
Copy link
Contributor

@msandstedt

@turon
Copy link
Contributor

turon commented Sep 7, 2022

I'm seeing this after git reset --hard origin/master this morning. I was up-to-date with master as recently as yesterday.

The only additional data I can think of is that I just installed Android Studio for Linux.

turon added a commit to turon/connectedhomeip that referenced this issue Sep 7, 2022
turon added a commit to turon/connectedhomeip that referenced this issue Sep 7, 2022
woody-apple pushed a commit to turon/connectedhomeip that referenced this issue Sep 8, 2022
@andy31415
Copy link
Contributor

When fixing this, please undo the change in #22461 (the uninitialized should remain an error)

@andy31415 andy31415 reopened this Sep 8, 2022
@andy31415
Copy link
Contributor

Re-opening to fix the circular dependency rather than disabling the warning in the compiler.

isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this issue Sep 16, 2022
…pleStateMachine test. (project-chip#22461)

* [build] Fix project-chip#21255 - allow circular initialization of SimpleStateMachine test.

* [build] Add comment per review feedback.
emargolis pushed a commit to emargolis/connectedhomeip-1 that referenced this issue Oct 17, 2022
…pleStateMachine test. (project-chip#22461)

* [build] Fix project-chip#21255 - allow circular initialization of SimpleStateMachine test.

* [build] Add comment per review feedback.
andy31415 added a commit that referenced this issue Oct 20, 2022
…ntation (#23239)

* Add missing pthread header (#22833)

* [build] Fix #21255 - allow circular initialization of SimpleStateMachine test. (#22461)

* [build] Fix #21255 - allow circular initialization of SimpleStateMachine test.

* [build] Add comment per review feedback.

* [Darwin][AttestationVerifier] Expose a mechanism to customise cd signing keys and use it in darwin (#22338)

* Add AttestationTrustStore::GetCertificationDeclarationCert virtual method to allow controllers passing in some CD certs

* Add cdCerts member to MTRControllerFactoryparams and override AttestationTrustStore::GetCertificationDeclarationCert

* Implement ArrayTrustStore::GetCertificationDeclarationSigningKey and initialize the test ArrayTrustStore store with the test CD cert

* Update the FileAttestationTrustStore to read a directory with der certs for certification declaration verification

* Add credentials/development/cd-certs/ and update chip-tool to use it if desired

* Update API to match conversation
- Remove CD stuff from FileAttestationTrustStore
- Refactor FileAttestationTrustStore to allow loading
  of any X.509 cert directory
- Add a command line to chip-tool to disallow test keys
  (`only-allow-trusted-cd-keys`)
- Add plumbing to enable CD keys lookup properly without mixing-up
  with PAA semantics
- Add official CD verifying key and official SDK CD test key
  in the default CD trust store as-is

* Update src/darwin to take into account the proposed changes

* Add unit test for `CsaCdKeysTrustStore`

Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com>

* [Attestation] Updated to Use CD Signed by a Valid CSA Cert (#22685)

* Updated CSA Official CD Signing Certificates (#23027)

* restyled.

* Remove fixed versioning for git in cirque (#23257)

Co-authored-by: Gene Harvey <gene.harvey@smartthings.com>
Co-authored-by: Martin Turon <mturon@google.com>
Co-authored-by: Vivien Nicolas <vnicolas@apple.com>
Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com>
Co-authored-by: Andrei Litvin <andy314@gmail.com>
mspang added a commit to mspang/connectedhomeip that referenced this issue Dec 6, 2022
This warning has not been fixed. Squelch it to remove noisy output in
the build log.

The underlying issue is tracked at project-chip#21255
mspang added a commit that referenced this issue Dec 8, 2022
This warning has not been fixed. Squelch it to remove noisy output in
the build log.

The underlying issue is tracked at #21255
@stale
Copy link

stale bot commented Mar 11, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale Stale issue or PR label Mar 11, 2023
@stale
Copy link

stale bot commented Mar 23, 2023

This stale issue has been automatically closed. Thank you for your contributions.

@stale stale bot closed this as completed Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Stale issue or PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants