Skip to content

Commit 107a8e5

Browse files
committed
Drop specs when building gem
Previously our key and cert test fixtures were shipped in the gem, which triggered security scanners. This deletes the `spec` directory from each platform specific gem and reduces the size of each gem by about 1MB when using the `bundle exec rake package:gem` task: Before: ``` $ ls -la pkg/puppet-8.4.0.178.gem -rw-rw-r-- 1 josh 1.5M Feb 9 11:21 pkg/puppet-8.4.0.178.gem ``` After: ``` $ ls -la pkg/puppet-8.4.0.177.gem -rw-rw-r-- 1 josh 2.8M Feb 9 11:21 pkg/puppet-8.4.0.177.gem ``` For reasons described in PA-5400 we don't currently use `gem build .gemspec` to build puppet gems. No changes are needed to the .gemspec or Gemfile, because they both include the content from project_data.yaml.
1 parent a3e77e7 commit 107a8e5

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ext/project_data.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ summary: 'Puppet, an automated configuration management tool'
77
description: 'Puppet, an automated configuration management tool'
88
version_file: 'lib/puppet/version.rb'
99
# files and gem_files are space separated lists
10-
files: '[A-Z]* install.rb bin lib conf man examples ext tasks spec locales'
11-
# Make sure these gem requirements are in sync with the gempspec and Gemfile
12-
gem_files: '[A-Z]* install.rb bin lib conf man examples ext tasks spec locales'
13-
gem_test_files: 'spec/**/*'
10+
files: '[A-Z]* install.rb bin lib conf man examples ext tasks locales'
11+
# Make sure these gem requirements are in sync with the gemspec and Gemfile
12+
gem_files: '[A-Z]* install.rb bin lib conf man examples ext tasks locales'
13+
gem_test_files:
1414
gem_executables: 'puppet'
1515
gem_default_executables: 'puppet'
1616
gem_license: 'Apache-2.0'

0 commit comments

Comments
 (0)