-
Notifications
You must be signed in to change notification settings - Fork 67
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
Unable to load plugin #63
Comments
I also haven't been able to get this to build against master of go-ipfs:
|
I'm not sure but go is weird about plugins. Setting Alternatively, you can build your plugin into go-ipfs by following the directions here https://github.com/ipfs/go-ipfs/blob/master/docs/plugins.md#preloaded-plugins.
Assuming your current working directory is go-ds-s3, did you delete the go.mod file? |
@Stebalien Thanks for your help. I've tried again starting from scratch in both repos. First, in go-ipfs I checked out release/v0.4.23 and applied the 2 line patch for the global pin lock Then in go-ds-s3 I checked out release/v0.4.23 and ran
That builds fine now (I think you're right - I must have deleted go.mod), but then when I try to load it in the ipfs built in step 1 I get the following from ipfs init:
This is a huge problem for us because we can't upgrade ipfs, so to avoid a global deadlock we have to disable gc entirely. |
Is it possible to strip down the amazon s3 lib such that it could be included in ipfs by default? I'm not sure if the go compiler does that, but based on the size of the .so I'd guess not. I was able to reduce the Java amazon S3 library down to 2.8mb, which would be 6% of the current ipfs binary size. |
This looks very similar to what I'm seeing. I'm going to try some of their ideas: golang/go#27751 UPDATE: none of that works. |
I've even gone so far as to try forking go-ipfs, renaming the module, adding the patch, pushing a new version tag to my repo, and then trying linking the s3 plugin to that. (in case the error was some issue with using a local go-ipfs version rather than a remote) Sadly that results are the same goprocess error when ipfs tries to load the module. |
Yeah, I'm seeing that error. GAH! |
TL;DR: Checkout the Ok, I found the issue.
GOFLAGS += -asmflags=all=-trimpath="$(GOPATH)" -gcflags=all=-trimpath="$(GOPATH)" Adding this to the 0.4.23 makefile fixes the problem. But really, just use the branch. |
Ah, thank you so much! You're a lifesaver! That fixes it. I've patched the go-ipfs makefile with that and everything works now. We've built our ipfs installer around optionally loading plugins, so didn't want to go down the "all in one" route. Thanks again! |
I've built this plugin and used it with ipfs v0.4.23 and it works fine. However, When I rebuild ipfs v0.4.23 with the 2 line diff from ipfs/go-ipfs-pinner#2 I get the following error:
@Stebalien Any idea what could be happening here? That goprocess repo hasn't changed in almost a year.
The text was updated successfully, but these errors were encountered: