Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

npm link shared mojits failed on 0.7.2 #1219

Closed
ftliou opened this issue Aug 14, 2013 · 2 comments
Closed

npm link shared mojits failed on 0.7.2 #1219

ftliou opened this issue Aug 14, 2013 · 2 comments

Comments

@ftliou
Copy link

ftliou commented Aug 14, 2013

Please refer to:
http://developer.yahoo.com/forum/Yahoo-Mojito/npm-link-shared-mojits-failed-on/1376445752796-c287e870-fd51-473e-ac9a-557335d087a3

In mojito/lib/app/autoload/store.server.js

       if (!this._libs.fs.lstatSync(dir).isDirectory()) {
            return;
        }

This block of code stops npm linked shared mojits from being loaded.

@caridy
Copy link
Contributor

caridy commented Aug 14, 2013

regression introduced by fix for issue #1017 /cc @lzhan @isao

lzhan pushed a commit to lzhan/mojito that referenced this issue Aug 14, 2013
lzhan pushed a commit that referenced this issue Aug 27, 2013
fix issue #1219 npm link shared mojits failed
@caridy
Copy link
Contributor

caridy commented Mar 3, 2014

@lzhan can you validate and close this?

@lzhan lzhan closed this as completed Mar 3, 2014
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