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

sconsUtils cannot install doc directories that do not contain a config file II

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: sconsUtils
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      Attempting a build following the merging of DM-35792 causes astshim to crash with the following:

      2022-08-01T22:18:07.544783Z] TypeError: stat: path should be string, bytes, os.PathLike or integer, not NodeList:
      		 
      [2022-08-01T22:18:07.544978Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/build/astshim/SConstruct", line 3:
      		 
      [2022-08-01T22:18:07.545058Z]     scripts.BasicSConstruct("astshim")
      		 
      [2022-08-01T22:18:07.545379Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/stack/lsst-scipipe-4.1.0/DarwinX86/sconsUtils/g5b9212a469/python/lsst/sconsUtils/scripts.py", line 51:
      		 
      [2022-08-01T22:18:07.545555Z]     cls.initialize(packageName, versionString, eupsProduct, eupsProductPath, cleanExt,
      		 
      [2022-08-01T22:18:07.545877Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/stack/lsst-scipipe-4.1.0/DarwinX86/sconsUtils/g5b9212a469/python/lsst/sconsUtils/scripts.py", line 153:
      		 
      [2022-08-01T22:18:07.545925Z]     SConscript(script)
      		 
      [2022-08-01T22:18:07.546243Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/miniconda/envs/lsst-scipipe-4.1.0/lib/python3.10/site-packages/SCons/Script/SConscript.py", line 660:
      		 
      [2022-08-01T22:18:07.546307Z]     return method(*args, **kw)
      		 
      [2022-08-01T22:18:07.546625Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/miniconda/envs/lsst-scipipe-4.1.0/lib/python3.10/site-packages/SCons/Script/SConscript.py", line 597:
      		 
      [2022-08-01T22:18:07.546731Z]     return _SConscript(self.fs, *files, **subst_kw)
      		 
      [2022-08-01T22:18:07.547048Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/miniconda/envs/lsst-scipipe-4.1.0/lib/python3.10/site-packages/SCons/Script/SConscript.py", line 285:
      		 
      [2022-08-01T22:18:07.547197Z]     exec(compile(scriptdata, scriptname, 'exec'), call_stack[-1].globals)
      		 
      [2022-08-01T22:18:07.547389Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/build/astshim/doc/SConscript", line 3:
      		 
      [2022-08-01T22:18:07.547459Z]     scripts.BasicSConscript.doc()
      		 
      [2022-08-01T22:18:07.547781Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/stack/lsst-scipipe-4.1.0/DarwinX86/sconsUtils/g5b9212a469/python/lsst/sconsUtils/scripts.py", line 509:
      		 
      [2022-08-01T22:18:07.547846Z]     result = state.env.Doxygen(
      		 
      [2022-08-01T22:18:07.548170Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/stack/lsst-scipipe-4.1.0/DarwinX86/sconsUtils/g5b9212a469/python/lsst/sconsUtils/builders.py", line 542:
      		 
      [2022-08-01T22:18:07.548238Z]     return builder(self, config)
      		 
      [2022-08-01T22:18:07.548560Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/stack/lsst-scipipe-4.1.0/DarwinX86/sconsUtils/g5b9212a469/python/lsst/sconsUtils/builders.py", line 342:
      		 
      [2022-08-01T22:18:07.548624Z]     if os.path.exists(config):
      		 
      [2022-08-01T22:18:07.548888Z]   File "/Users/square/j/ws/stack-os-matrix/macos/lsstsw/miniconda/envs/lsst-scipipe-4.1.0/lib/python3.10/genericpath.py", line 19:
      		 
      [2022-08-01T22:18:07.548925Z]     os.stat(path)

      Fix and confirm that the Stack builds with the change.

        Attachments

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-35792 sconsUtils cannot install doc directories that do not contain a config file

          • Done

          Improvement - An improvement or enhancement to an existing feature or task. DM-22823 Remove accidental Doxygen comments for namespace lsst

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            I ended up reverting DM-35792; this issue's new scope is to find another fix for the problem originally described on that issue.

            Show
            krzys Krzysztof Findeisen added a comment - I ended up reverting DM-35792 ; this issue's new scope is to find another fix for the problem originally described on that issue.
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment - - edited

            After looking at this some more, I think it will take someone more familiar with sconsUtils than me, so I'm un-assigning myself. Note that I've already pushed one commit, which resolves the shadowing issue that confused the response to DM-35792.

            Show
            krzys Krzysztof Findeisen added a comment - - edited After looking at this some more, I think it will take someone more familiar with sconsUtils than me, so I'm un-assigning myself. Note that I've already pushed one commit, which resolves the shadowing issue that confused the response to DM-35792 .

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              krzys Krzysztof Findeisen
              Watchers:
              Ian Sullivan, Kian-Tat Lim, Krzysztof Findeisen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.