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
depswriter does not produce the new deps format (i.e. module support) #382
Comments
I don't think anyone uses depswriter.py within Google, so (if I remember correctly) we want to try and get rid of it, but I don't know if we have any suitable replacement for open-source users. |
So basically this is a no-op for open source then? |
At least for the moment, yes :( |
I thought I updated this. Are you sure you are using the latest?
|
I am not 100% sure what the new format should be, but this looks like it is not (missing last argument?) |
drat, it was missed, I was thinking about something else. Thanks for def _GetDepsLine(path, js_source): provides = sorted(js_source.provides) return 'goog.addDependency('%s', %s, %s, %s);\n' % ( On Thu, Dec 4, 2014 at 2:39 PM, pstjvn notifications@github.com wrote:
|
Thank you |
Fixes #382 on github ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=81435962
Fixes google#382 on github ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=81435962
depswriter.py does not produce the new dependency format needed for the module support, thus it prevent the usage of modules in third party code.
Please push the needed updates to the open source repo.
Thanks.
The text was updated successfully, but these errors were encountered: