-
-
Couldn't load subscription status.
- Fork 4.6k
Support architecture limitations for apps and allow richdocumentscode_arm64 though htaccess #22844
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
Conversation
|
We have more places that would benefit if we make sure they can be covered. I had more in my head, but can't find them right now. So I only can link talk_matterbridge: https://github.com/nextcloud/talk_matterbridge/blob/master/lib/Migration/RegisterBinary.php#L50 which tries to identifiy 32bit, 64bit and arm64 |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
9465bd7 to
04eb1bb
Compare
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 33137: failuremariadb10.4-php7.3Show full logmysql8.0-php7.2Show full logacceptance-app-files-sharing
Show full log |
|
|
||
| $architecture = php_uname('m'); | ||
| if (PHP_OS_FAMILY === 'Linux' && in_array($architecture, ['x86_64', 'aarch64'])) { | ||
| $hubApps[] = 'richdocuments'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Collabora itself does not run on 32 bit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does but without the code builtin server it does not make much sense to install it automatically
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After some chat with @juliushaertl: This is on purpose because the hub bundle only makes sense to install rich documents by default if on one of those platforms where also the built in version of the server runs. Thus this is correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beside my comment this is fine 👍
Let me do that. |
See nextcloud/server#22844 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
See nextcloud/server#22844 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
ToDo: