Bug 5838

Summary: Video starts in an external window under Wayland
Product: Fedora Reporter: Gerald Cox <gbcox>
Component: smplayerAssignee: Sérgio Basto <sergio>
Status: RESOLVED EXPIRED    
Severity: enhancement CC: fedora, leigh123linux
Priority: P1    
Version: unspecified   
Hardware: x86_64   
OS: GNU/Linux   
namespace:

Description Gerald Cox 2020-11-16 03:32:23 CET
Using mpv with smplayer.  Testing and reporting bugs under plasma-wayland since the Fedora KDE team is wanting to make wayland the default.  

When playing a video under wayland, a external window is opened instead of playing the video in the main control window as in does under X.
Comment 1 Gerald Cox 2020-11-16 06:26:33 CET
There is an open enhancement for this upstream here:

https://sourceforge.net/p/smplayer/feature-requests/766/
Comment 2 Sérgio Basto 2020-11-21 20:53:36 CET
for me it is an kwayland problem , dnf remove kwin-wayland should fix the problem
Comment 3 Gerald Cox 2020-11-21 21:04:16 CET
(In reply to Sérgio Basto from comment #2)
> for me it is an kwayland problem , dnf remove kwin-wayland should fix the
> problem

Really Sergio?  You know (or should know) that there is an active proposal to make Wayland the Plasma default in F34.  If not, here it is:

https://fedoraproject.org/wiki/Changes/WaylandByDefaultForPlasma

What kind of response is that from a Fedora Contributor?
Comment 4 leigh scott 2020-12-27 09:55:50 CET
(In reply to Gerald Cox from comment #3)
> (In reply to Sérgio Basto from comment #2)
> > for me it is an kwayland problem , dnf remove kwin-wayland should fix the
> > problem
> 
> Really Sergio?  You know (or should know) that there is an active proposal
> to make Wayland the Plasma default in F34.  If not, here it is:

We aren't going to fix wayland induced issues, if you want it fixed report it to smplayer upstream.

> 
> https://fedoraproject.org/wiki/Changes/WaylandByDefaultForPlasma
> 
> What kind of response is that from a Fedora Contributor?

We aren't going to fix wayland induced issues, if you want it fixed report it to smplayer upstream.


We are here to fix packaging issues, not issues caused by wayland.
Comment 5 Sérgio Basto 2020-12-27 18:26:44 CET
IMHO , the bugs should be filed against kwayland and they (who want) run it by default, should look for the fixes, not us. 

Also, I'm against add workarounds on desktop files like this one [1] forcing apps running with QT_QPA_PLATFORM=xcb , because is just hidden the real problem. 

IMHO, try run it in wayland should be a optin . 

Thank you for the report , the bug may remain open until someone fix it , i.e. or smplayers developer or kwayland developers, this information can be useful for other users. 
Note upstream app developers can opt by not support wayland and internally force use of X11, but we shouldn't do that on desktop files because app could be called  by other method ... 

 
[1] 
https://pkgs.rpmfusion.org/cgit/free/shotcut.git/commit/?id=72db606a7e307e512124af3f8d3c27970b462996

https://pkgs.rpmfusion.org/cgit/free/shotcut.git/tree/Force_X.patch
Comment 6 Gerald Cox 2020-12-27 20:13:04 CET
(In reply to leigh scott from comment #4)
> (In reply to Gerald Cox from comment #3)
> > (In reply to Sérgio Basto from comment #2)
> > > for me it is an kwayland problem , dnf remove kwin-wayland should fix the
> > > problem
> > 
> > Really Sergio?  You know (or should know) that there is an active proposal
> > to make Wayland the Plasma default in F34.  If not, here it is:
> 
> We aren't going to fix wayland induced issues, if you want it fixed report
> it to smplayer upstream.

Did you read comment #1?  I reported it upstream.

> > 
> > https://fedoraproject.org/wiki/Changes/WaylandByDefaultForPlasma
> > 
> > What kind of response is that from a Fedora Contributor?
> 
> We aren't going to fix wayland induced issues, if you want it fixed report
> it to smplayer upstream.
> 
> 
> We are here to fix packaging issues, not issues caused by wayland.

Did you read comment #3?

This has become a packaging issue because of the stated Fedora direction.  Of course, I'm not expecting you to fix the issue, but as packagers you need to be aware that one of your packages is no longer going to work properly in the near future with the default KDE environment.
Comment 7 Gerald Cox 2020-12-27 20:37:38 CET
(In reply to Sérgio Basto from comment #5)
> IMHO , the bugs should be filed against kwayland and they (who want) run it
> by default, should look for the fixes, not us. 

I'm not asking you to look for fixes.  I filed this ticket so:
1.  People who experience this issue are aware it is a known issue and has been reported.
2.  Notify the package maintainers so they understand the issue and know it has been reported upstream, so they could communicate with upstream as they thought appropriate.
3.  Make sure you are aware that the plan is to make Wayland the default KDE environment for F34.

> 
> Also, I'm against add workarounds on desktop files like this one [1] forcing
> apps running with QT_QPA_PLATFORM=xcb , because is just hidden the real
> problem. 
> 
> IMHO, try run it in wayland should be a optin . 
> 
> Thank you for the report , the bug may remain open until someone fix it ,
> i.e. or smplayers developer or kwayland developers, this information can be
> useful for other users. 
> Note upstream app developers can opt by not support wayland and internally
> force use of X11, but we shouldn't do that on desktop files because app
> could be called  by other method ... 
> 
>  
> [1] 
> https://pkgs.rpmfusion.org/cgit/free/shotcut.git/commit/
> ?id=72db606a7e307e512124af3f8d3c27970b462996
> 
> https://pkgs.rpmfusion.org/cgit/free/shotcut.git/tree/Force_X.patch

You folks are completely missing the point and are being needlessly defensive.  Read my responses to Leigh Scott in comment #6.  

Unless this gets fixed somehow, people are going to be complaining when F34 becomes available, and I thought you might want to be aware, an apparent misjudgement on my part.  A good alternative which I've been testing which works great with wayland is celluloid - although it is GTK based and not Qt.  It is also available in rpmfusion.
Comment 8 leigh scott 2020-12-28 00:18:57 CET
(In reply to Gerald Cox from comment #7)

> > https://pkgs.rpmfusion.org/cgit/free/shotcut.git/tree/Force_X.patch
> 
> You folks are completely missing the point and are being needlessly
> defensive.  Read my responses to Leigh Scott in comment #6.  
> 
> Unless this gets fixed somehow, people are going to be complaining when F34
> becomes available, and I thought you might want to be aware, an apparent
> misjudgement on my part.  A good alternative which I've been testing which
> works great with wayland is celluloid - although it is GTK based and not Qt.
> It is also available in rpmfusion.

We have been aware of the wayland induced issues for a couple of years.
The fedora kde/qt team are aware of the issues wayland causes and just blame the apps.

https://lists.fedoraproject.org/archives/list/kde@lists.fedoraproject.org/thread/XTR7O3X2OSDBXRJBILL7B7NBAX3564EQ/#DJS7A2RWFIQSRJBV66M63QGJDJLKKEK5
Comment 9 Gerald Cox 2020-12-28 01:17:49 CET
(In reply to leigh scott from comment #8)
> 
> We have been aware of the wayland induced issues for a couple of years.
> The fedora kde/qt team are aware of the issues wayland causes and just blame
> the apps.
> 
> https://lists.fedoraproject.org/archives/list/kde@lists.fedoraproject.org/
> thread/XTR7O3X2OSDBXRJBILL7B7NBAX3564EQ/#DJS7A2RWFIQSRJBV66M63QGJDJLKKEK5

And now the departure time has been set, and the train is leaving the station.
Comment 10 leigh scott 2020-12-28 05:42:55 CET
(In reply to Gerald Cox from comment #9)

> And now the departure time has been set, and the train is leaving the
> station.


We think their train sucks and plan to stay at home instead.
Comment 11 Sérgio Basto 2020-12-28 11:48:10 CET
Today, I started to read this thread : https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/AERHGVALKUE6UDU3PP35XV7G5DMEX7RM/

I will write there , to try to revert "Wayland by Default for KDE Plasma Desktop" 
seems almost all the people have the same opinion ... that Plasma on Wayland is not ready for production

Thank you
Comment 12 Sérgio Basto 2021-01-17 12:16:46 CET
Smplayer 21.1.0 is in updates-testing , readme says [1] , can you test smplayer on wayland ?

Thank you 


[1] 
Version 21.1:

 * Initial support for Wayland.
 * Some fixes and improvements in the playlist.
 * Show cache info in the OSD (Shift + I).
 * New skin Mint-Y.
Comment 13 Gerald Cox 2021-01-17 20:24:10 CET
(In reply to Sérgio Basto from comment #12)
> Smplayer 21.1.0 is in updates-testing , readme says [1] , can you test
> smplayer on wayland ?
> 
> Thank you 
> 
> 
> [1] 
> Version 21.1:
> 
>  * Initial support for Wayland.
>  * Some fixes and improvements in the playlist.
>  * Show cache info in the OSD (Shift + I).
>  * New skin Mint-Y.

Thanks for taking the time to build and put out on testing.  I had built a copy the day it was published and it didn't work at all - so I thought I had done something wrong on the build.  Your build confirms I didn't.  It just sits there without doing anything on Wayland Gnome and Wayland Plasma.  Plasma-X11 seems to work fine though.  I'll report upstream.  It's not an emergency, people can just use Celluloid.  In all my testing, it appears to work fine, with no issues.
Comment 14 Sérgio Basto 2021-11-28 13:56:21 CET
is not fixed , smplayer doesn't work with Wayland
Comment 15 leigh scott 2024-05-03 10:56:53 CEST
smplayer has moved to fedora repo