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

sconsUtils should not report chat about the C++ compiler when invoked with -Q

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: sconsUtils
    • Labels:
      None
    • Team:
      Architecture

      Description

      $ scons -Q opt=3 lib python tests
      CC is clang version 6.0
      Checking for C++11 support
      C+11 supported with '-std=c+11'
      $

      With -Q the three lines about the C and C++ compiler should be omitted. They are currently generated by log.info() commands.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Hi Frossie, we think this is a SQuaRE issue...

            Show
            swinbank John Swinbank added a comment - Hi Frossie, we think this is a SQuaRE issue...
            Hide
            tjenness Tim Jenness added a comment -

            Should -Q also be suppressing the doxygen lines:

            buildConfig(["doc/doxygen.conf"], ["doc/doxygen.conf.in"])
            doxygen /Users/timj/work/lsstsw/src/sconsUtils/doc/doxygen.conf
            

            The first line, at least, seems a bit out of place to me.

            Show
            tjenness Tim Jenness added a comment - Should -Q also be suppressing the doxygen lines: buildConfig(["doc/doxygen.conf"], ["doc/doxygen.conf.in"]) doxygen /Users/timj/work/lsstsw/src/sconsUtils/doc/doxygen.conf The first line, at least, seems a bit out of place to me.
            Hide
            tjenness Tim Jenness added a comment -

            Please review this trivial change. Suppressing other messages can be combined in this ticket if we need to do that.

            Show
            tjenness Tim Jenness added a comment - Please review this trivial change. Suppressing other messages can be combined in this ticket if we need to do that.
            Hide
            rhl Robert Lupton added a comment -

            Looks good to merge. The code's fine, and it works.

            Show
            rhl Robert Lupton added a comment - Looks good to merge. The code's fine, and it works.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                rhl Robert Lupton
                Reviewers:
                Robert Lupton
                Watchers:
                John Swinbank, Robert Lupton, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel