Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot generate electron artifacts with the new CLI tool #559

Closed
kittaakos opened this issue Sep 25, 2017 · 5 comments
Closed

Cannot generate electron artifacts with the new CLI tool #559

kittaakos opened this issue Sep 25, 2017 · 5 comments
Assignees
Labels
bug bugs found in the application extension system issues related to the extension system

Comments

@kittaakos
Copy link
Contributor

Maybe it is not an issue at all, but something we need to add to the documentation. I have tried to install and use the new CLI with the following, but it failed:

Steps:

nvm use 8 && node --version && npm --version \
&& mkdir theia-electron-app \
&& cd theia-electron-app \
&& npm init -y \
&& npm i -S @theia/cli@next @theia/core@next @theia/editor@next @theia/filesystem@next @theia/monaco@next @theia/navigator@next @theia/preferences@next @theia/preferences-api@next @theia/process@next @theia/terminal@next @theia/workspace@next \
&& ./node_modules/.bin/theia generate --target=electron

Error:

failed to find @theia/cli
failed to find @theia/preferences-api

Full console output:

Akoss-MBP:foo akos.kitta$ nvm use 8 && node --version && npm --version \
> && mkdir theia-electron-app \
> && cd theia-electron-app \
> && npm init -y \
> && npm i -S @theia/cli@next @theia/core@next @theia/editor@next @theia/filesystem@next @theia/monaco@next @theia/navigator@next @theia/preferences@next @theia/preferences-api@next @theia/process@next @theia/terminal@next @theia/workspace@next \
> && ./node_modules/.bin/theia generate --target=electron
Now using node v8.5.0 (npm v5.3.0)
v8.5.0
5.3.0
Wrote to /Users/akos.kitta/Desktop/foo/theia-electron-app/package.json:

{
  "name": "theia-electron-app",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "typefox <npm@typefox.io>",
  "license": "ISC"
}



> dtrace-provider@0.8.5 install /Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/dtrace-provider
> node scripts/install.js


> fsevents@1.1.2 install /Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/fsevents
> node install

[fsevents] Success: "/Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/fsevents/lib/binding/Release/node-v57-darwin-x64/fse.node" already installed
Pass --update-binary to reinstall or --build-from-source to recompile

> node-pty@0.7.0 install /Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/node-pty
> node scripts/install.js

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

  CXX(target) Release/obj.target/pty/src/unix/pty.o
  SOLINK_MODULE(target) Release/pty.node

> electron@1.7.6 postinstall /Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/electron
> node install.js


> node-pty@0.7.0 postinstall /Users/akos.kitta/Desktop/foo/theia-electron-app/node_modules/node-pty
> node scripts/post-install.js

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN theia-electron-app@1.0.0 No description
npm WARN theia-electron-app@1.0.0 No repository field.

+ @theia/preferences-api@0.2.0-next.61c95132
+ @theia/cli@0.2.0-next.61c95132
+ @theia/core@0.2.0-next.61c95132
+ @theia/filesystem@0.2.0-next.61c95132
+ @theia/preferences@0.2.0-next.61c95132
+ @theia/editor@0.2.0-next.61c95132
+ @theia/monaco@0.2.0-next.61c95132
+ @theia/navigator@0.2.0-next.61c95132
+ @theia/process@0.2.0-next.61c95132
+ @theia/terminal@0.2.0-next.61c95132
+ @theia/workspace@0.2.0-next.61c95132
added 845 packages in 58.549s
failed to find @theia/cli
failed to find @theia/preferences-api
Akoss-MBP:theia-electron-app akos.kitta$ 

@kittaakos
Copy link
Contributor Author

Same with:

Now using node v8.0.0 (npm v5.0.3)
v8.0.0
5.0.3

@akosyakov akosyakov added the bug bugs found in the application label Sep 25, 2017
@akosyakov
Copy link
Member

akosyakov commented Sep 25, 2017

it is the issue, not all dependencies are extensions.

@akosyakov akosyakov added the extension system issues related to the extension system label Sep 25, 2017
@akosyakov akosyakov self-assigned this Sep 25, 2017
@akosyakov
Copy link
Member

Will be fixed as a part of #391

@kittaakos
Copy link
Contributor Author

One can ignore the error message.

@akosyakov
Copy link
Member

Should be fixed with #391

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application extension system issues related to the extension system
Projects
None yet
Development

No branches or pull requests

2 participants