-
-
Notifications
You must be signed in to change notification settings - Fork 114
Compilation should leverage my quad core laptop power #17
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
Comments
-jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
-jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
A first implementation is available in parallel branch. It lacks proper output management. At the moment, everything is thrown at stdout, meaning the output is likely to get clutter and logger messages will eventually contain syntax errors |
-jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
-jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
…ed way. Thus we are sure that those outputs do not interfere with our own output, otherwise leading to wrongly formatted log messages. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
-jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
…ed way. Thus we are sure that those outputs do not interfere with our own output, otherwise leading to wrongly formatted log messages. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
Maybe you should come up to speed on Go language? That's probably a shorter & more successful path than trying to convince other people to spend their time implementing a feature they've said they do not intend to create because it doesn't make practical sense. |
@PaulStoffregen Meanwhile would you guys offer some internal documentations? |
I'm pretty sure the only available documentation, aside from the source code, is this wiki: (notice this is 1 of the 14 pages) |
@PaulStoffregen So I, being a newbie to the Go language, have to reverse engineer an existing codebase. Yeah like that would help a lot. |
That's a really creative usage of the term "reverse engineer", which usually means figuring out how something works without access to its source code. |
@PaulStoffregen Backtracking a poorly-documented code base is no less difficult than black box analysis. |
This arduino-builder repository contains only 7816 lines in all its source files, not including the automated tests, and many of those lines are redundant license headers, comments or white space. It's simply not a huge code base. Almost all files and functions have very descriptive names, like "generate_buildpath_if_missing.go". There is a 14 page wiki which does document the data formats and general ideas of the build process. Calling this project poorly-documented sounds to me like whining. |
@PaulStoffregen All while I personally have zero days of Go knowledge. |
I miss @ffissore :( |
@xcvista imho the discussion is a little inappropriate. As Paul has said: its open source, go fix it or leave it if the developers have other priorities. If thats beyond your skills too bad. You may also have chances offering a bounty to make your contribution to the project. Or: give platformio a try, it has parallel builds and loads of other things that Arduino doesn't offer. |
Coming to think of it, if moving to using Makefiles as an intermediary step, we need and only need the discovery step to find all the libraries. For my personal preference I would first assemble a pre-build source tree that have all core and library code files linked or copied over, throw a generic portable makefile in there and exec |
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 Based on Federico Fissore code arduino@0a3d888 Removed an if which suppress some gcc errors here arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 Remove and if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes arduino#17 - Rebased from Federico Fissore branch arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96 - Add HumanTagsLogger compatibility - Remove an if which suppress some GCC error messages here: arduino@0a3d888#diff-8f074e2b3fc911dbe8255891fc1a9b13R96
If -jobs X param is specified and X is greater than 0, arduino-builder won't spawn more than X processes. Fixes #17 Signed-off-by: Federico Fissore <[email protected]>
ATM every gcc call is done sequentially, but both library and core compilation can run in parallel
The text was updated successfully, but these errors were encountered: