-
Notifications
You must be signed in to change notification settings - Fork 3
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
plugin crash #137
Comments
Hey @rickysarraf, yeah the plugin feature is ready and working. How do you build the plugin solib and photoprism binary? A big limitation of the plugins in their current form is that they need to be compiled with the exact same version of photoprism, as the version running. AFAIK you compile your stuff from source, so in your case it should be easy. For me at least, the following works fine:
|
Well. Yes. I have been compiling myself all this while. But lately, I'm leveraging GitHub Actions, taking ideas from your setup. So, I have the plugin built in the I hoped that would work but it certainly isn't working. I'll look more. |
@kvalev Any reason for not building it as part of the image build job itself ? Given the tight version requirement, wouldn't it be more appropriate to build the plugins right there itself. And then ship those plugins as is as part of the image ? |
@rickysarraf Lets continue the discussion in the linked PR. |
@kvalev
Not sure if the new plugin feature is ready for testing, but given that it was merged, I included that too in my this round of tests.
Is the plugin feature ready to test ?
I ask because right now I get the below crash.
Full log at: https://gist.github.com/rickysarraf/168ad4dc9b88a843ccb0b9f82ea6a080
The text was updated successfully, but these errors were encountered: