#2440 vm: revert to RawConfigParser
Merged 3 years ago by tkopecek. Opened 3 years ago by tkopecek.
tkopecek/koji issue2277  into  master

file modified
+2 -5
@@ -48,7 +48,7 @@ 

  # urllib is required by the SCM class which is substituted into this file

  # do not remove the import below

  from six.moves import urllib  # noqa: F401

- from six.moves.configparser import ConfigParser, SafeConfigParser

+ from six.moves.configparser import RawConfigParser

  

  MANAGER_PORT = 7000

  
@@ -222,10 +222,7 @@ 

          elif len(specfiles) > 1:

              raise BuildError('Multiple .ini files found')  # noqa: F821

  

-         if six.PY2:

-             conf = SafeConfigParser()

-         else:

-             conf = ConfigParser()

+         conf = RawConfigParser()

          conf.read(os.path.join(self.spec_dir, specfiles[0]))

  

          # [naming] section

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

3 years ago

@jcupova It is for windows builds. If you're not going to test it, I'll change the flag to no_qe.

Metadata Update from @tkopecek:
- Pull-request untagged with: testing-ready
- Pull-request tagged with: no_qe

3 years ago

Commit fae5bcc fixes this pull-request

Pull-Request has been merged by tkopecek

3 years ago