Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#57623 - [psi] Should not replace psi-plus

Attached to Project: Community Packages
Opened by drathir (drathir) - Saturday, 24 February 2018, 14:11 GMT
Last edited by Sergej Pupykin (sergej) - Wednesday, 18 April 2018, 18:48 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sergej Pupykin (sergej)
Architecture x86_64
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Package after moving to community repository have wrong name it should have psi-plus naming similar as other connected to it localise/addon packages:
- psi-l10n
- psi-plugins

also package psi-plus shouldn't conflict with psi package...


Additional info:
* package version(s)
latest
* config and/or log files etc.
:: Replace psi-plus with community/psi? [Y/n] n
:: Replace psi-plus-l10n with community/psi-l10n? [Y/n] n
:: Replace psi-plus-plugins with community/psi-plugins? [Y/n] n


Steps to reproduce:
just pacman -Syu

This task depends upon

Closed by  Sergej Pupykin (sergej)
Wednesday, 18 April 2018, 18:48 GMT
Reason for closing:  Fixed
Comment by Antonio Rojas (arojas) - Saturday, 24 February 2018, 18:09 GMT
Can you elaborate on why it is wrong?
Comment by drathir (drathir) - Sunday, 25 February 2018, 02:40 GMT
Its bc packages psi and psi-plus are two separated git repositories, its always like psi-plus is kinda more edge version with more recent patches/ideas applied, unlike psi its kind of abandoned legacy/stable version which occasionally time to time merging some patches and upgrades from psi-plus repository, and package was wrongly renamed from psi-plus to psi with one of last community migrations, according to repository used to build as source packages should be named psi-plus... in short psi is bare/legacy repository psi-plus is more latest power version based on psi repository with own patches applied in the way both are different/separate versions... Greetings...
Comment by Antonio Rojas (arojas) - Sunday, 25 February 2018, 09:16 GMT
From the upstream website:
Psi is a cross-platform powerful XMPP client designed for experienced users.
Psi+ is a development branch of Psi XMPP client.

The psi package contains the former, whose latest version is 1.3 https://github.com/psi-im/psi/releases instead of the latter, whose latest version is 1.2.241 https://github.com/psi-plus/psi-plus-snapshots/releases

So the package naming is correct and consistent with its content. Or are you requesting that psi-plus is provided instead of the stable psi? In that case this would be a feature request, not a bug report.
Comment by Doug Newgard (Scimmia) - Sunday, 25 February 2018, 23:41 GMT
It looks to me like the replaces is wrong, at least. It probably shouldn't have all of the conflicts, either.
Comment by drathir (drathir) - Monday, 26 February 2018, 00:05 GMT
@arojas its not like psi higher version number mean is more fresh than psi-plus one and even never was in past as remember bc psi-plus get kind of different versioning applied, also according to psi git https://github.com/psi-im/psi:
New in 1.0
- Almost all Psi+ patches were merged (a lot of features).
New in 1.1
- More Psi+ patches merged.

that like earlier mentioned psi merging from psi-plus time to time refreshing (legacy version) and never was fully merged but but randomly...

personally i see sane solve of this issue by
- rename into old psi-plus name and use https://github.com/psi-plus repository
or
- left psi naming and switch into https://github.com/psi-im/psi repository with delete conflict of psi-plus bc both version are different and standalone non overlapping versions (separate config dirs etc...) as always was...
Greetings...


Edited:
ofc according to choice that also need to be applied into this packages:
- psi-l10n
- psi-plugins
Comment by Sergej Pupykin (sergej) - Monday, 26 February 2018, 13:10 GMT
I removed conflicts and replaces.
Comment by drathir (drathir) - Monday, 26 February 2018, 22:55 GMT
@sergej Thanks a lot for fast fixing... In free time (if possible ofc) would be nice to take a look if https://git.archlinux.org/svntogit/community.git/tree/trunk/join.patch?h=packages/psi is still needed, personally not sure... Greetings...

Loading...