-
Notifications
You must be signed in to change notification settings - Fork 236
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
Please create new config file for arm devices #195
Comments
Been working on one here: All these apps have been tested with a Raspberry Pi 4. The docker image was either changed for ARM or if no arm version existed the app was removed. Waiting on some formatting changes but then you will be able to just load the ARM template. I always looking for more ARM apps to add too btw. |
I'll check this out, thanks. |
Hi coudnt find your email. When is this gone be available you reckon? I imported the list but nothing is getting displayed so i guess it hasnt been verified yet? |
Hey sorry! I thought this was all resolved. @SelfhostedPro can you help? |
I just tested the Arm template and all I can see is the last entry, that would be SWAG in this case: You can see it here: selfhosted_templates/Template/template.yml Lines 2158 to 2160 in 412780d
I then tested using a modified template with SWAG removed, and sure enough it's only Papermerge now showing up. I can see that SelfhostedPro/Yacht#75 has been merged for a while now, but something must've mocked it up. From a quick glance over the code, I think that the problem is here: https://github.com/SelfhostedPro/Yacht/blob/master/backend/api/db/crud/templates.py#L56 I think we should be using
I can't test the code unfortunately since I'm having issues with @SelfhostedPro, Let me know if you require more info. Hopefully this has been helpful to someone :) |
yaml.load_all() is unsafe which is why I'm using the safeload. The correct arm branch to use is this one: As the one you're trying has errors and is not valid yaml. |
Thank you! It's indeed working correctly.
I think it might be a good idea to point it out in the README somewhere. And on a related note, maybe link the new https://yacht.sh/docs/ in Yacht's repo description :) |
I'll definitely add the info there. I'm looking at changing around how the repo is laid out so there may be a different URL that's recommended in the future. I'll make sure whatever I wind up doing, that it's in the documentation. |
Some services does not work on arm devices like pi. Please create different config file that is tested on such devices so we know it works on devices like Pi. Thanks.
The text was updated successfully, but these errors were encountered: