Follow the chain of paths for local dependencies

[micky2be]

Look for local chain of dependencies based on parent dependencies path list.
Kept the pathfrom the root component.json during the lookup for backward compatibility

[micky2be]

Include fix from #336

[micky2be]

@visionmedia > Let me know if you wanna me to do any change on this

[tj]

hmm a test would be good if possible, locals were not influenced by the parent's .paths before this? should be

[micky2be]

Will make a test then.
But if you look at #324, you will see I'm not the only one having the problem

[micky2be]

@visionmedia Added a test to replicate our problem

[dominicbarnes]

+1

I just ran into this problem and merging in this branch solves it for me.

[micky2be]

@dominicbarnes I'm glad it fixed your problem

[tj]

test doesn't pass (missing component.json) but ill take a look

[micky2be]

Weird, I may have messed up during a merge or something.
I pushed a fix anyways

[tj]

still fails for me, there's no component.json:

SyntaxError: /Users/tj/projects/component/test/fixtures/local/component.json: Unexpected end of input
    at Object.parse (native)
    at Object.Module._extensions..json (module.js:475:27)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:362:17)
    at require (module.js:378:17)
    at loadConfig (/Users/tj/projects/component/bin/component-install:246:12)
    at Object.<anonymous> (/Users/tj/projects/component/bin/component-install:71:12)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)

[micky2be]

It's not like I wanna to pression on @visionmedia but this is getting old.

[tj]

well this is causing me more work than just fixing it haha, usually people run their own tests first :p

[micky2be]

Oh! The file was ignored by git. Pushing now

[tj]

damn, still missing the other ones, master has !test/fixtures/**/component.json in the .gitignore now so it should be less annoying that way, but "direct" and the other two are missing for me as well

[micky2be]

Git didn't see those even with the .gitignore update.
Gonna force them

On Jun 19, 2013, at 9:04, TJ Holowaychuk [email protected] wrote:

damn, still missing the other ones, master has
!test/fixtures/**/component.json in the .gitignore now so it should be less
annoying that way, but "direct" and the other two are missing for me as well

—
Reply to this email directly or view it on
GitHubhttps://github.com//pull/337#issuecomment-19651762
.

[micky2be]

Sorry for the delay, I had to do it from my phone

[tj]

hahaha that can't be too easy, thanks man

[mikerudolph]

What is the status on this fix? I don't see it ever being merged?

[micky2be]

It's in since v0.16.4

[mikerudolph]

I am still seeing this issue while running on latest, should I open a new issue?

[micky2be]

Not having any problems on my side. You sure all your paths are correct?
If you really have a problem you should definitely open an issue.
Don 't forget to mention versions (component, node), and describe your use case.