Bug 32460 - openbox and task-lxqt-minimal install gtk version of obconf
Summary: openbox and task-lxqt-minimal install gtk version of obconf
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 9
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA9-64-OK MGA9-32-OK
Keywords: advisory, validated_update
Depends on:
Blocks:
 
Reported: 2023-10-27 22:40 CEST by katnatek
Modified: 2023-10-31 22:36 CET (History)
5 users (show)

See Also:
Source RPM: openbox,obconf,obconf-qt,task-lxqt
CVE:
Status comment:


Attachments
Diff file for the openbox spec (1.08 KB, patch)
2023-10-27 22:43 CEST, katnatek
Details | Diff
Diff file for obconf spec (515 bytes, patch)
2023-10-27 22:46 CEST, katnatek
Details | Diff
Diff file for obconf-qt spec (464 bytes, patch)
2023-10-27 22:48 CEST, katnatek
Details | Diff
Diff file for task-lxqt spec (552 bytes, patch)
2023-10-27 22:54 CEST, katnatek
Details | Diff
Updated Diff file for the openbox spec (635 bytes, patch)
2023-10-28 19:23 CEST, katnatek
Details | Diff
output of LC_ALL=C urpmi --test task-lxqt-minimal (6.38 KB, text/plain)
2023-10-30 10:52 CET, Marja Van Waes
Details
Log of all my test (53.20 KB, text/plain)
2023-10-30 18:20 CET, katnatek
Details
Log of all my test on i586 (56.05 KB, text/plain)
2023-10-30 19:32 CET, katnatek
Details

Description katnatek 2023-10-27 22:40:03 CEST
Description of problem:

Install task-lxqt packages make the gtk version of obconf instead of obconf-qt, this is due openbox spec Recommends obconf

On bug#17499#c1 Rémi Verschelde  recommends create a virtual require but not was implemented

Version-Release number of selected component (if applicable):

openbox-3.6.1-13 , obconf-2.0.4-10 , obconf-qt-0.16.2-1 , task-lxqt-1.3.0-2 


How reproducible:


Steps to Reproduce:
LC_ALL=C urpmi --test task-lxqt-minimal
To satisfy dependencies, the following packages are going to be installed:
(test only, installation will not be actually done)
  Package                        Version      Release       Arch    
(medium "Core Release (distrib1)")
  bsdtar                         3.6.2        5.mga9        i586    
  chbg                           2.0.1        29.mga9       i586    (recommended)
  fortune-mod                    3.12.0       2.mga9        i586    (recommended)
  libfm                          1.3.2        4.mga9        i586    
  libfm-qt                       1.3.0        1.mga9        i586    
  libfm-qt13                     1.3.0        1.mga9        i586    
  libfm4                         1.3.2        4.mga9        i586    
  liblxqt                        1.3.0        1.mga9        i586    
  liblxqt-globalkeys-ui1         1.3.0        1.mga9        i586    
  liblxqt-globalkeys1            1.3.0        1.mga9        i586    
  liblxqt1                       1.3.0        1.mga9        i586    
  libmenu-cache3                 1.1.0        7.mga9        i586    
  libmuparser2                   2.3.4        1.mga9        i586    
  libobrender32                  3.6.1        13.mga9       i586    
  libobt2                        3.6.1        13.mga9       i586    
  libqtermwidget5_1              1.3.0        1.mga9        i586    
  libstartup-notification1_0     0.12         13.mga9       i586    
  libstatgrab10                  0.92.1       2.mga9        i586    
  libsysstat0                    0.4.6        3.mga9        i586    
  lxde-icon-theme                0.5.1        6.mga9        noarch  (recommended)
  lxmenu-data                    0.1.5        5.mga9        noarch  
  lxqt-about                     1.3.0        1.mga9        i586    
  lxqt-archiver                  0.8.0        1.mga9        i586    
  lxqt-config                    1.3.0        1.mga9        i586    
  lxqt-globalkeys                1.3.0        1.mga9        i586    
  lxqt-notificationd             1.3.0        1.mga9        i586    
  lxqt-panel                     1.3.0        1.mga9        i586    
  lxqt-policykit                 1.3.0        1.mga9        i586    
  lxqt-powermanagement           1.3.0        1.mga9        i586    
  lxqt-qtplugin                  1.3.0        1.mga9        i586    
  lxqt-runner                    1.3.0        1.mga9        i586    
  lxqt-session                   1.3.0        1.mga9        i586    
  lxqt-sudo                      1.3.0        1.mga9        i586    
  lxqt-themes                    1.3.0        1.mga9        noarch  
  lxshortcut                     1.3.2        4.mga9        i586    
  menu-cache                     1.1.0        7.mga9        i586    
  obconf                         2.0.4        10.mga9       i586    (recommended)
  openbox                        3.6.1        13.mga9       i586    
  pcmanfm-qt                     1.3.0        2.mga9        i586    
  picom                          10.2         1.mga9        i586    (recommended)
  picom-conf                     0.16.0       4.git2022073> i586    (recommended)
  python3-pyxdg                  0.28         1.mga9        noarch  
  qterminal                      1.3.0        1.mga9        i586    
  qtermwidget                    1.3.0        1.mga9        i586    
  task-lxqt-minimal              1.3.0        2.mga9        noarch  
  xdg-desktop-portal-lxqt        0.4.0        1.mga9        i586    
  xwininfo                       1.1.5        3.mga9        i586    (recommended)
(medium "Tainted Updates (distrib23)")
  xscreensaver                   6.08         1.mga9.taint> i586    (recommended)
  xscreensaver-base              6.08         1.mga9.taint> i586    (recommended)
  xscreensaver-common            6.08         1.mga9.taint> i586    (recommended)
77MB of additional disk space will be used.
42MB of packages will be retrieved.
Proceed with the installation of the 50 packages? (Y/n) y

Suggested changes in the specs fix the issue
Comment 1 katnatek 2023-10-27 22:43:31 CEST
Created attachment 14108 [details]
Diff file for the openbox spec

These changes allow task-lxqt-minimal recommend obconf-qt and add a task package to install openbox with obconf or obconf-qt
katnatek 2023-10-27 22:43:51 CEST

CC: (none) => ngompa13

Comment 2 katnatek 2023-10-27 22:46:26 CEST
Created attachment 14109 [details]
Diff file for obconf spec

This change make obconf provide a virtual require added openbox spec
Comment 3 katnatek 2023-10-27 22:48:35 CEST
Created attachment 14110 [details]
Diff file for obconf-qt spec

This change provides a virtual require in openbox spec
Comment 4 katnatek 2023-10-27 22:54:26 CEST
Created attachment 14111 [details]
Diff file for task-lxqt spec

These changes allow task-lxqt-minimal recommends obconf-qt allowing to manually switch to obconf if is necessary (not make obconf-qt a hard require)
katnatek 2023-10-27 22:55:50 CEST

Whiteboard: (none) => MGA9TOO

Comment 5 David GEIGER 2023-10-28 06:49:09 CEST
(In reply to katnatek from comment #1)
> Created attachment 14108 [details]
> Diff file for the openbox spec
> 
> These changes allow task-lxqt-minimal recommend obconf-qt and add a task
> package to install openbox with obconf or obconf-qt

Why add a task-openbox sub-pkg?

For me just replace "Recommends: obconf" with "Recommends: obox-conf" is enough.

CC: (none) => geiger.david68210

Comment 6 katnatek 2023-10-28 19:23:01 CEST
Created attachment 14113 [details]
Updated Diff file for the openbox spec

(In reply to David GEIGER from comment #5)
> Why add a task-openbox sub-pkg?
> 
> For me just replace "Recommends: obconf" with "Recommends: obox-conf" is
> enough.
You are right, I test on my system and just with that change in openbox spec and the changes made in the other specs the installation of openbox allows choosing between obconf and obconf-qt, and the task-lxqt packages get obconf-qt as dependency

Uploaded a new diff file for openbox spec

Attachment 14108 is obsolete: 0 => 1

Comment 7 Marja Van Waes 2023-10-29 14:22:16 CET
Katnatek: Like the other report, assigning to you and setting status to ASSIGNED, because you are already on it.

Assignee: bugsquad => j.alberto.vc
Status: NEW => ASSIGNED
CC: (none) => marja11

Comment 8 David GEIGER 2023-10-30 06:30:19 CET
Fixed for Cauldron!
Comment 9 David GEIGER 2023-10-30 06:34:29 CET
Assigning to QA,

Packages in 9/Core/Updates_testing:
======================
obconf-2.0.4-10.1.mga9
obconf-qt-0.16.2-1.1.mga9
openbox-3.6.1-13.1.mga9
libobt2-3.6.1-13.1.mga9
lib64obt2-3.6.1-13.1.mga9
libobrender32-3.6.1-13.1.mga9
lib64obrender32-3.6.1-13.1.mga9
libopenbox-devel-3.6.1-13.1.mga9
lib64openbox-devel-3.6.1-13.1.mga9
task-lxqt-1.3.0-2.1.mga9.noarch.rpm
task-lxqt-minimal-1.3.0-2.1.mga9.noarch.rpm

From SRPMS:
obconf-2.0.4-10.1.mga9.src.rpm
obconf-qt-0.16.2-1.1.mga9.src.rpm
openbox-3.6.1-13.1.mga9.src.rpm
task-lxqt-1.3.0-2.1.mga9.src.rpm

Version: Cauldron => 9
Assignee: j.alberto.vc => qa-bugs
Whiteboard: MGA9TOO => (none)

Comment 10 Marja Van Waes 2023-10-30 10:52:54 CET
Created attachment 14116 [details]
output of LC_ALL=C urpmi --test task-lxqt-minimal

In cauldron, when I do:

LC_ALL=C urpmi --test task-lxqt-minimal

I do not get a choice between obconf and obconf-qt, but obconf-qt is selected.

Is that expected when not having Gnome installed, or should I have gotten the choice anyway? I do have gtk4.0 and gtk+3.0 installed (and more gtk and gnome stuff)
Comment 11 Marja Van Waes 2023-10-30 11:11:56 CET
@ katnatek

If you assign a bug to yourself and set it to ASSIGNED at the same time that you CC your mentor and start working on it, then it is extremely unlikely someone else will also work on fixing the issue and push a fix before you and your mentor got that far.
Comment 12 katnatek 2023-10-30 17:25:50 CET
(In reply to Marja Van Waes from comment #10)
> Created attachment 14116 [details]
> output of LC_ALL=C urpmi --test task-lxqt-minimal
> 
> In cauldron, when I do:
> 
> LC_ALL=C urpmi --test task-lxqt-minimal
> 
> I do not get a choice between obconf and obconf-qt, but obconf-qt is
> selected.
> 
> Is that expected when not having Gnome installed, or should I have gotten
> the choice anyway? I do have gtk4.0 and gtk+3.0 installed (and more gtk and
> gnome stuff)

It's expected, you only get the choice when install openbox alone

 LC_ALL=C urpmi --test openbox
In order to satisfy the 'obox-conf' dependency, one of the following packages is needed:
 1- obconf-2.0.4-10.1.mga9.x86_64: Openbox preferences manager (to install)
 2- obconf-qt-0.16.2-1.1.mga9.x86_64: Openbox configuration tool for the LXQt desktop (to install)
What is your choice? (1-2) 

(In reply to Marja Van Waes from comment #11)
> @ katnatek
> 
> If you assign a bug to yourself and set it to ASSIGNED at the same time that
> you CC your mentor and start working on it, then it is extremely unlikely
> someone else will also work on fixing the issue and push a fix before you
> and your mentor got that far.

He asked me to do that, I guess for watch the progress
Comment 13 Marja Van Waes 2023-10-30 18:16:13 CET
(In reply to katnatek from comment #12)
> (In reply to Marja Van Waes from comment #10)
> > Created attachment 14116 [details]
> > output of LC_ALL=C urpmi --test task-lxqt-minimal
> > 
> > In cauldron, when I do:
> > 
> > LC_ALL=C urpmi --test task-lxqt-minimal
> > 
> > I do not get a choice between obconf and obconf-qt, but obconf-qt is
> > selected.
> > 
> > Is that expected when not having Gnome installed, or should I have gotten
> > the choice anyway? I do have gtk4.0 and gtk+3.0 installed (and more gtk and
> > gnome stuff)
> 
> It's expected, you only get the choice when install openbox alone. 

I confirm that I do get a choice when installing openbox alone.

Given that it is lxqt and not lxgtk, I don't think anyone will complain about not getting a choice when installing task-lxqt-minimal.

The description I had prepared was not correct, because I had misunderstood what the updates do.  Is this correct?:

description: |
  When installing task-lxqt-minimal, obconf (gtk) was installed instead of
  obconf-qt. This update fixes that issue.
  Apart from that, when installing openbox, it is now possible to choose 
  between obconf-qt and obconf.
Comment 14 katnatek 2023-10-30 18:20:20 CET
Created attachment 14117 [details]
Log of all my test

Test 1: install current version and update to new versions, check you can uninstall obconf

After the update you have both obconf and obconf-qt- you can uninstall obconf if you wish

Test 2: Clean Install the new versions
You only get obconf-qt

Test 3: Check install of task-lxde-minimal
Just to be sure that changes made not put noise in this

This comment is done from a lxqt session

All look good here, Test on Mageia 9 x86_64
Comment 15 katnatek 2023-10-30 18:22:54 CET
(In reply to Marja Van Waes from comment #13)
> I confirm that I do get a choice when installing openbox alone.
> 
> Given that it is lxqt and not lxgtk, I don't think anyone will complain
> about not getting a choice when installing task-lxqt-minimal.
> 
> The description I had prepared was not correct, because I had misunderstood
> what the updates do.  Is this correct?:
> 
> description: |
>   When installing task-lxqt-minimal, obconf (gtk) was installed instead of
>   obconf-qt. This update fixes that issue.
>   Apart from that, when installing openbox, it is now possible to choose 
>   between obconf-qt and obconf.

Looks good to me
katnatek 2023-10-30 18:24:34 CET

Whiteboard: (none) => MGA9-64-OK

Comment 16 Marja Van Waes 2023-10-30 18:30:31 CET
Advisory based on the SRPMs from comment 9 and the description from comment 13,  added to SVN. Please remove the "advisory" keyword if it needs to be changed. It also helps when obsolete advisories are tagged as "obsolete"

Keywords: (none) => advisory

Comment 17 katnatek 2023-10-30 19:32:03 CET
Created attachment 14118 [details]
Log of all my test on i586

Same test as in comment#14, Tested on Mageia 9 i586
All test goes as expected
This comment is done on lxqt session on Mageia 9 i586
katnatek 2023-10-30 19:35:34 CET

Whiteboard: MGA9-64-OK => MGA9-64-OK MGA9-32-OK

Comment 18 Thomas Andrews 2023-10-30 23:17:22 CET
Validating.

Keywords: (none) => validated_update
CC: (none) => andrewsfarm, sysadmin-bugs

Comment 19 Mageia Robot 2023-10-31 22:36:10 CET
An update for this issue has been pushed to the Mageia Updates repository.

https://advisories.mageia.org/MGAA-2023-0115.html

Resolution: (none) => FIXED
Status: ASSIGNED => RESOLVED


Note You need to log in before you can comment on or make changes to this bug.