Sketch is fully copied in tmp when building

Should fix: arduino/Arduino#5186

Signed-off-by: Cristian Maglie <[email protected]>

Author: cmaglie

src/arduino.cc/builder/sketch_loader.go

@@ -92,18 +92,11 @@ func collectAllSketchFiles(from string) ([]string, error) {
 	// Source files in the root are compiled, non-recursively. This
 	// is the only place where .ino files can be present.
 	rootExtensions := func(ext string) bool { return MAIN_FILE_VALID_EXTENSIONS[ext] || ADDITIONAL_FILE_VALID_EXTENSIONS[ext] }
-	err := utils.FindFilesInFolder(&filePaths, from, rootExtensions, false /* recurse */)
+	err := utils.FindFilesInFolder(&filePaths, from, rootExtensions, true /* recurse */)
 	if err != nil {
 		return nil, i18n.WrapError(err)
 	}
 
-	// The "src/" subdirectory of a sketch is compiled recursively
-	// (but .ino files are *not* compiled)
-	srcPath := filepath.Join(from, constants.SKETCH_FOLDER_SRC)
-	if info, err := os.Stat(srcPath); err == nil && info.IsDir() {
-		srcExtensions := func(ext string) bool { return ADDITIONAL_FILE_VALID_EXTENSIONS[ext] }
-		err = utils.FindFilesInFolder(&filePaths, srcPath, srcExtensions, true /* recurse */)
-	}
 	return filePaths, i18n.WrapError(err)
 }
 

src/arduino.cc/builder/test/sketch_loader_test.go

@@ -30,11 +30,12 @@
 package test
 
 import (
+	"path/filepath"
+	"testing"
+
 	"arduino.cc/builder"
 	"arduino.cc/builder/types"
 	"github.com/stretchr/testify/require"
-	"path/filepath"
-	"testing"
 )
 
 func TestLoadSketchWithFolder(t *testing.T) {
@@ -122,9 +123,12 @@ func TestLoadSketchFromFolder(t *testing.T) {
 
 	require.Equal(t, 0, len(sketch.OtherSketchFiles))
 
-	require.Equal(t, 2, len(sketch.AdditionalFiles))
-	require.Contains(t, sketch.AdditionalFiles[0].Name, "other.cpp")
-	require.Contains(t, sketch.AdditionalFiles[1].Name, "other.h")
+	require.Equal(t, 5, len(sketch.AdditionalFiles))
+	require.Contains(t, sketch.AdditionalFiles[0].Name, "sketch_with_subfolders/src/subfolder/other.cpp")
+	require.Contains(t, sketch.AdditionalFiles[1].Name, "sketch_with_subfolders/src/subfolder/other.h")
+	require.Contains(t, sketch.AdditionalFiles[2].Name, "sketch_with_subfolders/subfolder/dont_load_me.cpp")
+	require.Contains(t, sketch.AdditionalFiles[3].Name, "sketch_with_subfolders/subfolder/other.cpp")
+	require.Contains(t, sketch.AdditionalFiles[4].Name, "sketch_with_subfolders/subfolder/other.h")
 }
 
 func TestLoadSketchWithBackup(t *testing.T) {

src/arduino.cc/builder/test/sketch_with_subfolders/subfolder/other.cpp

@@ -0,0 +1,10 @@
+#include <Arduino.h> // Arduino 1.0
+
+#include "other.h"
+
+MyClass::MyClass() {
+}
+
+void MyClass::init ( Stream *stream ) {
+    controllerStream = stream;
+}

src/arduino.cc/builder/test/sketch_with_subfolders/subfolder/other.h

@@ -0,0 +1,12 @@
+#ifndef other__h
+#define other__h
+
+class MyClass {
+  public:
+    MyClass();
+    void init ( Stream *controllerStream );
+
+  private:
+    Stream *controllerStream;
+};
+#endif