Failed upgrade from 1.6.0~ynh4 to 1.6.6~ynh1 #193
Labels
Comments
|
Failed for me too with |
|
It seems the error come with this line on the script upgrade Line 203 in 47277f9 |
|
Same problem here : I've managed to solve the problem by uninstalling then reinstalling the app, so probably the preceding comment by rodinux is right and there is an issue with script upgrade. The problem with my solution is that by doing a fresh re-installation you lose all your saved contacts. I'll leave it this way for now the time to find a better solution. If you decide to go my way remind to do a backup first. |
|
same here too https://paste.yunohost.org/raw/ifaribarun |
|
I may be wrong, and don't do that unless you know better. But in such case, maybe the way to proceed would be removing (keeping a backup may be advised) the |
|
Same issue here: https://paste.yunohost.org/raw/bevogetomo |
|
Same here (Unable to install core dependencies with Composer) : https://paste.yunohost.org/raw/ziqudukavu Thank you for your help |
|
There is definitely something wrong with the upgrade script, as I understand we never update composer.json during upgrade Lines 110 to 120 in 47277f9 I think we should always run
Workaroundcd /var/www/roundcube
rm composer.json # will be copied from composer.json-dist
rm -r vendor # conflict with ldap old installation
# Answer Yes to installing roundcube plugin
COMPOSER_ALLOW_SUPERUSER=1 php8.1 /var/www/roundcube/composer.phar require roundcube/carddav 5.0.1 --with-all-dependencies
yunohost app upgrade roundcube -b |
Could you try do fix-PR to test it ?? |
|
I'm looking into it, trying to reproduce in a dev env |
|
@rodinux I looked into it more in depth, and the problem originate even before |
Draft
5 tasks
|
Re. composer.json, isn't that done by Roundcube's script already: https://github.com/roundcube/roundcubemail/blob/master/bin/update.sh#L177 ? |
|
Well, I found another way to upgrade. I have rename the file I put here a diff between the two files https://rodinux.fr/zerobin/?2ddcee3f36c33f74#3bAwsY5wFEvps6FYpqko6uFcKGs4o6V2KzJZxaFaxvWe |
|
Renaming composer.lock to something else worked for me too. Upgraded successfully. |
Could you share the one it generated after the upgrade then ? :) |
|
I can put here the new composer.lock generated here |
|
Great thank you ! So we could use this directly in the package ? |
I don't know, I just understand if you remove or rename this file, the process in install will generate it. |
|
Well, I've been trying to update with this solution but getting an error : seems that php-net_ldap3.git needs authentication now
|
Can you tell us how you process ? |
|
On my side i have done this to fix it: |
I have a lot of applications installed on my server. Will this method affect any of them? |
|
This should not affect other applications.
For me it was much simpler to upgrade; I just renamed
/var/www/roundcube/composer.lock to something else (I added .bak in the
end). Then the upgrade went through.
…On 17. 04. 24 00:51, Tio wrote:
On my side i have done this to fix it:
|cd /var/www/roundcube/ mv composer.lock composer.lock.bkp rm
composer.json rm -r vendor COMPOSER_ALLOW_SUPERUSER=1 php8.1
/var/www/roundcube/composer.phar require roundcube/carddav 5.0.1
--with-all-dependencies cd ../ yunohost app upgrade roundcube
--force cd /var/www/roundcube/ source /usr/share/yunohost/helpers
app=roundcube ynh_composer_exec --commands="require kolab/net_ldap3
--update-with-all-dependencies" --phpversion=8.1
--workdir=/var/www/roundcube ynh_composer_exec --commands="require
pear/net_sieve --update-with-all-dependencies" --phpversion=8.1
--workdir=/var/www/roundcube |
I have a lot of applications installed on my server. Will this method
affect any of them?
—
Reply to this email directly, view it on GitHub
<#193 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA7WIUV5IJSX35MLGX2SHQDY5WTQVAVCNFSM6AAAAABCNL45G6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGAZTOMJTGI>.
You are receiving this because you commented.Message ID:
***@***.***>
|
Ok that was easy and it worked. Thanks! |
|
AFAICS, there's a 1.6.6~ynh2 version that appeared recently... would it by chance address this issue (btw, I couldn't spot a changelog to try and track this, and the Git repo isn't really explicit on what's cooking, what would be in a release... but I may have overlooked some release labelling practices). |
|
In desperation, I deleted the lockfile. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Upgrade failed when try to upgrade by click on in the webadmin both VPS and ARM board 64 bits with the same error
Context
Steps to reproduce
Same error with CLI on both.
Expected behavior
*Upgrade from from 1.6.0 ynh4 to 1.6.6 ynh1 should work.
https://packages.debian.org/en/bookworm/php-guzzlehttp-guzzle
https://packages.debian.org/en/bookworm/php/php-bacon-qr-code
Logs
When an operation fails, YunoHost provides a simple way to share the logs.
ARM bord 64bits: https://paste.yunohost.org/raw/ecobolahup
EDIT :
Maybe composer in YunoHost should be update/upgrade because in the roundcubemail-1.6.6.tar.gz there is a file name composer.json-dist with these requirements about guzzlehttp/guzzle (6.5.8 instead of 7.3.0) and bacon/bacon-qr-code (missing instead of 2.0.0) :
The text was updated successfully, but these errors were encountered: