Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Incorporate plugins #347

Closed
sdboyer opened this issue Mar 23, 2017 · 2 comments
Closed

Incorporate plugins #347

sdboyer opened this issue Mar 23, 2017 · 2 comments

Comments

@sdboyer
Copy link
Member

sdboyer commented Mar 23, 2017

plugins are here as of Go 1.8. Despite some of the general issues and difficulties with getting plugins to compile using objects shared with the main program, it seems likely that we'll need to do something with them as part of dependency management.

Problem is, I'm not sure much we could really do via static analysis to determine what plugins are necessary. The import names are just strings, and if the focal point during analysis is an individual file in a particular package (with a plugin.Open call site), then the origin of those strings could easily be in a package from an entirely separate project, or generated based on the input to a running program.

If we can't provide any assurances about the completeness of analysis, then it suggests dep should handle plugins - if it does at all - through an entirely different mechanism than typical import-based dependencies. Perhaps explicit declaration in the manifest? I don't know.

In any case, this issue is here for some open discussion on the topic.

@davecheney
Copy link
Contributor

davecheney commented Mar 23, 2017 via email

@sdboyer
Copy link
Member Author

sdboyer commented Mar 25, 2017

@davecheney fully agree, this is not a priority. Really just opened it so that there's a place for any thoughts or ideas that might come up as we move along.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants