Introduce warnings.cache file

It shares the file with build options cache at the moment, since they match almost perfectly

Author: facchinm

builder.go

@@ -120,6 +120,7 @@ func (s *Builder) Run(ctx *types.Context) error {
 		&MergeSketchWithBootloader{},
 
 		&RecipeByPrefixSuffixRunner{Prefix: constants.HOOKS_POSTBUILD, Suffix: constants.HOOKS_PATTERN_SUFFIX},
+		&StoreWarningsMap{},
 	}
 
 	mainErr := runCommands(ctx, commands, true)

constants/constants.go

@@ -90,6 +90,7 @@ const FILE_PLATFORM_LOCAL_TXT = "platform.local.txt"
 const FILE_PLATFORM_TXT = "platform.txt"
 const FILE_PROGRAMMERS_TXT = "programmers.txt"
 const FILE_INCLUDES_CACHE = "includes.cache"
+const FILE_WARNINGS_CACHE = "warnings.cache"
 const FOLDER_BOOTLOADERS = "bootloaders"
 const FOLDER_CORE = "core"
 const FOLDER_CORES = "cores"

container_build_options.go

@@ -40,6 +40,7 @@ func (s *ContainerBuildOptions) Run(ctx *types.Context) error {
 	commands := []types.Command{
 		&CreateBuildOptionsMap{},
 		&LoadPreviousBuildOptionsMap{},
+		&LoadPreviousWarningsMap{},
 		&WipeoutBuildPathIfBuildOptionsChanged{},
 		&StoreBuildOptionsMap{},
 	}

load_previous_build_options.go

@@ -30,12 +30,14 @@
 package builder
 
 import (
-	"github.com/arduino/arduino-builder/constants"
-	"github.com/arduino/arduino-builder/i18n"
-	"github.com/arduino/arduino-builder/types"
+	"encoding/json"
 	"io/ioutil"
 	"os"
 	"path/filepath"
+
+	"github.com/arduino/arduino-builder/constants"
+	"github.com/arduino/arduino-builder/i18n"
+	"github.com/arduino/arduino-builder/types"
 )
 
 type LoadPreviousBuildOptionsMap struct{}
@@ -60,3 +62,27 @@ func (s *LoadPreviousBuildOptionsMap) Run(ctx *types.Context) error {
 
 	return nil
 }
+
+type LoadPreviousWarningsMap struct{}
+
+func (s *LoadPreviousWarningsMap) Run(ctx *types.Context) error {
+	cachedWarningsFile := filepath.Join(ctx.BuildPath, constants.FILE_WARNINGS_CACHE)
+	ctx.WarningsCache = make(map[string]string)
+
+	_, err := os.Stat(cachedWarningsFile)
+	if err != nil {
+		if os.IsNotExist(err) {
+			return nil
+		}
+		return i18n.WrapError(err)
+	}
+
+	bytes, err := ioutil.ReadFile(cachedWarningsFile)
+	if err != nil {
+		return i18n.WrapError(err)
+	}
+
+	json.Unmarshal(bytes, &ctx.WarningsCache)
+
+	return nil
+}

store_build_options_map.go

@@ -30,10 +30,12 @@
 package builder
 
 import (
+	"encoding/json"
+	"path/filepath"
+
 	"github.com/arduino/arduino-builder/constants"
 	"github.com/arduino/arduino-builder/types"
 	"github.com/arduino/arduino-builder/utils"
-	"path/filepath"
 )
 
 type StoreBuildOptionsMap struct{}
@@ -42,3 +44,11 @@ func (s *StoreBuildOptionsMap) Run(ctx *types.Context) error {
 	utils.WriteFile(filepath.Join(ctx.BuildPath, constants.BUILD_OPTIONS_FILE), ctx.BuildOptionsJson)
 	return nil
 }
+
+type StoreWarningsMap struct{}
+
+func (s *StoreWarningsMap) Run(ctx *types.Context) error {
+	data, _ := json.Marshal(ctx.WarningsCache)
+	utils.WriteFile(filepath.Join(ctx.BuildPath, constants.FILE_WARNINGS_CACHE), string(data))
+	return nil
+}

types/context.go

@@ -65,6 +65,7 @@ type Context struct {
 	CodeCompletions string
 
 	WarningsLevel string
+	WarningsCache map[string]string
 
 	// Libraries handling
 	Libraries                  []*Library