Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7163

pex_config should not mandate print function

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pex_config
    • Labels:
      None

      Description

      In DM-7047 we inadvertently added future print_function to the code that parses config files. This led to ci_hsc failing because it uses configs that have not yet been ported to modern python usage. Since some configs will never be converted pex_config should try not to change the parsing environment for configs.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Removed one line. ci_hsc now works. The fix was discussed on HipChat. No review required.

            Show
            tjenness Tim Jenness added a comment - Removed one line. ci_hsc now works. The fix was discussed on HipChat. No review required.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Watchers:
              Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.