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

Fix pathing issues for coffee #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jacebennett
Copy link

Fix for #3. Changed all npm executable references to use npm bin to resolve the correct path.

@jacebennett
Copy link
Author

Doubt this will work in a windows environment. Need to learn more about rake so I can do it in a more cross-platform fashion.

@@ -4,8 +4,8 @@ task :default => [:build]

desc 'Create js files from coffee sources'
task :build do
system("cd #{root} && bundle exec coffee --compile --output . src/isolate.coffee")
system("cd #{root} && bundle exec coffee --compile spec")
system("cd #{root} && bundle exec `npm bin`/coffee --compile --output . src/isolate.coffee")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since we're already cd-ing into the directory of the Rakefile, can't we just directly path to node_modules/.bin/coffee ?
though I do like the use of npm bin. I'll check it in my windows environment now.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah, didn't see that.

Sent from my iPhone

On Mar 22, 2013, at 8:30 PM, James Maroney notifications@github.com wrote:

In Rakefile:

@@ -4,8 +4,8 @@ task :default => [:build]

desc 'Create js files from coffee sources'
task :build do

  • system("cd #{root} && bundle exec coffee --compile --output . src/isolate.coffee")
  • system("cd #{root} && bundle exec coffee --compile spec")
  • system("cd #{root} && bundle exec npm bin/coffee --compile --output . src/isolate.coffee")
    since we're already cd-ing into the directory of the Rakefile, can't we just directly path to node_modules/.bin/coffee ?


Reply to this email directly or view it on GitHub.

Here are some updates that make the commands seem to execute cross-platform.
Sadly, they still don't behave correctly in Windows, so this
certainly isn't 100% right.
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