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

zpc: Issue with ninja while build #49

Open
vietnq-silabs opened this issue Mar 3, 2025 · 10 comments
Open

zpc: Issue with ninja while build #49

vietnq-silabs opened this issue Mar 3, 2025 · 10 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@vietnq-silabs
Copy link

I have this issue with ninja when I run ./helper.mk
I have attached the log here

build.zip

@rzr
Copy link
Collaborator

rzr commented Mar 3, 2025

Please comment the line in helper.mk

  #       || cat build/CMakeFiles/CMakeOutput.log

Maybe it is hidding an other issue, you are running in WSL2 arent you ?

can show results of

./helper.mk setup

I know some observed issues with audio dependencies (inherited from zap package we dont need)

@vietnq-silabs
Copy link
Author

This is the log from ./helper.mk setup
log_helper_setup.txt

@vietnq-silabs
Copy link
Author

vietnq-silabs commented Mar 3, 2025

This is the log when I comment # || cat build/CMakeFiles/CMakeOutput.log

Image

@rzr
Copy link
Collaborator

rzr commented Mar 4, 2025

Your setup step looks ok to me

https://github.com/user-attachments/files/19053665/log_helper_setup.txt

please remove build folder before running:

script z-wave-protocol-controller-49.log.txt

Then type

./helper.mk VERBOSE=1

And then exit

and attach z-wave-protocol-controller-49.log.txt to this ticket

Meanwhile let me try to rebuild on wsl2 ...

@vietnq-silabs
Copy link
Author

This is the z-wave-protocol-controller-49.log.txt log file, I think the build process is successfully. (I comment this line # || cat build/CMakeFiles/CMakeOutput.log in helper.mk)

z-wave-protocol-controller-49.log.txt

@rzr
Copy link
Collaborator

rzr commented Mar 4, 2025

So where was the failure. maybe I will remove this extra cat line it is misleading

@rzr rzr changed the title Issue with ninja while build zpc: Issue with ninja while build Mar 6, 2025
@rzr rzr added the documentation Improvements or additions to documentation label Mar 6, 2025
@vietnq-silabs
Copy link
Author

This is my script log after clone and rebuild

z-wave-protocol-controller-49.log.txt

@rzr
Copy link
Collaborator

rzr commented Mar 7, 2025

This is my script log after clone and rebuild

https://github.com/user-attachments/files/19124006/z-wave-protocol-controller-49.log.txt

 tail  ~/Downloads/z-wave-protocol-controller-49.log.txt 
CPack: -   Install component: uic-zpc
CPack: Create package
CPack: - package: /root/z-wave-protocol-controller/build/libunify_1.7.0-33-gc2a04c8a_x86_64.deb generated.
CPack: - package: /root/z-wave-protocol-controller/build/libunify-dev_1.7.0-33-gc2a04c8a_x86_64.deb generated.
CPack: - package: /root/z-wave-protocol-controller/build/uic-zpc_1.7.0_x86_64.deb generated.
[9/11] cd /root/z-wave-protocol-controller/build/_deps/unifysdk-build/applications/image_provider && /usr/bin/cmake -E env VERSION_STR=1.7.0 UNIFY_BINARY_DIR=/root/z-wave-protocol-controller/build CARGO_PROFILE_RELWITHDEBINFO_INHERITS=release CARGO_PROFILE_RELWITHDEBINFO_DEBUG=1 CARGO_PROFILE_RELWITHDEBINFO_DEBUG_ASSERTIONS=true CARGO_TARGET_DIR=/root/z-wave-protocol-controller/build/cargo/uic_image_provider_build /root/.cargo/bin/cargo deb -p uic-image-provider --no-strip --no-build --deb-version 1.7.0 --output /root/z-wave-protocol-controller/build/uic-image-provider_1.7.0-33-gc2a04c8a_x86_64.deb --target x86_64-unknown-linux-gnu --manifest-path /root/z-wave-protocol-controller/build/_deps/unifysdk-src/applications/image_provider/Cargo.toml --profile dev
warning: dev profile is not supported and will be a hard error in the future. cargo-deb is for making releases, and it doesn't make sense to use it with dev profiles.
warning: To enable debug symbols set `[profile.release] debug = true` instead.
/root/z-wave-protocol-controller/build/uic-image-provider_1.7.0-33-gc2a04c8a_x86_64.deb
[11/11] cd /root/z-wave-protocol-controller/build && mkdir -p /root/z-wave-protocol-controller/build/z-wave-protocol-controller_1.7.0_x86_64 && mv /root/z-

log looks good to me (packages are built). but i miss the last part ?

@rzr
Copy link
Collaborator

rzr commented Mar 18, 2025

Is this issue still present ?

Since i have slightly improved the quickstart guide, please review:

#41

@rzr rzr self-assigned this Mar 19, 2025
@rzr
Copy link
Collaborator

rzr commented Mar 19, 2025

I have merged the quickstart readme

Please close this ticket if no more support it needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants