modulesync 4.0.1 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf27b8e1ab5da1a382062fb50cf5fef7b1c24ec6df90988b9033b03a8293a5ae
-  data.tar.gz: f45b4745c56ecb09f2e41e7b7df376fa55da074d0e7eda123428282734ed1e4f
+  metadata.gz: 5e2ed989c9e8e1837a6839019ab67c2defd8c61c2c724e2f0b9691a51012258e
+  data.tar.gz: 94f5d3040444e96ed1f22a4860d9e2d1c15aace660300cc5f0e4ee2194eb4bfe
 SHA512:
-  metadata.gz: 04116e1a9f754869a52d7b008bc079ab5b9018fbc5a2d92a0f065a5b65dcdaf7970701e35787b2f781dc7a8b74856e7fc81c9b4bc2de76b6ac8d5006ea29dcc1
-  data.tar.gz: 5bb03dd6cfd926a4f4249647dd54b5cf088a81015d6bf5493a7e7c6cfff657597e82884421c48814aedfcf01dba4da99290c8a89f848ac8eb6d0b5bea1595477
+  metadata.gz: 1b2b04b08c73b482e4f33f6b06ee038c95b334480b55a61751dc2360484157b8007f30378bf6d1e134ac1448e20930473f2e139fe86d62ef5f34897c6b59c771
+  data.tar.gz: 18d9cc8cff9cfbf5e68fa2c1818802701387f768d0a293cd6a950cf2bad297cd9dc7e8822858bc0d427c1a143d4b7bef72d7ce28889c61841dbc3b5d40a56507

data/.github/workflows/ci.yml

@@ -46,6 +46,11 @@ jobs:
         run: bundle exec rake spec
       - name: Run behavior tests
         run: bundle exec cucumber
+        env:
+          GIT_COMMITTER_NAME: 'Aruba'
+          GIT_COMMITTER_EMAIL: 'aruba@example.com'
+          GIT_AUTHOR_NAME: 'Aruba'
+          GIT_AUTHOR_EMAIL: 'aruba@example.com'
       - name: Build gem
         run: gem build --strict --verbose *.gemspec
 

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to this project will be documented in this file.
 
+## [4.1.0](https://github.com/voxpupuli/modulesync/tree/4.1.0) (2025-10-16)
+
+[Full Changelog](https://github.com/voxpupuli/modulesync/compare/4.0.1...4.1.0)
+
+**Implemented enhancements:**
+
+- Add execute option to preserve env vars [\#321](https://github.com/voxpupuli/modulesync/pull/321) ([Joris29](https://github.com/Joris29))
+- Update git requirement from ~\> 3.0 [\#280](https://github.com/voxpupuli/modulesync/pull/280) ([dependabot[bot]](https://github.com/apps/dependabot))
+
 ## [4.0.1](https://github.com/voxpupuli/modulesync/tree/4.0.1) (2025-09-02)
 
 [Full Changelog](https://github.com/voxpupuli/modulesync/compare/4.0.0...4.0.1)

data/README.md

@@ -395,6 +395,14 @@ msync update -m "Commit message" --bump --tag --tag_pattern 'v%s'
 
 The default for the tag pattern is `%s`.
 
+#### Executing bundle command with a shared bundle path and gemfile
+
+By default `msync execute` will remove bundler related env vars (^BUNDLE|^SOURCE_DATE_EPOCH$|^GEM_|RUBY), you can use `--env` flag to keep specified env vars:
+
+```
+msync execute --env BUNDLE_PATH,BUNDLE_GEMFILE bundle exec rake lint
+```
+
 #### Updating the CHANGELOG
 
 When bumping the version in `metadata.json`, modulesync can let you

data/features/cli.feature

@@ -45,5 +45,5 @@ Feature: CLI
     When I successfully run `msync update --verbose --noop --namespace fakenamespace --branch command-line-branch`
     Then the output should contain:
       """
-      Creating new branch command-line-branch
+      Creating new branch command-line-branch from master
       """

data/features/execute.feature

@@ -49,3 +49,14 @@ Feature: execute
     [--fail-fast], [--no-fail-fast], [--skip-fail-fast]                 # Abort the run after a command execution failure
     """
     # NOTE: It seems there is a Thor bug here: default value is missing in help when sets to 'false'
+
+  Scenario: Show preserved bundle env var.
+    Given a basic setup with a puppet module "puppet-test" from "awesome"
+    And I set the environment variables to:
+      | variable   | value  |
+      | BUNLE_PATH | /opt/msync/bundle |
+    When I successfully run `msync exec --env BUNLE_PATH -- env`
+    Then the stdout should contain:
+    """
+    BUNLE_PATH=/opt/msync/bundle
+    """

data/lib/modulesync/cli.rb

@@ -184,6 +184,9 @@ module ModuleSync
              type: :boolean,
              desc: 'Abort the run after a command execution failure',
              default: CLI.defaults[:fail_fast].nil? || CLI.defaults[:fail_fast]
+      option :env,
+             aliases: '-e',
+             desc: 'Comma-separated list of environment variables to preserve.'
       def execute(*command_args)
         raise Thor::Error, 'COMMAND is a required argument' if command_args.empty?
 

data/lib/modulesync/repository.rb

@@ -145,10 +145,15 @@ module ModuleSync
         opts_commit = { amend: true } if options[:amend]
         opts_push = { force: true } if options[:force]
         if options[:pre_commit_script]
-          script = "#{File.dirname(__FILE__, 2)}/../contrib/#{options[:pre_commit_script]}"
-          `#{script} #{@directory}`
+          script = "#{File.dirname(__FILE__, 3)}/contrib/#{options[:pre_commit_script]}"
+          system(script, @directory)
+        end
+        if repo.status.changed.empty? && repo.status.added.empty? && repo.status.deleted.empty?
+          puts "There were no changes in '#{@directory}'. Not committing."
+          return false
+        else
+          repo.commit(message, opts_commit)
         end
-        repo.commit(message, opts_commit)
         if options[:remote_branch]
           if remote_branch_differ?(branch, options[:remote_branch])
             repo.push('origin', "#{branch}:#{options[:remote_branch]}", opts_push)
@@ -158,11 +163,8 @@ module ModuleSync
           repo.push('origin', branch, opts_push)
           puts "Changes have been pushed to: '#{branch}'"
         end
-      rescue Git::GitExecuteError => e
-        raise unless e.message.match?(/working (directory|tree) clean/)
-
-        puts "There were no changes in '#{@directory}'. Not committing."
-        return false
+      rescue Git::Error => e
+        raise e.message
       end
 
       true
@@ -191,7 +193,7 @@ module ModuleSync
       checkout_branch(options[:branch])
 
       $stdout.puts 'Files changed:'
-      repo.diff('HEAD', '--').each do |diff|
+      repo.diff('HEAD').each do |diff|
         $stdout.puts diff.patch
       end
 
@@ -203,7 +205,7 @@ module ModuleSync
       $stdout.puts "\n\n"
       $stdout.puts '--------------------------------'
 
-      git.diff('HEAD', '--').any? || untracked_unignored_files.any?
+      git.diff('HEAD').any? || untracked_unignored_files.any?
     end
 
     def puts(*)

data/lib/modulesync.rb

@@ -178,7 +178,7 @@ module ModuleSync
     # managed_modules is either an array or a hash
     managed_modules.each do |puppet_module|
       manage_module(puppet_module, module_files, defaults)
-    rescue ModuleSync::Error, Git::GitExecuteError => e
+    rescue ModuleSync::Error, Git::Error => e
       message = e.message || 'Error during `update`'
       warn "#{puppet_module.given_name}: #{message}"
       exit 1 unless options[:skip_broken]
@@ -216,11 +216,12 @@ module ModuleSync
       end
 
       command_args = cli_options[:command_args]
+      env_whitelist = (@options[:env] || '').split(',')
       local_script = File.expand_path command_args[0]
       command_args[0] = local_script if File.exist?(local_script)
 
-      # Remove bundler-related env vars to allow the subprocess to run `bundle`
-      command_env = ENV.reject { |k, _v| k.match?(/(^BUNDLE|^SOURCE_DATE_EPOCH$|^GEM_|RUBY)/) }
+      # Remove bundler-related env vars to allow the subprocess to run `bundle` unless explicitly whitelisted.
+      command_env = ENV.reject { |k, _v| k.match?(/(^BUNDLE|^SOURCE_DATE_EPOCH$|^GEM_|RUBY)/) && !env_whitelist.include?(k) }
 
       result = system command_env, *command_args, unsetenv_others: true, chdir: puppet_module.working_directory
       unless result

data/lib/monkey_patches.rb

@@ -2,9 +2,9 @@
 
 module Git
   module LibMonkeyPatch
-    # Monkey patch set_custom_git_env_variables due to our ::Git::GitExecuteError handling.
+    # Monkey patch set_custom_git_env_variables due to our ::Git::Error handling.
     #
-    # We rescue on the GitExecuteError and proceed differently based on the output of git.
+    # We rescue on the Git::Error and proceed differently based on the output of git.
     # This way makes code language-dependent, so here we ensure that Git gem throw git commands with the "C" language
     def set_custom_git_env_variables
       super

data/modulesync.gemspec

@@ -5,7 +5,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name                  = 'modulesync'
-  spec.version               = '4.0.1'
+  spec.version               = '4.1.0'
   spec.authors               = ['Vox Pupuli']
   spec.email                 = ['voxpupuli@groups.io']
   spec.summary               = 'Puppet Module Synchronizer'
@@ -24,7 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec', '~> 3.13'
   spec.add_development_dependency 'voxpupuli-rubocop', '~> 4.2.0'
 
-  spec.add_dependency 'git', '~>1.7'
+  spec.add_dependency 'git', '~> 3.0'
   spec.add_dependency 'gitlab', '>=4', '<6'
   spec.add_dependency 'octokit', '>=4', '<10'
   spec.add_dependency 'puppet-blacksmith', '>= 3.0', '< 9'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: modulesync
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.1.0
 platform: ruby
 authors:
 - Vox Pupuli
@@ -91,14 +91,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: gitlab
   requirement: !ruby/object:Gem::Requirement