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

Disable CC requirement for obs_base

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_base
    • Labels:
      None

      Description

      Remove the need for a C++ compiler as it's python-only.

        Attachments

          Activity

          Hide
          rowen Russell Owen added a comment -

          Looks good. I had no idea that option existed, but I build your branch and it worked and did not report looking for a C++ compiler.

          The pull request has not shown up here yet so I won't review on github unless you need me to do that. (I brought over the code and looked at it on my computer.)

          Show
          rowen Russell Owen added a comment - Looks good. I had no idea that option existed, but I build your branch and it worked and did not report looking for a C++ compiler. The pull request has not shown up here yet so I won't review on github unless you need me to do that. (I brought over the code and looked at it on my computer.)
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Thanks for looking at that. I didn't Jenkins it, but I did build it myself and it works for me too. I'll merge it in the morning.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Thanks for looking at that. I didn't Jenkins it, but I did build it myself and it works for me too. I'll merge it in the morning.
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          By which I mean now, as it's shown up here, so I'll just hit the button...

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - By which I mean now, as it's shown up here, so I'll just hit the button...
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Merged.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Merged.

            People

            Assignee:
            mfisherlevine Merlin Fisher-Levine
            Reporter:
            mfisherlevine Merlin Fisher-Levine
            Reviewers:
            Russell Owen
            Watchers:
            Merlin Fisher-Levine, Russell Owen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.