Skip to content

Commit

Permalink
[PR] Fix appveyor opam2 build
Browse files Browse the repository at this point in the history
Summary:
the appveyor setup script automatically pins `.` and installs its deps, so we don't need to do it again. further, doing it ourselves was failing because it seemed to think that `.` was a git repo instead of a directory:

```
[00:14:56] Package flowtype-ci does not exist, create as a NEW package? [Y/n] y
[00:14:56] [flowtype-ci.~dev: git]
[00:14:56] [flowtype-ci.~dev: git]
[00:14:56] [flowtype-ci.~dev: git]
[00:14:56] [flowtype-ci.~dev: git]
[00:14:56] [flowtype-ci.~dev: git]
[00:14:56] [flowtype-ci.~dev: git]
[00:14:57] [flowtype-ci.~dev: git]
[00:14:57] [ERROR] Could not synchronize C:/cygwin64/home/appveyor/.opam/ocaml-variants.4.05.0+mingw64c/.opam-switch/sources/flowtype-ci from "git+file://C:/projects/flow#HEAD":
[00:14:57]         "C:\\cygwin64\\bin\\git.exe fetch -q" exited with code 128
[00:14:57] [ERROR] Error getting source from git+file://C:/projects/flow#HEAD:
[00:14:57]           - git+file://C:/projects/flow#HEAD
```
Pull Request resolved: #7823

Differential Revision: D15781689

Pulled By: mroch

fbshipit-source-id: 1f214e34df4536700b90fed64368f4f41096d432
  • Loading branch information
mroch authored and facebook-github-bot committed Jun 12, 2019
1 parent 44f313b commit 630876d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
1 change: 1 addition & 0 deletions appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ environment:
FORK_USER: ocaml
FORK_BRANCH: master
CYG_ROOT: C:\cygwin64
PACKAGE: flowtype-ci
TESTS: false
INSTALL: false
DEPOPTS: false
Expand Down
5 changes: 1 addition & 4 deletions resources/appveyor/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,11 @@ pwd
# print opam config
opam config list

eval $(opam config env)
eval $(opam env)

# print ocaml config
ocamlc -config

cd "${APPVEYOR_BUILD_FOLDER}"
opam pin add flowtype-ci . -n
opam depext -u flowtype-ci
opam install flowtype-ci --deps-only
make all
make -C src/parser/ ../../_build/src/parser/test/run_tests.native

0 comments on commit 630876d

Please sign in to comment.