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

sconsUtils has a hard dependency on EUPS for both tests and installation

    XMLWordPrintable

    Details

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

      Description

      After some discussion on Data Management, its clear that sconsUtils is a hard requirement on EUPS for both tests and installation. It was decided by RFC-44 that tests should not depend on EUPS. However, I'd argue that sconsUtils should also not depend on EUPS as any package that uses sconsUtils (the virtual entirety of the stack) can not build or run tests without the presence of EUPS. The current situation is that the complete stack has a hard dependency on EUPS.

      Attempting to build sconUtils without the presence of EUPS. The tests fail attempting to import the eups module.

      $ SCONSUTILS_DIR=. scons -Q
      Unable to import eups; guessing flavor
      CC is gcc version 4.8.3
      Checking for C++11 support
      C++11 supported with '-std=c++11'
      Unable to import eups; guessing flavor
      Doxygen is not setup; skipping documentation build.
      ImportError: No module named eups:
        File "/home/vagrant/sconsUtils/SConstruct", line 9:
          scripts.BasicSConstruct.initialize(packageName="sconsUtils")
        File "python/lsst/sconsUtils/scripts.py", line 106:
          SCons.Script.SConscript(os.path.join(root, "SConscript"))
        File "/usr/lib/scons/SCons/Script/SConscript.py", line 609:
          return method(*args, **kw)
        File "/usr/lib/scons/SCons/Script/SConscript.py", line 546:
          return _SConscript(self.fs, *files, **subst_kw)
        File "/usr/lib/scons/SCons/Script/SConscript.py", line 260:
          exec _file_ in call_stack[-1].globals
        File "/home/vagrant/sconsUtils/tests/SConscript", line 5:
          import eups
      

      Attempting to bypass the test failures:

      [vagrant@jenkins-el7-1 sconsUtils]$ rm -rf tests
      [vagrant@jenkins-el7-1 sconsUtils]$ SCONSUTILS_DIR=. scons -Q install
      Unable to import eups; guessing flavor
      CC is gcc version 4.8.3
      Checking for C++11 support
      C++11 supported with '-std=c++11'
      Error with git version: uncommitted changes
      Found problem with version number; update or specify force=True to proceed
      

        Attachments

          Issue Links

            Activity

            jhoblitt Joshua Hoblitt created issue -
            jhoblitt Joshua Hoblitt made changes -
            Field Original Value New Value
            Link This issue blocks RFC-44 [ RFC-44 ]
            jhoblitt Joshua Hoblitt made changes -
            Assignee Joshua Hoblitt [ jhoblitt ] Jim Bosch [ jbosch ]
            jhoblitt Joshua Hoblitt made changes -
            Link This issue blocks DM-2768 [ DM-2768 ]
            jhoblitt Joshua Hoblitt made changes -
            Assignee Jim Bosch [ jbosch ] Frossie Economou [ frossie ]
            tjenness Tim Jenness made changes -
            Team SQuaRE [ 10302 ]
            jhoblitt Joshua Hoblitt made changes -
            Description After some discussion on Data Management, its clear that sconsUtils is a hard requirement on EUPS for both tests and installation. It was decided by RFC-44 that tests should not depend on EUPS. However, I'd argue that sconsUtils should also not depend on EUPS as any package that uses sconsUtils (the virtual entirety of the stack) can not build or run tests with the presence of EUPS. The current situation is that the complete stack has a hard dependency on EUPS.

            Attempting to build sconUtils without the presence of EUPS. The tests fail attempting to import the eups module.
            {code}
            $ SCONSUTILS_DIR=. scons -Q
            Unable to import eups; guessing flavor
            CC is gcc version 4.8.3
            Checking for C++11 support
            C++11 supported with '-std=c++11'
            Unable to import eups; guessing flavor
            Doxygen is not setup; skipping documentation build.
            ImportError: No module named eups:
              File "/home/vagrant/sconsUtils/SConstruct", line 9:
                scripts.BasicSConstruct.initialize(packageName="sconsUtils")
              File "python/lsst/sconsUtils/scripts.py", line 106:
                SCons.Script.SConscript(os.path.join(root, "SConscript"))
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 609:
                return method(*args, **kw)
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 546:
                return _SConscript(self.fs, *files, **subst_kw)
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 260:
                exec _file_ in call_stack[-1].globals
              File "/home/vagrant/sconsUtils/tests/SConscript", line 5:
                import eups
            {code}

            Attempting to bypass the test failures:
            {code}
            [vagrant@jenkins-el7-1 sconsUtils]$ rm -rf tests
            [vagrant@jenkins-el7-1 sconsUtils]$ SCONSUTILS_DIR=. scons -Q install
            Unable to import eups; guessing flavor
            CC is gcc version 4.8.3
            Checking for C++11 support
            C++11 supported with '-std=c++11'
            Error with git version: uncommitted changes
            Found problem with version number; update or specify force=True to proceed
            {code}
            After some discussion on Data Management, its clear that sconsUtils is a hard requirement on EUPS for both tests and installation. It was decided by RFC-44 that tests should not depend on EUPS. However, I'd argue that sconsUtils should also not depend on EUPS as any package that uses sconsUtils (the virtual entirety of the stack) can not build or run tests without the presence of EUPS. The current situation is that the complete stack has a hard dependency on EUPS.

            Attempting to build sconUtils without the presence of EUPS. The tests fail attempting to import the eups module.
            {code}
            $ SCONSUTILS_DIR=. scons -Q
            Unable to import eups; guessing flavor
            CC is gcc version 4.8.3
            Checking for C++11 support
            C++11 supported with '-std=c++11'
            Unable to import eups; guessing flavor
            Doxygen is not setup; skipping documentation build.
            ImportError: No module named eups:
              File "/home/vagrant/sconsUtils/SConstruct", line 9:
                scripts.BasicSConstruct.initialize(packageName="sconsUtils")
              File "python/lsst/sconsUtils/scripts.py", line 106:
                SCons.Script.SConscript(os.path.join(root, "SConscript"))
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 609:
                return method(*args, **kw)
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 546:
                return _SConscript(self.fs, *files, **subst_kw)
              File "/usr/lib/scons/SCons/Script/SConscript.py", line 260:
                exec _file_ in call_stack[-1].globals
              File "/home/vagrant/sconsUtils/tests/SConscript", line 5:
                import eups
            {code}

            Attempting to bypass the test failures:
            {code}
            [vagrant@jenkins-el7-1 sconsUtils]$ rm -rf tests
            [vagrant@jenkins-el7-1 sconsUtils]$ SCONSUTILS_DIR=. scons -Q install
            Unable to import eups; guessing flavor
            CC is gcc version 4.8.3
            Checking for C++11 support
            C++11 supported with '-std=c++11'
            Error with git version: uncommitted changes
            Found problem with version number; update or specify force=True to proceed
            {code}
            jhoblitt Joshua Hoblitt made changes -
            Assignee Frossie Economou [ frossie ] Joshua Hoblitt [ jhoblitt ]
            tjenness Tim Jenness made changes -
            Link This issue relates to RFC-69 [ RFC-69 ]
            jhoblitt Joshua Hoblitt made changes -
            Reviewers Jim Bosch, Kian-Tat Lim, Paul Price, Tim Jenness [ jbosch, ktl, price, tjenness ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            jhoblitt Joshua Hoblitt made changes -
            Link This issue relates to DM-3208 [ DM-3208 ]
            jhoblitt Joshua Hoblitt made changes -
            Epic Link DM-1141 [ 13950 ]
            jhoblitt Joshua Hoblitt made changes -
            Story Points 6
            jhoblitt Joshua Hoblitt made changes -
            Resolution Done [ 10000 ]
            Status In Review [ 10004 ] Done [ 10002 ]
            frossie Frossie Economou made changes -
            Epic Link DM-1141 [ 13950 ] DM-3425 [ 19388 ]
            tjenness Tim Jenness made changes -
            Link This issue is duplicated by DM-1164 [ DM-1164 ]
            jmatt J Matt Peterson [X] (Inactive) made changes -
            Assignee Joshua Hoblitt [ jhoblitt ] J Matt Peterson [ jmatt ]
            jmatt J Matt Peterson [X] (Inactive) made changes -
            Assignee J Matt Peterson [ jmatt ] Joshua Hoblitt [ jhoblitt ]
            swinbank John Swinbank made changes -
            Summary sconsUtil has a hard dependency on EUPS for both tests and installation sconsUtils has a hard dependency on EUPS for both tests and installation
            tjenness Tim Jenness made changes -
            Link This issue is triggered by RFC-69 [ RFC-69 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to RFC-69 [ RFC-69 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggering DM-3655 [ DM-3655 ]

              People

              Assignee:
              jhoblitt Joshua Hoblitt
              Reporter:
              jhoblitt Joshua Hoblitt
              Reviewers:
              Jim Bosch, Kian-Tat Lim, Paul Price, Tim Jenness
              Watchers:
              Frossie Economou, Jim Bosch, J Matt Peterson [X] (Inactive), Joshua Hoblitt, Kian-Tat Lim, Paul Price, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.