Skip to content
This repository was archived by the owner on Aug 23, 2018. It is now read-only.

Commit c83832c

Browse files
author
Richard Feldman
authored
Simplify npm installer (#194)
* 💀 dead code * Make Appveyor and Travis test global install too. * Remove obsolete binwrapper system. * Update to node 4.0.0, upgrade tar and request, remove promise and mkdirp * 🎨 comment * Clean up .gitignore * 💀 dead code * Use --no-spin on npm CI builds * Use binstall * Fix Travis * Make Appveyor's build approach consistent with Travis's * Upgrade to binstall 1.1.0 * Upgrade to binstall 1.2.0
1 parent 56dce57 commit c83832c

File tree

19 files changed

+75
-203
lines changed

19 files changed

+75
-203
lines changed

.travis.yml

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ addons:
1515
env:
1616
matrix:
1717
- ELM_VERSION=master TARGET_NODE_VERSION=node
18-
- ELM_VERSION=master TARGET_NODE_VERSION=0.10
18+
- ELM_VERSION=master TARGET_NODE_VERSION=4.0.0
1919
- ELM_VERSION=0.16 TARGET_NODE_VERSION=node
20-
- ELM_VERSION=0.16 TARGET_NODE_VERSION=0.10
20+
- ELM_VERSION=0.16 TARGET_NODE_VERSION=4.0.0
2121

2222
global:
2323
- secure: "RY4A5Bj0tN7rscwPh6eDJUmf2qWW6lXPBQrD74vg3hB1ubloCRuoXvBD8evUSQg6Kb1D7Dqi3FNpZmt9In7Tiz3Xy6Vb4Q0pbh1r8rq97ZuFFbxzPVRL2QuyYuqQfvsfoal4NVxGyhfTaLh1tCC/G6CjCjGDRXFcqhcz+CR0cEI="
@@ -61,7 +61,10 @@ script:
6161
- du -hs binaries.tar.gz
6262
- curl -T binaries.tar.gz -uelmtravis:$BINTRAY_API_KEY https://api.bintray.com/content/elmlang/elm-platform/npm/$ELM_VERSION/$ELM_VERSION/$BINARY_OS_NAME-x64.tar.gz
6363
- cd ../..
64-
- npm install ./npm
64+
- mkdir tmp
65+
- cd tmp
66+
- npm init --yes
67+
- npm install ../npm --no-spin
6568
- node_modules/.bin/elm-make --help
6669
- node_modules/.bin/elm-repl --help
6770
- node_modules/.bin/elm-package install --help
@@ -70,3 +73,13 @@ script:
7073
- node_modules/.bin/elm repl --help
7174
- node_modules/.bin/elm package install --help
7275
- node_modules/.bin/elm reactor --help
76+
- npm remove elm
77+
- npm install -g ../npm --no-spin
78+
- elm-make --help
79+
- elm-repl --help
80+
- elm-package install --help
81+
- elm-reactor --help
82+
- elm make --help
83+
- elm repl --help
84+
- elm package install --help
85+
- elm reactor --help

appveyor.yml

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
environment:
22
matrix:
3-
- nodejs_version: "5.0"
4-
- nodejs_version: "0.10"
3+
- nodejs_version: "6.0"
4+
- nodejs_version: "4.0"
55

66
platform:
77
- x86
@@ -16,7 +16,10 @@ install:
1616
test_script:
1717
- node --version
1818
- npm --version
19-
- npm install installers\npm
19+
- mkdir tmp
20+
- cd tmp
21+
- npm init --yes
22+
- npm install ..\installers\npm --no-spin
2023
- node_modules\.bin\elm-make --help
2124
- node_modules\.bin\elm-repl --help
2225
- node_modules\.bin\elm-package install --help
@@ -25,5 +28,15 @@ test_script:
2528
- node_modules\.bin\elm repl --help
2629
- node_modules\.bin\elm package install --help
2730
- node_modules\.bin\elm reactor --help
31+
- npm remove elm
32+
- npm install -g ..\installers\npm --no-spin
33+
- elm-make --help
34+
- elm-repl --help
35+
- elm-package install --help
36+
- elm-reactor --help
37+
- elm make --help
38+
- elm repl --help
39+
- elm package install --help
40+
- elm reactor --help
2841

2942
build: off

installers/npm/.gitignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
BuildFromSource.hs
21
node_modules/
3-
Elm-Platform/

installers/npm/.npmignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
README.md
2+
.gitignore
3+
.git

installers/npm/binaries/elm

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
echo "The npm installation of Elm did not complete successfully! Please try reinstalling Elm through npm. If you still see this message even after reinstalling, please file a bug at https://github.com/elm-lang/elm-platform/issues to let us know. You can find manual installers at http://elm-lang.org/install"

installers/npm/binaries/elm-make

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
echo "The npm installation of Elm did not complete successfully! Please try reinstalling Elm through npm. If you still see this message even after reinstalling, please file a bug at https://github.com/elm-lang/elm-platform/issues to let us know. You can find manual installers at http://elm-lang.org/install"
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
echo "The npm installation of Elm did not complete successfully! Please try reinstalling Elm through npm. If you still see this message even after reinstalling, please file a bug at https://github.com/elm-lang/elm-platform/issues to let us know. You can find manual installers at http://elm-lang.org/install"
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
echo "The npm installation of Elm did not complete successfully! Please try reinstalling Elm through npm. If you still see this message even after reinstalling, please file a bug at https://github.com/elm-lang/elm-platform/issues to let us know. You can find manual installers at http://elm-lang.org/install"

installers/npm/binaries/elm-repl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
echo "The npm installation of Elm did not complete successfully! Please try reinstalling Elm through npm. If you still see this message even after reinstalling, please file a bug at https://github.com/elm-lang/elm-platform/issues to let us know. You can find manual installers at http://elm-lang.org/install"

installers/npm/binwrap.js

Lines changed: 0 additions & 12 deletions
This file was deleted.

0 commit comments

Comments
 (0)