Skip to content
This repository has been archived by the owner on Feb 13, 2019. It is now read-only.

local happyplan bin is fucked up #50

Closed
Wolfr opened this issue May 26, 2013 · 10 comments
Closed

local happyplan bin is fucked up #50

Wolfr opened this issue May 26, 2013 · 10 comments

Comments

@Wolfr
Copy link

Wolfr commented May 26, 2013

I wanted to check out the themes functionaliy but it's a bit unclear, the docs on github homepage refer to a "happy" command now so I went on to install the binary in the repo (bin/happy)

  • Should add to docs how to install the happy bin
  • Should make happy installable via brew?

If i move the bin to my /usr/local/bin I have problems

wolfr3:happy-plan noone$ cp bin/happy /usr/local/bin/
wolfr3:happy-plan noone$ happy
-bash: /usr/local/bin/happy: Permission denied
wolfr3:happy-plan noone$ chmod +x /usr/local/bin/happy 
wolfr3:happy-plan noone$ happy

module.js:340
    throw err;
          ^
Error: Cannot find module 'celeri'
    at Function.Module._resolveFilename (module.js:338:15)
    (...)

is this about https://github.com/crcn/celeri ?

@MoOx
Copy link
Member

MoOx commented May 26, 2013

It seems you didnt read enough the README https://github.com/kud/happy-plan#requirements

But I'll advise you to not use latest unstable feature that are subject to lots of changes since we are going to deliver hp as npm package.

On 26 mai 2013, at 11:22, Johan Ronsse notifications@github.com wrote:

I wanted to check out the themes functionaliy but it's a bit unclear, the docs on github homepage refer to a "happy" command now so I went on to install the binary in the repo (bin/happy)

Should add to docs how to install the happy bin
Should make happy installable via brew?
If i move the bin to my /usr/local/bin I have problems

wolfr3:happy-plan noone$ cp bin/happy /usr/local/bin/
wolfr3:happy-plan noone$ happy
-bash: /usr/local/bin/happy: Permission denied
wolfr3:happy-plan noone$ chmod +x /usr/local/bin/happy
wolfr3:happy-plan noone$ happy

module.js:340
throw err;
^
Error: Cannot find module 'celeri'
at Function.Module._resolveFilename (module.js:338:15)
(...)
is this about https://github.com/crcn/celeri ?


Reply to this email directly or view it on GitHub.

@kud
Copy link
Contributor

kud commented May 26, 2013

@Wolfr Do a npm update or npm install in the root project, it will install celeri ;) Plus, do not move the bin:

Either you start with bin/happy when you're in the root project, either you add ./bin in your PATH.

@Wolfr
Copy link
Author

Wolfr commented May 27, 2013

npm install gave this error https://gist.github.com/Wolfr/5656897
Just reporting

@kud
Copy link
Contributor

kud commented May 27, 2013

@MoOx
Copy link
Member

MoOx commented May 28, 2013

brew install pixman cairo ask me to install xquartz oO. Seems weird to have to install that just for celeri.
Maybe we should go back to commander & the way other do (command subcommand lauch command-subcommand )

@MoOx
Copy link
Member

MoOx commented May 29, 2013

@kud Maybe we should just get rid of the local bin happyplan in favor of the Gruntfile. It seems grunt-cli is lauching directly the fist local Gruntfile it found & do require(Gruntfile).cli(). No bin here

https://github.com/gruntjs/grunt-cli/blob/master/bin/grunt#L41
https://github.com/gruntjs/grunt/blob/master/lib/grunt.js#L35

We should just use Grunt API to launch command instead of celeri or commander.

We need to go forward with this so I can move on for the #41

@MoOx
Copy link
Member

MoOx commented May 30, 2013

Goto #51

@MoOx MoOx closed this as completed May 30, 2013
@MoOx
Copy link
Member

MoOx commented May 30, 2013

@Wolfr @kud https://npmjs.org/package/happyplan-cli will launch spawn grunt.
We won't use grunt directly because we need a specific order in task. happyplan run grunt under the hood but an happyplan project won't require anywmore a Gruntfile.js. More infos soon guys.

@kud
Copy link
Contributor

kud commented May 30, 2013

Sounds good for me!

@Wolfr
Copy link
Author

Wolfr commented May 30, 2013

Alright!

On Thu, May 30, 2013 at 9:32 AM, Erwann Mest notifications@github.comwrote:

Sounds good for me!


Reply to this email directly or view it on GitHubhttps://github.com//issues/50#issuecomment-18665634
.

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

No branches or pull requests

3 participants