-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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 folder only based on sketch name + path #5107
Comments
Hi @tbowmo, Adding an arch:board field to the md5ified string would be a good solution, while timestamping would make the folder non deterministic. |
Sounds reasonable.. was just easier for me to get timestamp, when doing initial experimenting. submitted PR #5130 |
IIRC I have some pending patches that remove the determinism again, since it's not strictly needed. I'll see if I can find it again and what the status is. |
As long as we can have multiple builds on the same sketch running in parallel for multiple architectures, then it would be great.. |
Ah, found it. It's the "Randomize the the build path name" commit in #4263. That PR is scheduled to be merged soon after 1.6.10 is released, last I heard. |
Verified to be ok.. |
the build path is getting created in
https://github.com/arduino/Arduino/blob/master/arduino-core/src/processing/app/BaseNoGui.java#L132
And is based solely on the path and name of the sketch to compile. If we use arduino in a CI system, and want to do things in parallel for multiple archs, then they will collide with each other.
Can we have a bit more unique build path created? By adding timestamp for example in the string that get's md5ifized?
The text was updated successfully, but these errors were encountered: