Build using OSX on travis

[jgraham]

This change is

[jgraham]

r? @andreastt

[andreastt]

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

---

a discussion (no related file):
Looks like you accidentally changed all the indentation from tabs to spaces in ci.sh.

---

.travis.yml, line 58 [r1] (raw file):

    - "geckodriver-$TRAVIS_TAG-arm7hf.tar.gz"
    - "geckodriver-$TRAVIS_TAG-linux64.tar.gz"
    - "geckodriver-$TRAVIS-osx.tar.gz"

Missing the _TAG from $TRAVIS_TAG.

---

ci.sh, line 101 [r1] (raw file):

        ;;
    x86_64-apple-darwin)
        target=osx

Apple changed the official name of Mac OS X to “macOS” a few weeks ago, so we should use that since X is being phased out.

---

Comments from Reviewable

[andreastt]

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

---

Comments from Reviewable

[andreastt]

Review status: all files reviewed at latest revision, all discussions resolved.

---

a discussion (no related file):

Previously, andreastt (Andreas Tolfsen) wrote…

Looks like you accidentally changed all the indentation from tabs to spaces in ci.sh.

Okay, I’m going to accept this under in the hope that we will soon get rid of (most of) this file.

---

Comments from Reviewable

[andreastt]

@jgraham LGTM, but please squash first.