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

added some node packages #4734

Merged
merged 1 commit into from
Nov 2, 2014
Merged

added some node packages #4734

merged 1 commit into from
Nov 2, 2014

Conversation

flosse
Copy link
Contributor

@flosse flosse commented Oct 30, 2014

No description provided.

@edwtjo
Copy link
Member

edwtjo commented Oct 31, 2014

I'm not sure but you should probably squash the commits into one since the generated code isn't really interesting. Someone else with more intimate knowledge of the nodePackages should chime in before you make that change though.

@flosse
Copy link
Contributor Author

flosse commented Oct 31, 2014

I'm not sure but you you should probably squash the commits into one

done :)

@edwtjo
Copy link
Member

edwtjo commented Nov 2, 2014

Building with: for i in nib npm-check-updates node-stringprep ltx node-xmpp node-xmpp-core node-xmpp-client node-xmpp-component node-xmpp-server node-xmpp-serviceadmin node-xmpp-joap plc 7f jfs toobusy cordova sloc webpack;do nix-build -A nodePackages.$i;done

Getting:

npm ERR! notsup Unsupported
npm ERR! notsup Not compatible with your operating system or architecture: fsevents@0.2.1
npm ERR! notsup Valid OS:    darwin
npm ERR! notsup Valid Arch:  any
npm ERR! notsup Actual OS:   linux
npm ERR! notsup Actual Arch: x64

@flosse
Copy link
Contributor Author

flosse commented Nov 2, 2014

The old fsevents is required by an old chockidar thats required by watchpack :-\

@flosse
Copy link
Contributor Author

flosse commented Nov 2, 2014

I updated the commit (without webpack). Hopefully it works now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants