Skip to content

Commit

Permalink
fix(ci): Fix CI for Onivim 2 - revert fetch dependencies (#997)
Browse files Browse the repository at this point in the history
Unfortunately, I'm still seeing issues with the set of dependencies for the `fetch-lwt` library, when building Onivim 2 - we get this error:

I've been trying to avoid reverting this, and maintaining a separate 'oni2' branch w/o it - but it's a lot of extra overhead for me (we can't actually use `fetch` in Onivim 2, either - lack of Windows support & it isn't integrated with libuv).

When building from Onivim 2, I get these errors:
```
Bryans-Mac-mini:oni2 bryphe$ esy run -f
info building @reason-native-web/piaf@1.3.0@d41d8cd9
error: build failed with exit code: 1
  build log:
    # esy-build-package: building: @reason-native-web/piaf@1.3.0
    # esy-build-package: pwd: /Users/bryphe/.esy/source/i/reason_native_web__s__piaf__1.3.0__aac2317a
    # esy-build-package: running: 'dune' 'build' '--only-packages=piaf' '--profile=release' '-j' '4' '--root=./piaf'
    Entering directory '/Users/bryphe/.esy/source/i/reason_native_web__s__piaf__1.3.0__aac2317a/piaf'
    File "lib/http_intf.ml", line 1:
    Error: The files lib/.piaf.objs/byte/piaf__Scheme.cmi
           and /Users/bryphe/.esy/3__________________________________________________________________/i/reason_native_web__s__h2_lwt_unix-0.6.1000-eb9002c6/lib/h2-lwt-unix/h2_lwt_unix.cmi
           make inconsistent assumptions over interface Gluten_lwt_unix__Ssl_io
    error: command failed: 'dune' 'build' '--only-packages=piaf' '--profile=release' '-j' '4' '--root=./piaf' (exited with 1)
    esy-build-package: exiting with errors above...
    
  building @reason-native-web/piaf@1.3.0
esy: exiting due to errors above
```

I'd definitely like to have the `fetch` API - some details here: #996 , and really appreciate all the pioneering work you've done @lessp ! If you have any other ideas, @lessp - let me know 👍
  • Loading branch information
bryphe committed Sep 1, 2020
1 parent f5732e7 commit 11d4fb4
Show file tree
Hide file tree
Showing 238 changed files with 1,605 additions and 14,072 deletions.
1,116 changes: 174 additions & 942 deletions bench.esy.lock/index.json

Large diffs are not rendered by default.

32 changes: 0 additions & 32 deletions bench.esy.lock/opam/angstrom.0.14.1/opam

This file was deleted.

38 changes: 0 additions & 38 deletions bench.esy.lock/opam/base64.3.4.0/opam

This file was deleted.

23 changes: 0 additions & 23 deletions bench.esy.lock/opam/bigarray-compat.1.0.0/opam

This file was deleted.

43 changes: 0 additions & 43 deletions bench.esy.lock/opam/bigstringaf.0.6.1/opam

This file was deleted.

10 changes: 5 additions & 5 deletions bench.esy.lock/opam/charInfo_width.1.1.0/opam

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

30 changes: 0 additions & 30 deletions bench.esy.lock/opam/conf-autoconf.0.1/opam

This file was deleted.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

42 changes: 0 additions & 42 deletions bench.esy.lock/opam/dune-private-libs.2.6.2/opam

This file was deleted.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

24 changes: 0 additions & 24 deletions bench.esy.lock/opam/faraday-lwt-unix.0.7.1/opam

This file was deleted.

23 changes: 0 additions & 23 deletions bench.esy.lock/opam/faraday-lwt.0.7.1/opam

This file was deleted.

30 changes: 0 additions & 30 deletions bench.esy.lock/opam/faraday.0.7.1/opam

This file was deleted.

0 comments on commit 11d4fb4

Please sign in to comment.