#230 Report autorebuild disabled by default
Closed 6 years ago by bfontecc. Opened 6 years ago by bfontecc.
bfontecc/rpkg bfontecc  into  master

file modified
+5 -3
@@ -2648,15 +2648,17 @@ 

  

      def container_build_setup(self, get_autorebuild=None,

                                set_autorebuild=None):

+         autorebuild_default = 'disabled'

'disabled' is not a valid option.

          cfp = configparser.SafeConfigParser()

          if os.path.exists(self.osbs_config_filename):

              cfp.read(self.osbs_config_filename)

  

          if get_autorebuild is not None:

-             if not cfp.has_option('autorebuild', 'enabled'):

-                 self.log.info('true')

+             if not cfp.has_option('autorebuild', autorebuild_default):

+                 self.log.info('false')

              else:

-                 self.log.info('true' if cfp.getboolean('autorebuild', 'enabled') else 'false')

This shouldn't change.

+                 self.log.info('false' if cfp.getboolean('autorebuild',

+                                                         autorebuild_default) else 'true')

          elif set_autorebuild is not None:

              if not cfp.has_section('autorebuild'):

                  cfp.add_section('autorebuild')

'disabled' is not a valid option.

Could you also include some reason why this change is made in the commit message?

Pull-Request has been closed by bfontecc

6 years ago