Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork35
Upcoming rewrite of git history#103
-
Hi everyone
So we overhauled the CI and use it as an artifact provider! When#102 will be merged, every time we push to This will primarily benefit users who don't want to install the whole zephyr toolchain to just modify the core/library files, while the "hardcore" users will not see any change in their workflow (except a much cleaner git history 😉 ) The only drawback (@KurtE@mjs513) is that |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 7 comments 7 replies
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
Maybe something like git stash will still allow you to get around this. I usually have run into this with the current stuff anyway as I But I can also imagine we might need to do a fresh reload after this PR is done! |
BetaWas this translation helpful?Give feedback.
All reactions
-
Looks like: maybe yesterday you updated the main project here with this change? Any more updates to our work flow. That is if I sync up to this, and then do: Can we then still simply choose which board we want in the IDE and do the flash? Or do we need to run some other script to then I did see in the readme that you can do the ./extra/build.sh on a specific board Looks like most of the other PRs have been pulled in and closed. Note On portenta H7, IO pins. Currently the only ones that |
BetaWas this translation helpful?Give feedback.
All reactions
-
@KurtE Also noticed looks like there is going to be 2 build options for GIga - one with and one without the display - or am I reading that wrong? As for the Portenta H7 - kind of limited with out the high density pins defined - or are you plaining on separate builds for the different carrier boards? Inquiring minds want to know - otherwise will probably add them locally. Also any idea when the next official release will be? |
BetaWas this translation helpful?Give feedback.
All reactions
-
Yes, this is now merged. The outputs from the build procedure are nownot committed andignored, so a Thepros of not having binary / build output files in the repository:
Thecons:
How toupdate the artifacts after a git command changes the source files:
We also still need to clear the repository of the old files - that will be a "ground zero" day where all commits are redone; we may even move to a different repository. I think the next release will have this and the tool properly packaged for everyone to use! Hope this clears a few questions 🙂 |
BetaWas this translation helpful?Give feedback.
All reactions
-
Thanks@pillo79, Yes it helps! It is great to not have all of those conflicts show up, let along the time it takes to upload/download all of these files! But I am guessing that this tool will only help sync up, if/when I sync up to thehttps://www.github.com/arduino/ArduinoCore-zephyr Likewise, I assume, it will not work, if I sync up to the USB fixes PR, it won't bring down the artifacts for this? Although, maybe it does? if the github checks builds the variants and keeps the artifacts. Side note: I appreciate how the build_all.sh now automatically builds all of the current variant!
Although it is probably just me 😆 |
BetaWas this translation helpful?Give feedback.
All reactions
-
You are correct, this will no longer work for the two cases you mentioned:
However,
Re/ the side note, I agree! Just pushed an update allowing |
BetaWas this translation helpful?Give feedback.
All reactions
-
Thanks@pillo79 - Yep, the side note makes it a lot easier to remember... Will try out the stuff that you mentioned that@mjs513 Currently I am doing it with a set of rsync commands between the ubuntu FS and windows FS. As a different side note, trying to figure out why some of our test sketches where we are trying out things on the display, work on the zephyr_GIGA_shield_touchpaint-250429a.zip I added simple fill the screen with 4 colors directly into our SDRAM buffer and then tell it display. This was just a check to make Debugged it into stm32_ltdc_write call: where I added prints for passed in stuff: And the prints show: So looks like the memory we allocated for our frame buffer in SDRAM Overlaps the memory for the pending buffer; 0xc0000080 So something not in place for the shared memory pool or ??? ... Investigating |
BetaWas this translation helpful?Give feedback.
All reactions
-
quick update: if I hack it up to use the shared memory allocator, it appears to fix it: Not sure why the difference between builds... but.. |
BetaWas this translation helpful?Give feedback.
All reactions
-
@KurtE as you said not sure why specifing using sdram for full screen does not work. |
BetaWas this translation helpful?Give feedback.
All reactions
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
Makes life a lot simpler when we are only working with one board Have to get use to the changes you are making. Argh EDIT: Hope you update the readme with the these changes so they are all in one place |
BetaWas this translation helpful?Give feedback.
All reactions
👍 1
-
tried running synch artifacts and got ??? what am I missing :) |
BetaWas this translation helpful?Give feedback.
All reactions
-
Error handling can definitely be improved 🤣 the current syntax is: so the simplest call is |
BetaWas this translation helpful?Give feedback.
All reactions
-
Just tried it with a fresh copy of the core: learning process for me -@KurtE is more familar with this. |
BetaWas this translation helpful?Give feedback.
All reactions
-
@facchinm@pillo79 Wondering if this one should be closed and/or also updated the more recent issue |
BetaWas this translation helpful?Give feedback.
