-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
make tm #137
Comments
hmm, must not have any user bundles or something, try |
I was fine solving it, was just reporting case you can think of how to fix the script :)
do have bundles here:
|
And:
|
This fix doesn't work and in fact breaks it in all cases now ;) I just ran make tm and it created the directory fine then copied the bundle into that directory. Unfortunately as the directory already existed the budle is now an extra level deep and is not detected by textmate. mkdir -p $(TM_DEST) you could probably then change the next line to: cp -fr editors/$(TM_BUNDLE) $(TM_DEST) but it will work either way I think |
haha my bad, yeah you're right i didn't test that very well, thanks |
ah, cool - i'm learning a bit about github and wasn't sure you would see sorry for the overkill :o On 18 July 2012 21:32, TJ Holowaychuk <
|
no worries! thanks for pointing it out |
I'm still experiencing the issue where it copies the bundle two directories deep. I installed Mocha via NPM with the -g (global) flag. I used the solution from #483; cd into the global directory, then run make. I see the bundle, but I had to copy the nested bundle into the parent (Bundles/) directory. |
@zedd45 I haven't released a new version yet, but yes you should clone the repo etc and do |
it shouldn't even be in npm ill remove it |
Clone -> make tm works perfectly. Thanks! |
I have mocha.tmbundle: No such file or directory error and there is not nested folder in Library/Application Support/Textmate/Bundles. Any ideas? UPDATE: npm didn't include editors folder, weird... I wonder what else is missing? |
@azat-co it's in .npmignore, just clone the repo |
@visionmedia, yes I figured out to grab it from the repo. thanks! :) |
The text was updated successfully, but these errors were encountered: