Skip to content

Commit

Permalink
Don't remove empty directories after build.
Browse files Browse the repository at this point in the history
Deletion of these empty directories is flawed on docker with overlayfs,
so we better avoid it.
The build artifacts are still removed after gem install.

This fixes nokogiri issue sparklemotion#1370 .
For further details to the root issue see: moby/moby#19082
  • Loading branch information
larskanis committed Jan 5, 2016
1 parent 6b5be24 commit a53e702
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion ext/nokogiri/extconf.rb
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ def do_clean
# clean the ports build directory
Pathname.glob(pwd.join('tmp', '*', 'ports')) { |dir|
FileUtils.rm_rf(dir, verbose: true)
FileUtils.rmdir(dir.parent, parents: true, verbose: true)
}

if enable_config('static')
Expand Down

0 comments on commit a53e702

Please sign in to comment.