ember-cli-version-checker icon indicating copy to clipboard operation
ember-cli-version-checker copied to clipboard

issue with npm link of ember-cli on windows.

Open stefanpenner opened this issue 8 years ago • 6 comments

version ends up being undefined for ember-cli all the time. Might something to do with symlinks/junctions on windows?

stefanpenner avatar Dec 04 '16 01:12 stefanpenner

It's possible, I'll update my ievms and try to repro locally.

rwjblue avatar Dec 04 '16 01:12 rwjblue

@rwjblue i'll try tonight/tomorrow to debug this. I have a good local reproduction, just tired so i figured I would leave a reminder issue :)

stefanpenner avatar Dec 04 '16 01:12 stefanpenner

Curious if this is still an issue.

rwjblue avatar Mar 23 '20 01:03 rwjblue

@rwjblue @stefanpenner I am having the same issue. I am running yarn link and .exists() always returns undefined

suchitadoshi1987 avatar Apr 30 '20 01:04 suchitadoshi1987

@suchitadoshi1987 are you sure? This issue I believe is windows specific. Do you have a reproduction handy?

stefanpenner avatar Apr 30 '20 19:04 stefanpenner

I am not sure it's 100 % related to the same issue but I am also getting undefined when checking the version property of a package.

I was trying to work on this PR: https://github.com/typed-ember/ember-cli-typescript/pull/1142

And as we can see in the original PR, the author left a note:

Seems like somehow VersionChecker isn't getting the dependencies in the ember-cli-blueprint-test-helpers context 🤔

I am also getting the same error.

I tried to dig into this and it turns out, for some reason I don't understand, the following function resolvePackagePath returns null for the package ember-data (while it is correctly detected in the dependencies of the project): https://github.com/ember-cli/ember-cli-version-checker/blob/513e0d25bb75b6073f1455b6a63135b84fbfb10b/src/dependency-version-checker.js#L41

Because this._jsonPath is set to null, the following if() that are supposed to set this._version are not entered and the property stays undefined.

MrChocolatine avatar Feb 14 '21 14:02 MrChocolatine