onivim / oni

Oni: Modern Modal Editing - powered by Neovim
https://www.onivim.io
MIT License
11.35k stars 299 forks source link

Can't build on OSX sierra #143

Closed metalelf0 closed 7 years ago

metalelf0 commented 7 years ago

Hi, I'm trying to build master on osx 10.12.2 following the build instructions in the README file and it's failing with these errors:

metalelf0:oni/ (master) $ npm run build                                                                                                                                                                                                                              [10:04:25]

> oni-vim@0.1.6 build /Users/metalelf0/sources/oni
> npm run build:browser && npm run build:plugin_api && npm run build:plugins

> oni-vim@0.1.6 build:browser /Users/metalelf0/sources/oni
> webpack --config browser/webpack.production.config.js

ts-loader: Using typescript@2.1.4 and /Users/metalelf0/sources/oni/browser/tsconfig.src.json
Hash: 7ae28ed3d5bc222dd09e
Version: webpack 1.14.0
Time: 5142ms
    Asset     Size  Chunks             Chunk Names
bundle.js  1.79 MB       0  [emitted]  main
   [0] multi main 28 bytes {0} [built]
    + 322 hidden modules

ERROR in ./browser/src/Plugins/PluginManager.ts
(284,31): error TS2345: Argument of type 'string | Buffer' is not assignable to parameter of type 'string'.
  Type 'Buffer' is not assignable to type 'string'.

ERROR in ./browser/src/Plugins/PluginManager.ts
Module build failed: Error: Typescript emitted no output for /Users/metalelf0/sources/oni/browser/src/Plugins/PluginManager.ts.
    at Object.loader (/Users/metalelf0/sources/oni/node_modules/ts-loader/dist/index.js:33:15)
 @ ./browser/src/index.tsx 10:24-58

> oni-vim@0.1.6 build:plugin_api /Users/metalelf0/sources/oni
> cd plugin_api && tsc -p tsconfig.src.json

> oni-vim@0.1.6 build:plugins /Users/metalelf0/sources/oni
> cd vim/core/oni-plugin-typescript && npm run build

> oni-plugin-typescript@0.0.1 build /Users/metalelf0/sources/oni/vim/core/oni-plugin-typescript
> npm install && tsc

npm WARN oni-plugin-typescript@0.0.1 No description
npm WARN oni-plugin-typescript@0.0.1 No repository field.
npm WARN oni-plugin-typescript@0.0.1 No license field.

I've also tried building v0.1.6 and v0.1.6.1, and I still get the same error. Obviously I have no local changes. I'm using node 7.3.0 and npm 3.10.10.

If you need further info please ask. Thanks for your time and keep up the great work!

bryphe commented 7 years ago

Thanks for the detailed report, @metalelf0

Looks like the latest version of TypeScript found an issue that previous versions didn't :) I have this fixed in PR #144

metalelf0 commented 7 years ago

Thanks a lot, it builds correctly now ;) Great work!