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

Config.loadFromStream does not accept StringIO objects (and maybe files)

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Invalid
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pex_config
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture

      Description

      pex.config.Config.loadFromStream's documentation, name, and symmetry with saveToStream all state that it should accept arbitrary file-like objects, but (at least on Python 3) it appears to only accept strings, because it passes its first argument directly to exec, and that accepts only string or bytes objects.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            It looks like Russell Owen was aware of this issue given the code comments associated with DM-3630 in commit fcf2f7d48b4409510552106a877c7ee533481f0b

            Show
            tjenness Tim Jenness added a comment - It looks like Russell Owen was aware of this issue given the code comments associated with DM-3630 in commit fcf2f7d48b4409510552106a877c7ee533481f0b
            Hide
            tjenness Tim Jenness added a comment -

            The documentation does seem to be lying about this still.

            Show
            tjenness Tim Jenness added a comment - The documentation does seem to be lying about this still.
            Hide
            jbosch Jim Bosch added a comment -

            I somehow managed to rewrite this ticket almost verbatim almost 5 years later without remembering this one. Work has been done there, so I'm closing this as the duplicate.

            Show
            jbosch Jim Bosch added a comment - I somehow managed to rewrite this ticket almost verbatim almost 5 years later without remembering this one. Work has been done there, so I'm closing this as the duplicate.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jbosch Jim Bosch
              Watchers:
              Jim Bosch, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.