-
Notifications
You must be signed in to change notification settings - Fork 229
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
feat: add W600 auto build #281
Conversation
Thanks, did you manage to make it work without any changes to parent SDK? |
🎉 This PR is included in version 1.13.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Just using the SDK submodule already added to the main repo (the direct link to https://github.com/iprak/OpenW600) - no other changes were needed and seems to compile successfully. Did you want this changed to the @openshwprojects fork instead if you're using that instead now? EDIT: PR for this change here - take or leave: #282 |
Thanks @talltechdude , @btsimonh has recently added CPP support and IR library, but despite the local compilation going great, it seems that online Github compile fails. It looks like it's trying to feed a CPP file like a .o file. Do you know where to look for an error? |
most likely it's checking out a specific commit of the SDK repo? |
yes - there is some procedure to update the submodules in the APP repo, and then commit the new repository point which it is linked to. I don't remember the details, but basically it's a git thing you need to do.... currently it's referencing a specific commit of the SDK in the app repo? |
try latest update on my branch? sourcetree method here: |
Also fixed a case sensitive header file reference :(. |
I assumed it's done automatically now? I am trying "submodule update --remote" but it seems to be downloading the other repos on my disk to the App dir, which not what I wanted |
just merge new PR :). |
Ah, that makes sense that its not auto-updating the new repos, they were missing from the Makefile. PR here #319 to hopefully resolve that moving forward and keep them auto-updated to |
Adding W600 build to automated build and releases