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

newinstall.sh depends on `which` -- an undocumented dependency

    Details

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

      Description

      Paul Price reported via slack yesterday that which is required to run newinstall.sh on the official docker.io/centos images. Currently, which is not a documented dependency. Additionally, the eups build requires which.

      Paul Price [8:09 PM]
      Just discovered that the `which` package is missing from the list of Centos prereqs (https://pipelines.lsst.io/install/prereqs/centos.html).
       
      (If I Docker `FROM centos:7`, I don't have `which`, but `newinstall.sh` uses it.) (edited)

        Attachments

          Issue Links

            Activity

            Hide
            jhoblitt Joshua Hoblitt added a comment -

            I have opened DM-14080 and DM-14081 against architecture for the repos Tim Jenness suggested should be deprecated.

            Per Kian-Tat Lim request, I have updated the dev guide PR to have a separate shellcheck example.

            Show
            jhoblitt Joshua Hoblitt added a comment - I have opened DM-14080 and DM-14081 against architecture for the repos Tim Jenness suggested should be deprecated. Per Kian-Tat Lim request, I have updated the dev guide PR to have a separate shellcheck example.
            Hide
            ktl Kian-Tat Lim added a comment -

            I approve the dev guide change.

            Show
            ktl Kian-Tat Lim added a comment - I approve the dev guide change.
            Hide
            ktl Kian-Tat Lim added a comment -

            Is this still waiting on something?  I'm removing myself from the reviewers list.

            Show
            ktl Kian-Tat Lim added a comment - Is this still waiting on something?  I'm removing myself from the reviewers list.
            Hide
            tjenness Tim Jenness added a comment -

            It seems to be the mariadb and dev guide PRs that are still open. Kian-Tat Lim I think if you approve the dev guide changes we can put the ticket in Reviewed state.

            Show
            tjenness Tim Jenness added a comment - It seems to be the mariadb and dev guide PRs that are still open. Kian-Tat Lim I think if you approve the dev guide changes we can put the ticket in Reviewed state.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            The dev guide PR have been retracted and no further action items remain on this ticket.

            Show
            jhoblitt Joshua Hoblitt added a comment - The dev guide PR have been retracted and no further action items remain on this ticket.

              People

              • Assignee:
                jhoblitt Joshua Hoblitt
                Reporter:
                jhoblitt Joshua Hoblitt
                Reviewers:
                Jonathan Sick
                Watchers:
                Gabriele Comoretto, Jonathan Sick, Joshua Hoblitt, Kian-Tat Lim, Paul Price, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel