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

Consider eliminating the ups cfg file for a pure python package

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: templates
    • Labels:
      None

      Description

      A pure python package "foo" does not need ups/foo.cfg if SConstruct specifies noCfgFile=True, e.g.:

      scripts.BasicSConstruct("ts_foo", disableCc=True, noCfgFile=True)
      

      Consider whether to take advantage of this and eliminate the unnecessary file. I admit it'll be harder to add C++ code later, but it's already a bit of a chore and not a common change.

      I consider this a "nice to have" since the current system works just fine.

        Attachments

          Activity

          Hide
          jsick Jonathan Sick added a comment -

          Are you happy with this change, Kian-Tat Lim?

          Show
          jsick Jonathan Sick added a comment - Are you happy with this change, Kian-Tat Lim ?
          Hide
          ktl Kian-Tat Lim added a comment -

          Yes.  I think it's reasonably clear what's going on and it simplifies pure-Python packages.

          Show
          ktl Kian-Tat Lim added a comment - Yes.  I think it's reasonably clear what's going on and it simplifies pure-Python packages.
          Hide
          jsick Jonathan Sick added a comment -

          See https://github.com/lsst/templates/pull/54

          Does this update look good to you?

          Show
          jsick Jonathan Sick added a comment - See https://github.com/lsst/templates/pull/54 Does this update look good to you?
          Hide
          rowen Russell Owen added a comment -

          This looks very nice. Thank you for improvement.

          Show
          rowen Russell Owen added a comment - This looks very nice. Thank you for improvement.
          Hide
          jsick Jonathan Sick added a comment -

          The update is available from sqrbot-jr create project now.

          Show
          jsick Jonathan Sick added a comment - The update is available from sqrbot-jr create project now.

            People

            • Assignee:
              jsick Jonathan Sick
              Reporter:
              rowen Russell Owen
              Reviewers:
              Russell Owen
              Watchers:
              Jonathan Sick, Kian-Tat Lim, Russell Owen
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel