GitLab

* fix filter sketch source

* removed lines generated from preprocessor

* remove file added by mistake

* remove lines generated by preprocessor in c_make file

* introduce again ContainerAddPrototypes

* Migrate release creation responsibility from goreleaser to GH actions

* replace s3 pointer with secret

* Cosmetics on .goreleaser.yml

* Cosmetics on .goreleaser.yml again

* Cleanup and cosmetics

Fix https://github.com/arduino/Arduino/issues/9724

Fix #493
Fix https://github.com/arduino/Arduino/issues/9287

General change:
 - library compilation bails out if the precompiled object is found.
   This allows mixed libraries that fallback using sources if no suitable precompiled object is found

ARM float specification change:
 - Cortex M4+ allows specifying different flags for floating point ABI
   This patch introduces a second level of subfolder that MUST be used is -mfpu or -mfloat-abi are specified on the commandline
   Since there's no clear specification (unlike build.mcu), the values are extracted from c++ recipe

   For example, for a target which commandline contains `-mfloat-abi=hard -mfpu=fpv4-sp-d16` , the precompiled search path will be
   `$libfolder/cortex-m4/fpv4-sp-d16-hard`
   If that folder doesn't exist the library will be compiled from sources

Fixes https://github.com/arduino/arduino-builder/issues/256

* Reorganized tests

* Fixed library.IsArchitectureIndependent method.

Now it returns true for libraries that do not specify "architecture"
fiels.

* Fixed yet another even more convoluted case in bundle lib selection

Fix #572

* Makes linter happy...

* feat: Add cache clean

Add command `cache clean` to clean the cache files
depending on the loaction of the caching files
under different OS.

* feat: Add test for cache clean

Clean cache files with command `arduino-cli cache clean`.

* simplify test code
Co-authored-by: Massimiliano Pippi <mpippi@gmail.com>

Fix #534

* Added test for new use cases

* Moved PriorityForArchitecture in librariesresolver

The method is used only in library resolver, there is not point in
having it as method of Library.

Removed also the unused SortByArchitecturePriority method.

* Fixed library priority selection

* Added another fix and test for lib priority

* move compilation tests to pytest

* moved core tests to pytest

* moved core install/upgrade/uninstall to pytest

* Added detected_boards fixture
Co-authored-by: Maurizio Branca <m.branca@arduino.cc>

* add upload tests, remove mocked ones

* use fixture

* docs + dont run go tests anymore

* remove debug prints, bump pytest to latest

* skip problematic test on win
Co-authored-by: Maurizio Branca <m.branca@arduino.cc>

* use a real archive as a fake core

* fix core search for 3rd party cores with tests

* Removed ignored GRPC api

* Regenerate proto files

* go mod tidy
Co-authored-by: Roberto Sora <r.sora@arduino.cc>

* [skip changelog] Pass path and contents separately to SketchSaveItemCpp

Previously, these two arguments were wrapped together in a sketch.Item,
but since all callers build such an item during the call, there is no
compelling reason to do it like this. This commit splits the Item
parameter into a separate path and contents, which prepares for removing
the file contents from sketch.Item later.

* [skip changelog] Let SketchMergeSources return error

This adds an error return value, which is currently always nil. This
prepares for making changes that require returning errors.

* [skip changelog] Let sketch.Item.GetSourceStr return error

This adds an error return value to this method, which is currently
always nil. This prepares for actually returning errors later.

* Load sketch file contents only when needed

Previously, the full contents of *all* sketch files would be loaded into
memory. This includes all source and header files inside the sketch
directory, even when they will not even be compiled (e.g.
subdirectories other than src). In practice, only the .ino file contents
will actually be used, so these are now read on demand.

Note that when copying the sketch into the build directory, the contents
of all these sketch files *is* used, but that code (`writeIfDifferent()`
in `arduino/builder/sketch.go`) already did not use the preloaded data
but read the file contents when copying.

For small sketches, this does not make much of a difference, but bigger
sketches, especially when they include libraries, core definitions,
tools, examples, documentation, etc. the memory usage can quite explode,
for no good reason.

[skip changelog] Demonstrate gRPC interface's LibraryResolveDependencies method in client_example (#552)

- Demonstrate SetValue, Merge, GetValue, and GetAll.
- Switch from the deprecated configuration request field of Init to using the Settings service to configure arduino-cli's directories.

* [skip changelog] [skip ci] Clarify additional-urls option usage

* rephrase flag warning

* stress 3rd party point

* avoid paniking if main sketch file is missing

* rewrite the sketch contents collection logic + tests

* fix tests

* fix regression on board attach

* fix integration tests

* Improve error message on build failure
* properly report an error message

* remove error checking on unused proto field, added deprecation comment

* fix typo in function name

* do not os.exit in library code

* create data dir tree if it doesnt exist

* Improve error message when port or fqbn flags are missing

* Redirect stderr for integration tests too

* convert to rST
* downloads links in table
* refactored the walkthrough
* slim down install instructions

* rename Sketchbook folder to User

* missing join on path

* define proto messages and service

* run protoc on settings interface, regenerate code

* add service implementation and tests

* remove test leftovers

* use pip-tools, bump packages to support 3.8

* Bump Python to 3.8 in the CI environment