GitLab

* When running debugger always use forward slash in paths

* Add more debugging output when running debugger commandline

* Temporary fix for arduino.samd core 1.8.5

* Fix gdb timeout on Windows

(cherry picked from commit 561618a4)

* Allow SIGINT (ctrl-C) to be forwarded to debugger process

* Send the correct signal to process

(cherry picked from commit 85effc8d)

* add debug symbols option in CLI and gRPC insterface

* Calculate precompiled core path usign also build optimization flags

* Align failing lecacy tests with new hashing behaviour

* Implement first draft of debugger gRPC service

* Working stdio streaming

* Improved stdio passing via GRPC

* Adjusted protoc definitions

* Handle errors gracefully

* Add recipe calculation to debug command

* First implementation of debug

* updated client example for testing

* Add test for debug recipe generation

* Implement debug command

* Implement copyStream

* Refactor stream helpers

* Extract recipe creation from debug command

* Cosmetics here and there

* Refreshed client example

* Replace with utils function

* Remove debug leftover

* Refreshed client example

* Moved debug proto to its package

* Removed sketch.json

* Apply general cosmetics

* Add test binaries

* Added test case for windows path flavor

* Use path.FromSlash to test debug tool command generation cross platform easily

* Avoid pipe leaking via closing readers and writes in case of abnormal termination

* Update client example to better catch gdb prompt

* Error messages cosmetics

* Use errors.Wrap instead of fmt.Errorf
Co-Authored-By: Massimiliano Pippi <m.pippi@arduino.cc>

* Use errors.Wrap instead of fmt.Errorf
Co-Authored-By: Massimiliano Pippi <m.pippi@arduino.cc>
Co-authored-by: Cristian Maglie <c.maglie@bug.st>
Co-authored-by: Massimiliano Pippi <mpippi@gmail.com>

- Consumers of the CLI have a better exit code when it fails at startup
 - Logged message to stdout, so gRPC clients know, it is time to connect
Signed-off-by: Akos Kitta <kittaakos@typefox.io>

* 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