-
Notifications
You must be signed in to change notification settings - Fork 13
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
Problem with changing dependencies versions #14
Comments
Not sure I fully understand what you're asking? Basically the way browserify-incremental works is that if a file's mtime has not changed, then the cached version will be used, otherwise it will be rebuilt. If you switch versions of a dependency (eg. by npm install) then the dependency's files mtimes should have changed, causing the cached version to be invalidated. |
@jsdf I think the original reporter meant "file you're requiring" by "dependency." If so, I've been bitten by this, and have a repro:
So the first build succeeds, as expected, and everything's fine. Then you have to build again, because of a separate bug: #8. Then the third build should fail, as we've deleted the In practice this shows up for us (fortunately) as build failures on Travis when the tests work fine locally, but it's still annoying to track down. |
@jsdf mtimes will not work for NPM packages; NPM sets the mtime of files based on the time at which the package was created, not the time at which the files were updated on the local system. For example, I have react-bootstrap 0.25.2 fetched locally. I fetched this package on 20 October, but the version was released on 12 September. Directories have mtimes of 20 October, but all the JS files involved have mtimes of 12 September. |
Well, if I remember well I'm still being bitten by this issue, but only when I am using a linked dependency (via |
I can't reproduce (and I couldn't understand the generated source also), but apparently there is an issue with dependencies that change versions and/or deleted dependencies. Maybe browserify-incremental keeps using the same old versions. Is it possible that this is happening?
The text was updated successfully, but these errors were encountered: