Skip to content

Template file extensions are not supported in sketches #1317

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
per1234 opened this issue Mar 9, 2018 · 7 comments
Closed

Template file extensions are not supported in sketches #1317

per1234 opened this issue Mar 9, 2018 · 7 comments
Labels
topic: code Related to content of the project itself topic: documentation Related to documentation for the project type: enhancement Proposed improvement

Comments

@per1234
Copy link
Contributor

per1234 commented Mar 9, 2018

Files with .tpp or .ipp extension in the sketch folder are not copied to the temporary build folder.

Reference: oxullo/Arduino-MAX30100#46

Fixed (partially) by #1316

@sehrgut
Copy link

sehrgut commented Aug 4, 2020

This has been open since 2018. Can we please have these standard extensions now?

@per1234 per1234 transferred this issue from arduino/arduino-builder Jun 14, 2021
@per1234 per1234 added topic: code Related to content of the project itself topic: documentation Related to documentation for the project type: enhancement Proposed improvement labels Jun 14, 2021
@silvanocerza
Copy link
Contributor

@ubidefeo @cmaglie
Do we want to add support for this extension? There is a recent PR adding it #1316 but am not sure if we really want to do this.
What about the Java IDE? I guess we'd have to do it there too? Or maybe not? 👀

@tshcherban
Copy link
Contributor

@silvanocerza is there some particular problem why not add those files support?
I'm often putting libraries into 'src' subfolder within a sketch directory, and some of them are written with modern c++ features including .tpp
IDE support is not necessary, since library file is not edited so often, but compilation would be nice.

@silvanocerza
Copy link
Contributor

No particular problem really, just wanted to ask feedback to others since it's a long standing issue and there might be reasons it has not been done already.

As far as I know C and C++ don't have specifications for file extensions, you could even omit them I think. Nonetheless I understand it might be useful to add support for other extensions.

My main concern is that adding this one would mean having to release a new version of the Java IDE to support it there too, I understand you might not need it but I would very much prefer if we keep them in sync for now.

@tshcherban
Copy link
Contributor

OK, i see. Just was not very familiar with IDE repo, thought that this list (of extensions) is shared and reused in IDE Gui code.

@silvanocerza
Copy link
Contributor

Nope, only the new IDE 2.0 uses the same extensions since it uses the CLI as its backend, the Java IDE supported extensions are defined here:

https://github.com/arduino/Arduino/blob/a55a6d2a9d62b7bddbd9dbe9de784cbbbf6540a3/arduino-core/src/processing/app/Sketch.java#L19-L23

@silvanocerza
Copy link
Contributor

Fixed by #1316.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

No branches or pull requests

4 participants