-
-
Notifications
You must be signed in to change notification settings - Fork 114
1.6.7 hourly build directories issue #65
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
From @per1234 on November 9, 2015 2:36 @pierrotm777 are you referring to issue arduino/Arduino#3909? |
From @pierrotm777 on November 10, 2015 9:13 Yes , this is the same issue ! |
|
In fact , this is the case but before that wasn't a problem ! |
Yes, @pierrotm777. What I meant to say is that (maybe) arduino-builder is collecting even .ino file in subfolders, while it really shouldn't. Still to confirm though |
Yes , this exact ! |
See #65 Signed-off-by: Federico Fissore <[email protected]>
@pierrotm777 I've verified that arduino-builder does NOT load .ino files stored in subfolders of the main .ino file. This is the code that makes it happen https://github.com/arduino/arduino-builder/blob/master/src/arduino.cc/builder/sketch_loader.go#L117-L119 |
SynchTwinRcEngine_WithMacros_V3.zip.txt
|
Ahh so it's not about .ino files in subfolders, it's about .c/.cpp files... Ok, then I'm sorry but this was a feature added some months ago as a subfeature of arduino/Arduino#3080 |
Ok, but the 1.6.6 hasn't this issue ! |
i'm finding the same problem with 1.6.7 but not 1.6.6 - if you output to build-path, then re-run arduino-builder, it seems to create a subdirectory of build-path with the previous build in it or something odd... |
@sej7278, again, I think your issue is different from this issue to, so it's best reported separately. As for the original issue here, it seems that now any source files (.c / .cpp) in subdirectories of the sketch folder are compiled. @pierrotm777 has some documentation or example files, which include a .c file, which he does not intend to compile, but they are compiled (which didn't happen before). I'm not sure if this is an intentional change, though. To fix #3080, all files in the sketch directory were copied in to the build directory, so the preprocessed .ino files could refer to these files (e.g. #include a file in a subdirectory). However, it seems that in addition to copying the files, any source files in subdirectories are now also compiled, which could be viewed as a new feature, but it also potentially breaks sketches (and no longer provides any place to "hide" source files in a sketch). I haven't looked at the source code in detail yet, but perhaps this should be fixed for the next release, before people start relying on this behaviour? |
moved my comments to #89 |
This was fixed by #148, which should be in the upcoming 1.6.10 released. |
From @pierrotm777 on November 8, 2015 14:18
If a directory exist into the same folder as the main .ino file, all files .ino found into the directory are read by the compiler and return an error .
This issue existed into the 1.6.6 hourly build and solved with the 1.6.6 final version .
Copied from original issue: arduino/Arduino#4111
The text was updated successfully, but these errors were encountered: