Skip to content

Commit b06c7ec

Browse files
author
Federico Fissore
committed
.S files of a sketch were ignored. Fixes #74
Signed-off-by: Federico Fissore <[email protected]>
1 parent 8067591 commit b06c7ec

File tree

4 files changed

+8
-6
lines changed

4 files changed

+8
-6
lines changed

Diff for: src/arduino.cc/builder/builder.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ import (
4141
)
4242

4343
var MAIN_FILE_VALID_EXTENSIONS = map[string]bool{".ino": true, ".pde": true}
44-
var ADDITIONAL_FILE_VALID_EXTENSIONS = map[string]bool{".h": true, ".c": true, ".hpp": true, ".cpp": true}
45-
var ADDITIONAL_FILE_VALID_EXTENSIONS_NO_HEADERS = map[string]bool{".c": true, ".cpp": true}
44+
var ADDITIONAL_FILE_VALID_EXTENSIONS = map[string]bool{".h": true, ".c": true, ".hpp": true, ".cpp": true, ".s": true}
45+
var ADDITIONAL_FILE_VALID_EXTENSIONS_NO_HEADERS = map[string]bool{".c": true, ".cpp": true, ".s": true}
4646

4747
var LIBRARY_MANDATORY_PROPERTIES = []string{constants.LIBRARY_NAME, constants.LIBRARY_VERSION, constants.LIBRARY_AUTHOR, constants.LIBRARY_MAINTAINER}
4848
var LIBRARY_NOT_SO_MANDATORY_PROPERTIES = []string{constants.LIBRARY_SENTENCE, constants.LIBRARY_PARAGRAPH, constants.LIBRARY_URL}

Diff for: src/arduino.cc/builder/test/additional_sketch_files_copier_test.go

+3-2
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,12 @@ func TestCopyOtherFiles(t *testing.T) {
7878

7979
files, err1 := gohasissues.ReadDir(filepath.Join(buildPath, constants.FOLDER_SKETCH))
8080
NoError(t, err1)
81-
require.Equal(t, 2, len(files))
81+
require.Equal(t, 3, len(files))
8282

8383
sort.Sort(ByFileInfoName(files))
8484
require.Equal(t, "header.h", files[0].Name())
85-
require.Equal(t, "subfolder", files[1].Name())
85+
require.Equal(t, "s_file.S", files[1].Name())
86+
require.Equal(t, "subfolder", files[2].Name())
8687

8788
files, err1 = gohasissues.ReadDir(filepath.Join(buildPath, constants.FOLDER_SKETCH, "subfolder"))
8889
NoError(t, err1)

Diff for: src/arduino.cc/builder/test/sketch1/s_file.S

Whitespace-only changes.

Diff for: src/arduino.cc/builder/test/sketch_loader_test.go

+3-2
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,10 @@ func TestLoadSketch(t *testing.T) {
9595
require.Contains(t, sketch.OtherSketchFiles[0].Name, "old.pde")
9696
require.Contains(t, sketch.OtherSketchFiles[1].Name, "other.ino")
9797

98-
require.Equal(t, 2, len(sketch.AdditionalFiles))
98+
require.Equal(t, 3, len(sketch.AdditionalFiles))
9999
require.Contains(t, sketch.AdditionalFiles[0].Name, "header.h")
100-
require.Contains(t, sketch.AdditionalFiles[1].Name, "helper.h")
100+
require.Contains(t, sketch.AdditionalFiles[1].Name, "s_file.S")
101+
require.Contains(t, sketch.AdditionalFiles[2].Name, "helper.h")
101102
}
102103

103104
func TestFailToLoadSketchFromFolder(t *testing.T) {

0 commit comments

Comments
 (0)