Bug 26787 - Mssing xscreensaver if only LXDE is installed
Summary: Mssing xscreensaver if only LXDE is installed
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Installer (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: ISO building group
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-13 09:23 CEST by J Scheurich
Modified: 2020-06-19 20:28 CEST (History)
4 users (show)

See Also:
Source RPM: lxde
CVE:
Status comment:


Attachments

Description J Scheurich 2020-06-13 09:23:30 CEST
Description of problem:
Mssing xscreensaver if lxde is installed
If the lxde variant is installed, panel start button -> logout -> lock screen
is not working till the xscreensaver pacage is amnually installed.

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

How reproducible:

Install magaia 7, select LXDE desktop
loggin and try  panel start button -> logout -> lock screen nothing happens,
shuld start a screen locker...
Comment 1 J Scheurich 2020-06-13 09:25:17 CEST
Version-Release number of selected component (if applicable):
Mageai 7
Comment 2 Lewis Smith 2020-06-13 21:29:14 CEST
Thank you for reporting this. A little background, please.
Can you say how you installed Mageia - from the 7.1 Classic ISO, or netinstall.
If the former, did you choose *only* LXDE?
Did you update the system either during installation, or on re-booting?

It has to be said that nothing can be done about the 7.1 installer since its release. But this information may be useful for the forthcoming Mageia 8, and the bug remains a useful reference. I will ask for a check.

CC: (none) => lewyssmith

Comment 3 J Scheurich 2020-06-14 06:22:08 CEST
> did you choose *only* LXDE?

I choose only LXDE.

> Can you say how you installed Mageia

From a 4GB ISO, not from the netinstall.
I ttied the netinstall first, but run into problems with my USB-Wlan Adapter

Product: 802.11n WLAN Adapter
[    3.352515] usb 1-1.2: Manufacturer: Realtek
Comment 4 Lewis Smith 2020-06-14 20:22:45 CEST
Thank you for the clarification.
Because a 7.1 installer bug is un-fixable, please accept apologies for closing this; take wontFix as cannotFix.
I have noted this fault in the Mageia 7 Errata notes:
 https://wiki.mageia.org/en/Mageia_7_Errata#LXDE_screensaver.2Flocker

Resolution: (none) => WONTFIX
Status: NEW => RESOLVED

Comment 5 Olav Vitters 2020-06-15 15:59:28 CEST
I'm not following why xscreensaver is only in tainted. A quick check shows that xscreensaver has some copyrighted images related to the Matrix screensaver. The package does allow building it for the normal Cauldron section. Loads of distributions include a normal version of xscreensaver.

I think this could be fixed: provide two versions of xscreenaver (trained, non tainted), then add a dependency (recommends/requires/suggests) from some lxde package to xscreensaver.

Reopening, feel free to wontfix again, but think more can be done here.

Resolution: WONTFIX => (none)
Status: RESOLVED => REOPENED
CC: (none) => olav

Comment 6 Nicolas Salguero 2020-06-15 16:22:29 CEST
Hi,

I checked again and I am sure that xscreensaver is already pushed to core and tainted.

Moreover, lxde-common recommends xscreensaver but I think that the Mageia 7 ISO was too big and xscreensaver was removed from the ISO for that.

Best regards,

Nico.

CC: (none) => nicolas.salguero

Comment 7 Olav Vitters 2020-06-15 16:53:57 CEST
Argh, indeed. I used urpmq to check this, it only showed tainted. I expected it to show a result per media. Maybe also good to add this to meta-task rpmsrate-raw. Potentially we could maybe provide an update, though this might be a bit excessive.
Comment 8 Lewis Smith 2020-06-16 11:02:43 CEST
I also thought that urpmq would show all available versions; in fact it *does*:
 $ urpmq -i xscreensaver
 xscreensaver-5.42-1.mga7.src.rpm
 xscreensaver-5.42-1.mga7.tainted.src.rpm
So I simply did not check the O/P suffciently, just the last item; and my Wiki comment about it being in 'tainted' was wrong - someone has already corrected it. Thanks.

> Olav:
> I think this could be fixed: provide two versions of xscreenaver (trained,
> non tainted), then add a dependency (recommends/requires/suggests) from
> some lxde package to xscreensaver.
> ---
> Nicolas:
> I checked again and I am sure that xscreensaver is already pushed to core
> and tainted.
> lxde-common recommends xscreensaver
> the Mageia 7 ISO was too big and xscreensaver was removed from the ISO for that
So I think we can safely say the problem is fixed! Nicolas' last remark explains everything.

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

Comment 9 Lewis Smith 2020-06-18 12:04:28 CEST
Ah.
I have just installed Mageia 8 alpha 1 Classic x64 ISO, choosing *only* LXDE desktop, default application groups; and...
On the installed system, 'xscreensaver' is *not* installed; but 'xdg-screensaver' is (from pkg 'xdg-utils'). This is a command line tool, not the same thing.

I can find no mention in the Tools menus (Desktop Preferences, Desktop Session Settings, or anything else) of establishing a screensaver. Perhaps this is just me v LXDE.
----------
To test that last point, I have just installed xscreensaver, which pulled in other things:
  chbg                           2.0.1        27.mga8       x86_64  
  fortune-mod                    2.28.0       1.mga8        x86_64  
  lib64glade2.0_0                2.6.4        14.mga8       x86_64  
  xscreensaver                   5.44         1.mga8        x86_64  
  xscreensaver-common            5.44         1.mga8        x86_64  
which are worth 2.8MB re comment 6.
And to invoke it, it has its own independent Tools menu entry.

If there are other applications we could ditch from the ISO to make space, I suggest a post on qa-discuss for candidates. I vote immediately for Dasher, for example (nothing against it).

Assignee: bugsquad => isobuild
Hardware: i586 => All
Resolution: FIXED => (none)
Status: RESOLVED => REOPENED
Version: 7 => Cauldron
Summary: Mssing xscreensaver if lxde is installed => Mssing xscreensaver if only LXDE is installed

Comment 10 Martin Whitaker 2020-06-18 13:36:19 CEST
xscreensaver was not removed from the ISO to save space. It is no longer included because it is not mentioned in /usr/share/meta-task/rpmsrate-raw and no package that is included requires it. The ISO build process does not take into account packages that are merely recommended.

In mga5, xfce4-settings required xscreensaver, and that was enough to get it included on the ISO. That requirement was gone in mga6.

If you want xscreensaver to be included, you either need to make it a hard requirement in task-lxde or add it to /usr/share/meta-task/rpmsrate-raw.

CC: (none) => mageia

Comment 11 Lewis Smith 2020-06-19 20:28:35 CEST
(In reply to Martin Whitaker from comment #10)
Thanks for your reply.
> If you want xscreensaver to be included, you either need to make it a hard
> requirement in task-lxde or add it to /usr/share/meta-task/rpmsrate-raw.
I imagine there is a way of making it a requirement of 'task-lxde' if no other screensaver is already installed; otherwise a hard requirement.
---------
As so often, DavidG has pre-empted action, in response to the initial complaint comment 0:-
task-lxde, Sat Jun 13 2020
- add missing recommended package on xscreensaver (mga#26787)

It would have helped to say so, David... mais merci beaucoup.

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


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