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

lsstsw breakage with spaces in paths

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Team:
      Architecture

      Description

      There are still some issues relating to using lsstsw to build the stack when spaces are in the path to the $LSSTSW location. This is a fine thing to sort out on Rodeo Day...

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          tjenness Tim Jenness added a comment - - edited

          Unbelievable:

          ::: Deploying Miniconda 3.19.0 for MacOSX-x86_64
          		 
          ######################################################################## 100.0%
          		 
          ERROR: Cannot install into directories with spaces

          which is an Anaconda error and nothing to do with LSSTSW.

          Show
          tjenness Tim Jenness added a comment - - edited Unbelievable: ::: Deploying Miniconda 3.19.0 for MacOSX-x86_64 ######################################################################## 100.0% ERROR: Cannot install into directories with spaces which is an Anaconda error and nothing to do with LSSTSW.
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Also, the numdiff install target is completely broken (both the Makefile and the shtool installer).

          Show
          tjenness Tim Jenness added a comment - Also, the numdiff install target is completely broken (both the Makefile and the shtool installer).
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          EUPS fixes can be found at https://github.com/RobertLuptonTheGood/eups/pull/95

          Show
          tjenness Tim Jenness added a comment - EUPS fixes can be found at https://github.com/RobertLuptonTheGood/eups/pull/95
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          I have made a patch to lsst_build to enable the generated _build.sh to run with spaces in PATH. With that change the basic build system now works (ignoring miniconda and numdiff which have to be worked around manually) and the focus shifts to individual packages. cfitsio fails immediately because its eupspkg.cfg.sh does not quote the prefix arguments. I think there may be issues with the cfitsio Makefile.

          Show
          tjenness Tim Jenness added a comment - I have made a patch to lsst_build to enable the generated _build.sh to run with spaces in PATH. With that change the basic build system now works (ignoring miniconda and numdiff which have to be worked around manually) and the focus shifts to individual packages. cfitsio fails immediately because its eupspkg.cfg.sh does not quote the prefix arguments. I think there may be issues with the cfitsio Makefile .
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Mainly quoting. Minor patch to sconsUtils to fix double quoting.

          Show
          tjenness Tim Jenness added a comment - Mainly quoting. Minor patch to sconsUtils to fix double quoting.
          Hide
          Permalink
          jhoblitt Joshua Hoblitt added a comment -

          All 3 PRs looked good and have been merged.

          Show
          jhoblitt Joshua Hoblitt added a comment - All 3 PRs looked good and have been merged.

            People

            Assignee:
            tjenness Tim Jenness
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            Joshua Hoblitt
            Watchers:
            Joshua Hoblitt, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.