]> granicus.if.org Git - python/commitdiff
removing the force-optimized option as discussed in #1533164
authorTarek Ziadé <ziade.tarek@gmail.com>
Mon, 2 Mar 2009 05:38:44 +0000 (05:38 +0000)
committerTarek Ziadé <ziade.tarek@gmail.com>
Mon, 2 Mar 2009 05:38:44 +0000 (05:38 +0000)
Lib/distutils/command/bdist_rpm.py
Lib/distutils/tests/test_bdist_rpm.py

index 34822ec70aaf2348e07200ac0005fcbdca6f523e..1eccc6a48a9a73cbb3cdd9f13677163d508bbe5a 100644 (file)
@@ -126,18 +126,10 @@ class bdist_rpm (Command):
 
         ('quiet', 'q',
          "Run the INSTALL phase of RPM building in quiet mode"),
-
-        # Forces the -O1 option when calling the install command,
-        # so the rpm contains all files needed for proper operation under
-        # SELinux. Some systems checks for this on build-time and will
-        # fail without this.
-        ('force-optimize', None,
-         "Forces the -O1 option when calling the install command"),
-
         ]
 
     boolean_options = ['keep-temp', 'use-rpm-opt-flags', 'rpm3-mode',
-                       'no-autoreq', 'quiet', 'force-optimize']
+                       'no-autoreq', 'quiet']
 
     negative_opt = {'no-keep-temp': 'keep-temp',
                     'no-rpm-opt-flags': 'use-rpm-opt-flags',
@@ -188,7 +180,6 @@ class bdist_rpm (Command):
 
         self.force_arch = None
         self.quiet = 0
-        self.force_optimize = 1
 
     # initialize_options()
 
@@ -504,14 +495,8 @@ class bdist_rpm (Command):
         # that we open and interpolate into the spec file, but the defaults
         # are just text that we drop in as-is.  Hmmm.
 
-        # forcing -O1 if force-optimize
-        if self.force_optimize:
-            optimize = ' -O1'
-        else:
-            optimize = ''
-
-        install_cmd = ('%s install%s --root=$RPM_BUILD_ROOT '
-                       '--record=INSTALLED_FILES') % (def_setup_call, optimize)
+        install_cmd = ('%s install -O1 --root=$RPM_BUILD_ROOT '
+                       '--record=INSTALLED_FILES') % def_setup_call
 
         script_options = [
             ('prep', 'prep_script', "%setup -n %{name}-%{unmangled_version}"),
index ff40628f422056668ec1b080411e4d4f319eac15..2d84007f877a46e4f3735e999b2e0479a5218407 100644 (file)
@@ -109,8 +109,6 @@ class BuildRpmTestCase(support.TempdirManager,
         cmd = bdist_rpm(dist)
         cmd.fix_python = True
 
-        # running with force-optimize = 1
-        # and quiet = 1
         cmd.quiet = 1
         cmd.ensure_finalized()
         cmd.run()
@@ -119,29 +117,6 @@ class BuildRpmTestCase(support.TempdirManager,
         self.assert_('foo-0.1-1.noarch.rpm' in dist_created)
         os.remove(os.path.join(pkg_dir, 'dist', 'foo-0.1-1.noarch.rpm'))
 
-        # XXX I am unable yet to make this test work without
-        # spurious stderr output
-        # so returning until distutils.spawn acts better
-        return
-
-        # running with force-optimize = 0
-        cmd.force_optimize = 0
-        try:
-            # XXX How to prevent the spawned
-            # rpmbuild command to display errors ?
-            # this can be a problem for buildbots
-            cmd.ensure_finalized()
-            cmd.run()
-        except DistutilsExecError:
-            # happens only under Fedora/RedHat
-            # and some flavors of Linux
-            # otherwise it's a bug
-            if sys.platform == 'linux2':
-                return
-
-        dist_created = os.listdir(os.path.join(pkg_dir, 'dist'))
-        self.assert_('foo-0.1-1.noarch.rpm' in dist_created)
-
 def test_suite():
     return unittest.makeSuite(BuildRpmTestCase)