Skip to content

Commit ba9c12b

Browse files
author
James Foster
authored
Merge pull request #290 from jgfoster/hide_build_artifacts
This simply pushes the build artifacts down one level into a `.arduino_ci` directory. Otherwise, all is the same.
2 parents a242213 + 4b1285f commit ba9c12b

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

Diff for: CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
1111
- `--min-free-space=N` command-line argument to fail if free space is below requred value
1212

1313
### Changed
14+
- Put build artifacts in a separate directory to reduce clutter.
1415
- Change 266 files from CRLF to LF.
1516
- Update .gitattributes so we have consistent line endings
1617
- Run tests on push as well as on a pull request so developers can see impact

Diff for: lib/arduino_ci/cpp_library.rb

+4-1
Original file line numberDiff line numberDiff line change
@@ -491,7 +491,10 @@ def test_args(aux_libraries, ci_gcc_config)
491491
# @return [Pathname] path to the compiled test executable
492492
def build_for_test_with_configuration(test_file, aux_libraries, gcc_binary, ci_gcc_config)
493493
base = test_file.basename
494-
executable = Pathname.new("unittest_#{base}.bin").expand_path
494+
# hide build artifacts
495+
build_dir = '.arduino_ci'
496+
Dir.mkdir build_dir unless File.exist?(build_dir)
497+
executable = Pathname.new("#{build_dir}/unittest_#{base}.bin").expand_path
495498
File.delete(executable) if File.exist?(executable)
496499
arg_sets = []
497500
arg_sets << ["-std=c++0x", "-o", executable.to_s, "-DARDUINO=100"]

0 commit comments

Comments
 (0)