-
Notifications
You must be signed in to change notification settings - Fork 110
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
What happens when I install go-ipfs
?
#204
Comments
Also, the load of the package file a the start of Lines 342 to 360 in a904dc7
This undocumented exported function in turn calls another (also undocumented) function with the same name and a confusing Lines 31 to 51 in a904dc7
|
(Part of #203.)
I need to put this in a document PR but before that I'm leaving a rough description here so someone can validate if this is correct.
The
make install
ingo-ipfs
command ends up calling (not sure how)gx install --global
.The
InstallCommand
is called where the branch with no arguments is executed (because the install process is being done in the package of the current directory),gx/main.go
Lines 359 to 380 in a904dc7
In
InstallPath
the--global
flag is forwarded andgx-go
will return$GOPATH/src
. The--global
flag seems to be used only for this call and for thepost-install
hook which it appears to ignore it, so right now it seems this flag is only used to determine the install path, its purpose should be clearly documented.The
InstallDeps
function will do two things (more clearly differentiated in #197), fetch all the dependencies (direct and transitive ones), saving them in the install path returned by thegx-go
sub-tool and then it will call thepost-install
hook of this same sub-tool.gx-go
will implement that hook rewriting the import statements in the.go
files of those dependencies making them point to the install path with thegx/ipfs
path appended (and how do we call that sub-path of the install path?), i.e.,$GOPATH/src/gx/ipfs/
.It may be useful to open an in issue in
gx-go
to review the rewrite process and document it here.The text was updated successfully, but these errors were encountered: