YunoHost-Apps / roundcube_ynh

Roundcube package for YunoHost
https://roundcube.net/
GNU Affero General Public License v3.0
11 stars 22 forks source link

Install failed on testing #6

Closed maniackcrudelis closed 6 years ago

maniackcrudelis commented 7 years ago

Roundcube fails on testing because the temp directory is set by root, and after that, composer is executed with admin.

So, admin has not enought right access to work.

NicolasPetton commented 7 years ago

I also face the same issue on YNH 2.6.

magikcypress commented 7 years ago

If you want, you can try this version that is not totally upgraded

https://github.com/magikcypress/roundcube_ynh

redvoodoochild commented 7 years ago

Same Here, with YHN 2.6.4

magikcypress commented 7 years ago

@redvoodoochild Can you send a log for this problem ? Thank

redvoodoochild commented 7 years ago

Well, Now I can't : i've installed the version you posted above : https://github.com/magikcypress/roundcube_ynh And it's working like a charm.

The error was that composer could'nt be installed. Regards

NicolasPetton commented 7 years ago

Here's the log on YNH 2.6:

+ cp /var/cache/yunohost/from_file/roundcube_ynh-38bb39f7389c99086865230db189463e83227afa/sources/composer.json-dist /tmp/tmp.h9FdCmOorl/composer.json-dist
+ init_composer /tmp/tmp.h9FdCmOorl
+ local DESTDIR=/tmp/tmp.h9FdCmOorl
+ local AS_USER=admin
+ curl -sS https://getcomposer.org/installer
+ exec_as admin COMPOSER_HOME=/tmp/tmp.h9FdCmOorl/.composer php -- --quiet --install-dir=/tmp/tmp.h9FdCmOorl
+ local USER=admin
+ shift 1
++ whoami
+ [[ admin = root ]]
+ sudo sudo -u admin COMPOSER_HOME=/tmp/tmp.h9FdCmOorl/.composer php -- --quiet --install-dir=/tmp/tmp.h9FdCmOorl
The installation directory "/tmp/tmp.h9FdCmOorl" is not writable
+ ynh_die 'Unable to install Composer'
Warning: Unable to install Composer
+ echo 'Unable to install Composer'
+ exit 1
maniackcrudelis commented 6 years ago

Fixed by https://github.com/YunoHost-Apps/roundcube_ynh/pull/13

NicolasPetton commented 6 years ago

@maniackcrudelis awesome, thanks!