Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-125

Add pyyaml package to build

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      This JIRA issue

      Description

      daf_persistence uses YAML format policy files. In order to read YAML files we need a Python YAML reader. This RFC proposes (retroactively) that we adopt the PyYAML package and create a corresponding EUPS package.

        Attachments

          Issue Links

            Activity

            Hide
            jhoblitt Joshua Hoblitt added a comment -

            +1

            Show
            jhoblitt Joshua Hoblitt added a comment - +1
            Hide
            jsick Jonathan Sick added a comment -

            Would the ruamel package (a fork of pyyaml) be better in the long run? One of the big wins is round-trip preservation of YAML comments.

            Show
            jsick Jonathan Sick added a comment - Would the ruamel package (a fork of pyyaml) be better in the long run? One of the big wins is round-trip preservation of YAML comments.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            The pypi download stats suggest that pyyaml is about two orders of magnitude more popular.

            Show
            jhoblitt Joshua Hoblitt added a comment - The pypi download stats suggest that pyyaml is about two orders of magnitude more popular.
            Hide
            tjenness Tim Jenness added a comment -

            Is ruamel completely compatible (ie do we just change the import line?)

            Show
            tjenness Tim Jenness added a comment - Is ruamel completely compatible (ie do we just change the import line?)
            Hide
            jsick Jonathan Sick added a comment -

            I don’t think there’s any guarantee of API compatibility. I’m fine with pyyaml, but if we anticipate wanting to write YAML for humans, and not just consume it, then ruamel is worth consideration.

            Show
            jsick Jonathan Sick added a comment - I don’t think there’s any guarantee of API compatibility. I’m fine with pyyaml, but if we anticipate wanting to write YAML for humans, and not just consume it, then ruamel is worth consideration.
            Hide
            tjenness Tim Jenness added a comment -

            PyYAML will be used initially, without the LibYAML dependency.

            Show
            tjenness Tim Jenness added a comment - PyYAML will be used initially, without the LibYAML dependency.
            Hide
            tjenness Tim Jenness added a comment -

            The PyYAML package has been created.

            Show
            tjenness Tim Jenness added a comment - The PyYAML package has been created.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Watchers:
              Jonathan Sick, Joshua Hoblitt, Nate Pease [X] (Inactive), Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins Builds

                  No builds found.