This is a patch to allow the user to search sketches by default in the /libraries folder if it exists. This feature is referenced and/or requested here:

arduino/arduino-builder#223
#4936
#4936 (comment)
#9507
https://github.com/arduino/Arduino/issues/7755
#11106

This adds this feature to the Arduino IDE provided that the user has enabled it in their preferences. This preference is disabled by default in order to avoid breaking past behavior by default (in the exceedingly unlikely case that the user had a libraries directory in their sketch that they did NOT wish to be compiled).

This feature is not needed to be implemented in arduino-cli, because the user of arduino-cli could always manually add "--librariespwd" or similar very easily in their command line build. However, it may be a good idea to add by default as a potential future feature.

All Submissions:

Have you followed the guidelines in our Contributing document?
Have you checked to ensure there aren't other openPull Requests for the same update/change?

New Feature Submissions:

Does your submission pass tests?
Have you lint your code locally prior to submission?

Add the ability to find libraries in sketch folder.

48baf42

matthijskooijman reviewed

Dec 29, 2020

View reviewed changes

Copy link

Collaborator

matthijskooijman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I'd be very happy if this was finally supported. I hadn't realized that this would be so easy to implement.

This preference is disabled by default in order to avoid breaking past behavior by default (in the exceedingly unlikely case that the user had a libraries directory in their sketch that they did NOT wish to be compiled).

I wonder if this shouldn't just be enabled by default. If this is enabled by default, it really makes it easier to distribute self-contained sketches, without having to instruct recipients to explicitly enable this extra preference.

As you say, it seems rather unlikely that sketches have an existing libraries directory (if they have it, I'd think it would contain libraries intended to be used, originally by copying into the sketchbook libraries dir, which would now just be easier. Some sketches might have modified libraries intended to be included with relative includes rather than through the Arduino-generated include paths, but those would typically be insrc/libraries and even if not, such direct includes would still work and not trigger library detection for these libraries).

Going even further, I wonder if we need a visible preference for this at all? If this is just enabled by default, having a preference inpreferences.txt (but not shown in the GUI) to disable it just in case could be fine.

This feature is not needed to be implemented in arduino-cli, because the user of arduino-cli could always manually add "--libraries pwd" or similar very easily in their command line build. However, it may be a good idea to add by default as a potential future feature.

I would really want to see this in arduino-cli as well (also enabled by default IMHO), since then a single sketch will end up behaving the same between -cli and the IDE. I suspect that this is similarly easy to implement in arduino-cli, though.

Did you check what the priority of this extra libraries directory will be? IIRC arduino-builder has some code to apply different priorities to different library folders (In particular the platform-specific libraries are treated specially, probably using the order specified on the commandline for other libraries?).

I haven't tested this yet, will do that soon as well.

app/src/cc/arduino/view/preferences/Preferences.java


		PreferencesData.setBoolean("ide.accessible",accessibleIDEBox.isSelected());

		PreferencesData.setBoolean("build.load_sketch_libraries",loadSketchLibrariesBox.isSelected());

Copy link

Collaborator

matthijskooijmanDec 29, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Should this beuse_sketch_libraries instead? These libraries are not loaded by default, they are made available and used when required. "Use" is also the term used in the checkbox text?

Copy link

Author

Steve132 commentedDec 29, 2020

Going even further, I wonder if we need a visible preference for this at all? If this is just enabled by default, having a preference in preferences.txt (but not shown in the GUI) to disable it just in case could be fine.

I honestly tend to agree, but since this is my first contribution to Arduino I thought it might be better to avoid any breaking changes and to make sure I dotted my i's and crossed my t's with regards to possibly making breaking changes.

I would really want to see this in arduino-cli as well (also enabled by default IMHO), since then a single sketch will end up behaving the same between -cli and the IDE. I suspect that this is similarly easy to implement in arduino-cli, though.

This is true. I didn't do this yet, but I can if this patch gets merged. I don't want to make the change to arduino-cli until then.

Did you check what the priority of this extra libraries directory will be? IIRC arduino-builder has some code to apply different priorities to different library folders (In particular the platform-specific libraries are treated specially, probably using the order specified on the commandline for other libraries?).

I saw that, but the only way I mitigated it is to make sure this comes first.

Copy link

CLAassistant commentedApr 9, 2021•
edited
Loading

All committers have signed the CLA.

Copy link

jwktje commentedAug 4, 2021

I really would love this feature to be added.

Copy link

Stefanhg commentedMay 14, 2022

Why is this still not added??!?
It is such a basic function of wanting to modify a library because it is missing a function or because you are working in a team and want to ensure your sketch shares the same library.

per1234 added feature request

A request to make an enhancement (not a bug fix)

Component: CompilationRelated to compilation of Arduino sketches labels

May 15, 2022

Copy link

Levi-Lesches commentedMar 20, 2023

Hi, what's the status of this PR?

Copy link

stg commentedDec 6, 2023

This is a simple, basic and completely obvious feature, that I for one think should be enabled by default. It needs to be implemented yesterday. Not having this has been a huge problem for so long I can't even remember when I first ran into it and wondered how it had been missed. The src/ workaround does not work when libraries reference other libraries.