-
Notifications
You must be signed in to change notification settings - Fork 106
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
Dependency not updating correctly #267
Comments
Hmm. This is surprising. So the situation was that in I wonder if we can reproduce this behaviour. Do you have a check-in of your previous |
I'm afraid I don't have a check-in and can't reproduce. But no, "scss": {} never made it into smart.json, which I think was the source of the problem. |
Okay, I was able to reproduce this. I don't have code, but this is precisely what happened:
My wild guess is that meteorite is treating the symlinks in the packages directory as authoritative in a case in which it shouldn't... but I'm hoping that step-by-step is enough info for you :-). |
Ahh right. Yes. I see what happened. You are correct in your analysis. If there's a package symlinked in FWIW there are times when you are better off just messing with |
This was tricky to debug, so I'll need to go into some explanation.
The SCSS package was a dependency of two of my manually-added packages, Bourbon and Neat. At first install, the SCSS version was 0.8.3. Then Meteor 0.8 hits, and SCSS releases 0.8.4 to be compatible with Blaze. However, no amount of "mrt update" would bump SCSS from 0.8.3 to 0.8.4. The solution was to "mrt remove Bourbon", "mrt remove Neat", and manually remove the scss symlink in my packages dir. Only then could I "mrt add scss" and get the newest version.
I assume the desired behavior would have been to have meteorite update the dependency, yes? I did check Bourbon and Neat to see if they had locked into 0.8.3, and they did not.
The text was updated successfully, but these errors were encountered: