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

add gfortran to OSX build deps for oorb build

    XMLWordPrintable

    Details

      Description

      gfortran is required to build oorb on osx, which is a dep of lsst_sims. Brew has removed the gfortran package and the front end is now provided by the brew gcc package/bottle.

      This is a sims deps but I believe gfortran is technically also a pipelines dep.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            lsst_sims wanted gfortran to be available on Jenkins so they could build oorb. That's now irrelevant given that gfortran is now part of the conda env. If no-one actually needs gfortran explicitly any more we can close the ticket. This is a low priority since it's an old ticket and gfortran is now there.

            Show
            tjenness Tim Jenness added a comment - lsst_sims wanted gfortran to be available on Jenkins so they could build oorb. That's now irrelevant given that gfortran is now part of the conda env. If no-one actually needs gfortran explicitly any more we can close the ticket. This is a low priority since it's an old ticket and gfortran is now there.
            Hide
            swinbank John Swinbank added a comment -

            But you say above that gfortran isn't listed in the bleed, and neither is oorb as far as I know... so why is gfortran available in the env? What's pulling it in?

            Anyway, I think it's not impossible we will want oorb/gfortran back at some point, but I don't think it matters: if neither _sims nor _distrib needs it now, I'm in favour of dropping it (assuming it isn't depended on by something else which we do need). We can always add it back when necessary.

            Show
            swinbank John Swinbank added a comment - But you say above that gfortran isn't listed in the bleed, and neither is oorb as far as I know... so why is gfortran available in the env? What's pulling it in? Anyway, I think it's not impossible we will want oorb/gfortran back at some point, but I don't think it matters: if neither _sims nor _distrib needs it now, I'm in favour of dropping it (assuming it isn't depended on by something else which we do need). We can always add it back when necessary.
            Hide
            tjenness Tim Jenness added a comment -

            I thought this ticket was asking for gfortran because sims wanted to build oorb, not that they wanted to put oorb in the env.

            I don't know what is pulling gfortran into the env. Sounds like we should just INVALID this ticket and forget about it. Not really worth this long debate.

            Show
            tjenness Tim Jenness added a comment - I thought this ticket was asking for gfortran because sims wanted to build oorb, not that they wanted to put oorb in the env. I don't know what is pulling gfortran into the env. Sounds like we should just INVALID this ticket and forget about it. Not really worth this long debate.
            Hide
            bvan Brian Van Klaveren added a comment -

            gfortran comes as part of the compilers package. It's not explicit, but neither is gcc/g++ in this case.

            Show
            bvan Brian Van Klaveren added a comment - gfortran comes as part of the compilers package. It's not explicit, but neither is gcc/g++ in this case.
            Hide
            swinbank John Swinbank added a comment -

            If we're getting gfortran because of compilers, then there's certainly no action here. Let's mark it invalid.

            Show
            swinbank John Swinbank added a comment - If we're getting gfortran because of compilers , then there's certainly no action here. Let's mark it invalid.

              People

              Assignee:
              bvan Brian Van Klaveren
              Reporter:
              jhoblitt Joshua Hoblitt
              Watchers:
              Brian Van Klaveren, Frossie Economou, John Parejko, John Swinbank, Jonathan Sick, Joshua Hoblitt, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: