Skip to content

Sketch .c, .cpp, and .h files not searched for library includes. [imported] #636

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.

Sign up for GitHub

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

Jump to bottom
Closed
cmaglie opened this issue Nov 15, 2012 · 0 comments
Closed

Sketch .c, .cpp, and .h files not searched for library includes. [imported] #636

cmaglie opened this issue Nov 15, 2012 · 0 comments
Assignees
@ffissore
Labels
Type: Wontfix Arduino has decided that it will not resolve the reported issue or implement the requested feature
Milestone
Release 1.6.6

Comments

@cmaglie
Copy link
Member

cmaglie commented Nov 15, 2012

This is Issue 636 moved from a Google Code project.
Added by 2011-09-09T15:36:21.000Z by [email protected].
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Defect, Priority-Medium, Component-IDE

Original description

What steps will reproduce the problem?

  1. Create sketch with additional .h/.cpp files (class)
  2. In the .cpp include an Arduino library
    Do not include said library in the main sketch file
  3. Build the sketch

What is the expected output? What do you see instead?
Expected: The sketch should build.
Result: Build error due to missing include directories

What version of the Arduino software are you using? On what operating
system? Which Arduino board are you using?
Arduino 0022 on Arch Linux with a variety of boards

Please provide any additional information below.
Adding the same library to the main file resolves this issue since then the scrips include the neccessary include directores in the build process.

It seems to me that the build script should add the include directories needed even if I don't include them in the main file, or at least give me and option of specifying include directories manually.
This was referenced Dec 8, 2013
Closed
Closed
@ffissore ffissore added the New label Feb 27, 2014
matthijskooijman added a commit to matthijskooijman/Arduino that referenced this issue Jul 10, 2014
@matthijskooijman
Move include detection from PdeProcessor/Sketch to the new HeuristicR…
aa08d13 
…esolver class

This offers a more consistent interface to the library autodetection
that can next be expanded to also detect inter-library dependencies.

As a side effect, the resolver now looks through all sketch code,
including any .c, .cpp and .h files. Previously, only the .ino and .pde
files were inspected during preprocessing. This fixes arduino#636.

This commit is based on code by Christian Maglie.
@ffissore ffissore self-assigned this Jul 1, 2015
@ffissore ffissore added the Type: Wontfix Arduino has decided that it will not resolve the reported issue or implement the requested feature label Jul 3, 2015
@ffissore ffissore added this to the Release 1.6.6 milestone Jul 3, 2015
@ffissore ffissore closed this as completed Jul 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ffissore ffissore

Labels
Type: Wontfix Arduino has decided that it will not resolve the reported issue or implement the requested feature
Projects
None yet
Milestone
Release 1.6.6
Development

No branches or pull requests
3 participants
@matthijskooijman @cmaglie @ffissore