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

Make config files and command line specifications of configurations consistent

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      PR

      Description

      There is an irritating inconsistency between how you specify configuration parameters on the command line (config.foo.bar) and in a configuration file (root.foo.bar). This is especially annoying when you use --show config to obtain the name, cut and paste the answer into a file, and are rewarded with:

      error: cannot load config file 'ptfConfig.py': name 'config' is not defined

      This RFC proposes that we adopt a consist naming convention. I prefer config everywhere, but (as Kian-Tat Lim points out) that's a slightly more complex change than adopting root everywhere. On the other hand, I think that adopting config is much clearer, and wouldn't tempt us to change the the pipe_base's -show config[=glob] option.

        Attachments

          Issue Links

            Activity

            rhl Robert Lupton created issue -
            ktl Kian-Tat Lim made changes -
            Field Original Value New Value
            Resolution Done [ 10000 ]
            Status Proposed [ 10805 ] Adopted [ 10806 ]
            rowen Russell Owen made changes -
            Link This issue is triggering DM-3630 [ DM-3630 ]
            tjenness Tim Jenness made changes -
            Status Adopted [ 10806 ] Implemented [ 11105 ]
            jbosch Jim Bosch made changes -
            Link This issue relates to DM-32759 [ DM-32759 ]

              People

              Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Watchers:
              Gregory Dubois-Felsmann, Jim Bosch, John Swinbank, Kian-Tat Lim, Paul Price, Robert Lupton, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.