GitLab

* Migrate relevant documentation from arduino/Arduino wiki

* Trim trailing whitespace

* Add newline to end of file

* Increase heading levels so that the minimum level is h2

This is required for table of contents generation.

* Use supported line break markup

The backslash at end of line to cause line break is not supported by the documentation build system.

* Update links to migrated pages

* Fix list markup

The documentation build system requires a blank line before the start of a list

* legacy: inline compileFilesWithExtensionWithRecipe function

This will help for a better progress management in a next commit since
now we know in advance how many file will be compiled in this task.

* Added missing copyright header

* Improved progress report during Compile

* legacy: use github.com/pkg/errors for error handling

* legacy: updated ErrrofWithLogger to use github.com/pkg/errors

* legacy: removed use of github.com/go-errors/errors in test helpers

* legacy: removed github.com/go-errors/errors from deps

* Added dry-run mode in Compile command

* Update cli/compile/compile.go
Co-Authored-By: Roberto Sora <r.sora@arduino.cc>
Co-authored-by: Roberto Sora <r.sora@arduino.cc>

* Apply cosmetics

* Implement ugly telemetry POC

* Added prefix and moved Insrumentation inside the command package

* Refactor the telemetry module

* Implement configuration via Viper

* Add stats flush in case of a not daemonized cli daemon proces

* Add repertory to store installation id and secret

* Repertory force write

* Cosmetics

* Use viper config for repertory dir

* Add test for repertory file creation

* Add testing for telemetry deaemon and repertory

* Wait for repertory and kill daemon

* Updated pyinvoke to use async feature to test the daemon

* Updated daemon test timeout

* Cosmetics

* Set getDefaultArduinoDataDir as unexported

* Cosmetics

* Cosmetics

* Cosmetics

* Lint on repertory module

* Set SIGTERM as kill signal in case of win platform to overcome pyinvoke bug

* Import platform as a module

* Reverse platform if for signal value

* Extract pid value

* Remove print leftover

* Add better error handling in repertory creation

* Update docs with old README extract

* Remove telemetry.pattern setting from docs

* Remove serverPattern config option for telemetry

* Upgrade viper to 1.6.2

* Defer stats Increment in compile command and explicit set for success/failure

* Fix board list help message

* Implement stats flush anyway to leverage module no-op in case of no handler configured

* Rename "repertory" module in "inventory" and refactor Sanitize function

* Sanitize ExportFile in command/compile

* Refactor daemon start fixture to include daemon process cleanup

* Use defer function to push success tag correctly updated

* Use named return parameters to handle success tagging for a command stat

* Remove upload port as required CLI parameter in "upload" command

* Add "board attach" integ test

* Read upload port from sketch.json

* Fix 'attach' help message

* fix builds when merging PRs

* move contributing guidelines into docs

* restore markdown format for the README

* move away documentation from the README, use dedicated pages

* add logo

* added command reference generator

* add mkdocs build

* add GH pages deploy step

* publish docs only when pr is merged

* added arduino logo

* automate grpc docs generation

* improve Taskfile to support local development

* add docs for working on docs, much meta

* Added 'license' and 'provides_includes' fields in lib search

* Added 'dependencies' field in lib search

* Do not output empty field in 'lib search'

Fields 'license', 'provided includes' and 'dependencies' are printed
only if populated.

* 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

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

* Send the correct signal to process

* 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.