Skip to content

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

Closed
ffissore opened this issue Nov 16, 2015 · 14 comments
Closed

1.6.7 hourly build directories issue #65

ffissore opened this issue Nov 16, 2015 · 14 comments
Labels
type: imperfection Perceived defect in any part of project

Comments

@ffissore
Copy link
Contributor

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

@ffissore ffissore self-assigned this Nov 16, 2015
@ffissore
Copy link
Contributor Author

From @per1234 on November 9, 2015 2:36

@pierrotm777 are you referring to issue arduino/Arduino#3909?

@ffissore
Copy link
Contributor Author

From @pierrotm777 on November 10, 2015 9:13

Yes , this is the same issue !

@ffissore
Copy link
Contributor Author

.ino files should belong to main folder only, not subfolders

@pierrotm777
Copy link

In fact , this is the case but before that wasn't a problem !
I like to keep some older file into subfolders .

@ffissore
Copy link
Contributor Author

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

@pierrotm777
Copy link

Yes , this exact !

ffissore pushed a commit that referenced this issue Nov 17, 2015
@ffissore
Copy link
Contributor Author

@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
Can you share the sketch you are using?

@pierrotm777
Copy link

SynchTwinRcEngine_WithMacros_V3.zip.txt
I send you my sources .
The error is:
C:\Users\pierre\AppData\Local\Temp\build11326b2d920aae776cf88a2171973031.tmp\sketch\Telemetrie en cours\M-Link\main.c:50:17: fatal error: htc.h: No such file or directory

 #include <htc.h>

                 ^

compilation terminated.

@ffissore
Copy link
Contributor Author

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

@pierrotm777
Copy link

Ok, but the 1.6.6 hasn't this issue !
Do i delete this subfolder or you think to do something ? :-)

@sej7278
Copy link

sej7278 commented Jan 3, 2016

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...

@matthijskooijman
Copy link
Collaborator

@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?

@sej7278
Copy link

sej7278 commented Jan 4, 2016

moved my comments to #89

@mastrolinux mastrolinux modified the milestone: 1.3.9 Jan 8, 2016
@cmaglie cmaglie modified the milestone: 1.3.9 Feb 4, 2016
@matthijskooijman
Copy link
Collaborator

This was fixed by #148, which should be in the upcoming 1.6.10 released.

@rsora rsora added the type: imperfection Perceived defect in any part of project label Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: imperfection Perceived defect in any part of project
Projects
None yet
Development

No branches or pull requests

7 participants