Re: XWayland Video Bridge in kdereview

2023-07-04 Thread Neal Gompa
On Tue, Jul 4, 2023 at 7:12 PM Aleix Pol  wrote:
>
> I'm not so sure. Is the fact that it works on a non-Plasma system by
> design or just a happy coincidence?
>

The initial blog post about it implies it's supposed to be by design
even though it wasn't explicitly tested:
https://blog.davidedmundson.co.uk/blog/xwaylandvideobridge/




--
真実はいつも一つ!/ Always, there's only one truth!


Re: XWayland Video Bridge in kdereview

2023-07-04 Thread Aleix Pol
I'm not so sure. Is the fact that it works on a non-Plasma system by
design or just a happy coincidence?

On Tue, Jul 4, 2023 at 3:03 PM Neal Gompa  wrote:
>
> On Fri, Jun 30, 2023 at 7:09 AM Ben Cooksley  wrote:
> >
> > On Fri, Jun 30, 2023 at 10:01 AM Aleix Pol  wrote:
> >>
> >> On Mon, Jun 19, 2023 at 10:00 PM Jonathan Riddell  
> >> wrote:
> >> >
> >> > I got this compiling on neon unstable.  Our kpipewire is qt6 so I'm not 
> >> > sure how that affects it.  I don't have a way to test it quickly though.
> >> > https://build.neon.kde.org/job/jammy_unstable_neon-packaging_xwaylandvideobridge/
> >>
> >> The Qt 6 port is still ongoing:
> >> https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/11
> >>
> >> > How come it's in the system group on invent if you want it to be part of 
> >> > Plasma?
> >>
> >> To be honest, the invent groups are a mystery to me.
> >
> >
> > That is of some degree of concern that you don't understand the groupings.
> >
> > For the most part the groups are intended to reflect to an extent how the 
> > software relates to each other. You will find all of the Educational 
> > software under Education for instance, and all Games in Games.
> > Likewise, applications for handling images and other similar content are in 
> > Graphics, those that deal with video and audio are in Multimedia and 
> > libraries for use by other developers are in Libraries.
> >
> > All fairly straightforward I would have hoped.
> >
> > The reason for the placement of the XWayland Video Bridge comes back to the 
> > initial ticket that requested it be moved from a personal repository - 
> > which you filed Aleix.
> > This was targeting Graphics, which is clearly inappropriate as it would 
> > have put it alongside Krita and Digikam. For want of a better place as it 
> > was a bit of infrastructure / behind the scenes tooling it was therefore 
> > put in System.
> >
> > There was never any mention in the ticket that it was going to be exclusive 
> > to Plasma and not usable outside of a Plasma setting, hence why that was 
> > not considered.
> >
>
> It is supposed to be useful outside of KDE Plasma too, so that
> assumption is correct.
>
>
> --
> 真実はいつも一つ!/ Always, there's only one truth!


Re: XWayland Video Bridge in kdereview

2023-07-04 Thread Neal Gompa
On Fri, Jun 30, 2023 at 7:09 AM Ben Cooksley  wrote:
>
> On Fri, Jun 30, 2023 at 10:01 AM Aleix Pol  wrote:
>>
>> On Mon, Jun 19, 2023 at 10:00 PM Jonathan Riddell  wrote:
>> >
>> > I got this compiling on neon unstable.  Our kpipewire is qt6 so I'm not 
>> > sure how that affects it.  I don't have a way to test it quickly though.
>> > https://build.neon.kde.org/job/jammy_unstable_neon-packaging_xwaylandvideobridge/
>>
>> The Qt 6 port is still ongoing:
>> https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/11
>>
>> > How come it's in the system group on invent if you want it to be part of 
>> > Plasma?
>>
>> To be honest, the invent groups are a mystery to me.
>
>
> That is of some degree of concern that you don't understand the groupings.
>
> For the most part the groups are intended to reflect to an extent how the 
> software relates to each other. You will find all of the Educational software 
> under Education for instance, and all Games in Games.
> Likewise, applications for handling images and other similar content are in 
> Graphics, those that deal with video and audio are in Multimedia and 
> libraries for use by other developers are in Libraries.
>
> All fairly straightforward I would have hoped.
>
> The reason for the placement of the XWayland Video Bridge comes back to the 
> initial ticket that requested it be moved from a personal repository - which 
> you filed Aleix.
> This was targeting Graphics, which is clearly inappropriate as it would have 
> put it alongside Krita and Digikam. For want of a better place as it was a 
> bit of infrastructure / behind the scenes tooling it was therefore put in 
> System.
>
> There was never any mention in the ticket that it was going to be exclusive 
> to Plasma and not usable outside of a Plasma setting, hence why that was not 
> considered.
>

It is supposed to be useful outside of KDE Plasma too, so that
assumption is correct.


-- 
真実はいつも一つ!/ Always, there's only one truth!


Re: XWayland Video Bridge in kdereview

2023-07-04 Thread Jonathan Riddell
I opened an issue in line with the new process

https://invent.kde.org/system/xwaylandvideobridge/-/issues/2



On Fri, 16 Jun 2023 at 17:44, Aleix Pol  wrote:

> Hi Everyone,
> As discussed by david in his blog post [1], we worked on the component
> in question to solve the problem of X11 applications that want to use
> the screencasting services on Wayland.
>
> https://invent.kde.org/system/xwaylandvideobridge
>
> While there's some corners to polish here and there, it's ready to be
> used by everyone who needs it. In fact, people are using it from the
> flatpak and distros want to ship it so we should probably make it
> easier and regulated through a proper release.
>
> First it should go through kdereview, then I'd suggest starting to
> release it with Plasma 5.27 in one of the minor releases. I don't
> think it's something we have done before but it doesn't add new
> dependencies and it has an impact in our product offering which is,
> incidentally, frozen due to the switch to *6.
>
> I would appreciate the review of the module, I guess the "how to
> release it" is something we can discuss in a couple of weeks once this
> is sorted.
>
> Thanks everyone!
> Aleix
>
> [1] https://blog.davidedmundson.co.uk/blog/xwaylandvideobridge/
>


Re: XWayland Video Bridge in kdereview

2023-06-30 Thread Ben Cooksley
On Fri, Jun 30, 2023 at 10:01 AM Aleix Pol  wrote:

> On Mon, Jun 19, 2023 at 10:00 PM Jonathan Riddell  wrote:
> >
> > I got this compiling on neon unstable.  Our kpipewire is qt6 so I'm not
> sure how that affects it.  I don't have a way to test it quickly though.
> >
> https://build.neon.kde.org/job/jammy_unstable_neon-packaging_xwaylandvideobridge/
>
> The Qt 6 port is still ongoing:
> https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/11
>
> > How come it's in the system group on invent if you want it to be part of
> Plasma?
>
> To be honest, the invent groups are a mystery to me.
>

That is of some degree of concern that you don't understand the groupings.

For the most part the groups are intended to reflect to an extent how the
software relates to each other. You will find all of the Educational
software under Education for instance, and all Games in Games.
Likewise, applications for handling images and other similar content are in
Graphics, those that deal with video and audio are in Multimedia and
libraries for use by other developers are in Libraries.

All fairly straightforward I would have hoped.

The reason for the placement of the XWayland Video Bridge comes back to the
initial ticket that requested it be moved from a personal repository -
which you filed Aleix.
This was targeting Graphics, which is clearly inappropriate as it would
have put it alongside Krita and Digikam. For want of a better place as it
was a bit of infrastructure / behind the scenes tooling it was therefore
put in System.

There was never any mention in the ticket that it was going to be exclusive
to Plasma and not usable outside of a Plasma setting, hence why that was
not considered.


>
> Aleix
>

Cheers,
Ben


Re: XWayland Video Bridge in kdereview

2023-06-29 Thread Aleix Pol
On Mon, Jun 19, 2023 at 10:00 PM Jonathan Riddell  wrote:
>
> I got this compiling on neon unstable.  Our kpipewire is qt6 so I'm not sure 
> how that affects it.  I don't have a way to test it quickly though.
> https://build.neon.kde.org/job/jammy_unstable_neon-packaging_xwaylandvideobridge/

The Qt 6 port is still ongoing:
https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/11

> How come it's in the system group on invent if you want it to be part of 
> Plasma?

To be honest, the invent groups are a mystery to me.

Aleix


Re: XWayland Video Bridge in kdereview

2023-06-19 Thread Jonathan Riddell
I got this compiling on neon unstable.  Our kpipewire is qt6 so I'm not
sure how that affects it.  I don't have a way to test it quickly though.
https://build.neon.kde.org/job/jammy_unstable_neon-packaging_xwaylandvideobridge/

How come it's in the system group on invent if you want it to be part of
Plasma?

Jonathan



On Fri, 16 Jun 2023 at 17:44, Aleix Pol  wrote:

> Hi Everyone,
> As discussed by david in his blog post [1], we worked on the component
> in question to solve the problem of X11 applications that want to use
> the screencasting services on Wayland.
>
> https://invent.kde.org/system/xwaylandvideobridge
>
> While there's some corners to polish here and there, it's ready to be
> used by everyone who needs it. In fact, people are using it from the
> flatpak and distros want to ship it so we should probably make it
> easier and regulated through a proper release.
>
> First it should go through kdereview, then I'd suggest starting to
> release it with Plasma 5.27 in one of the minor releases. I don't
> think it's something we have done before but it doesn't add new
> dependencies and it has an impact in our product offering which is,
> incidentally, frozen due to the switch to *6.
>
> I would appreciate the review of the module, I guess the "how to
> release it" is something we can discuss in a couple of weeks once this
> is sorted.
>
> Thanks everyone!
> Aleix
>
> [1] https://blog.davidedmundson.co.uk/blog/xwaylandvideobridge/
>


Re: XWayland Video Bridge in kdereview

2023-06-19 Thread Jonathan Riddell
I don't want to add it to the Plasma release for a bugfix version.  But
I'll happily release it alongside it.

Jonathan


On Mon, 19 Jun 2023 at 09:48, Sandro Knauß  wrote:

> Hey,
>
> > First it should go through kdereview, then I'd suggest starting to
> > release it with Plasma 5.27 in one of the minor releases. I don't
> > think it's something we have done before but it doesn't add new
> > dependencies and it has an impact in our product offering which is,
> > incidentally, frozen due to the switch to *6.
>
> With my Debian hat on: Please make sure nothing depend on this new
> application
> for 5.27.
> Background: We successfully could convinced the Debian release team, that
> it
> is safe to ship new minor Plasma releases to Debian stable with the result
> shipping 5.27.5 in bookworm (released some days ago). The main reason was
> that
> the minor releases are only bugfix releases and no new features are added.
> If you do not break this rule, that we can ship the minor releases to
> bookworm
> (without XWayland Video Bridge, as no new applications are allowed).
>
> Regards,
>
> hefee
>
>
>


Re: XWayland Video Bridge in kdereview

2023-06-19 Thread Sandro Knauß
Hey,

> First it should go through kdereview, then I'd suggest starting to
> release it with Plasma 5.27 in one of the minor releases. I don't
> think it's something we have done before but it doesn't add new
> dependencies and it has an impact in our product offering which is,
> incidentally, frozen due to the switch to *6.

With my Debian hat on: Please make sure nothing depend on this new application 
for 5.27. 
Background: We successfully could convinced the Debian release team, that it 
is safe to ship new minor Plasma releases to Debian stable with the result 
shipping 5.27.5 in bookworm (released some days ago). The main reason was that 
the minor releases are only bugfix releases and no new features are added. 
If you do not break this rule, that we can ship the minor releases to bookworm 
(without XWayland Video Bridge, as no new applications are allowed).

Regards,

hefee




signature.asc
Description: This is a digitally signed message part.


Re: XWayland Video Bridge in kdereview

2023-06-16 Thread Aleix Pol
On Fri, Jun 16, 2023 at 10:31 PM Albert Astals Cid  wrote:
>
> El divendres, 16 de juny de 2023, a les 18:43:59 (CEST), Aleix Pol va
> escriure:
> > Hi Everyone,
> > As discussed by david in his blog post [1], we worked on the component
> > in question to solve the problem of X11 applications that want to use
> > the screencasting services on Wayland.
> >
> > https://invent.kde.org/system/xwaylandvideobridge
> >
> > While there's some corners to polish here and there, it's ready to be
> > used by everyone who needs it. In fact, people are using it from the
> > flatpak and distros want to ship it so we should probably make it
> > easier and regulated through a proper release.
>
> CI seems borked
>
> https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/9
>
> Cheers,
>   Albert
>
> >
> > First it should go through kdereview, then I'd suggest starting to
> > release it with Plasma 5.27 in one of the minor releases. I don't
> > think it's something we have done before but it doesn't add new
> > dependencies and it has an impact in our product offering which is,
> > incidentally, frozen due to the switch to *6.
> >
> > I would appreciate the review of the module, I guess the "how to
> > release it" is something we can discuss in a couple of weeks once this
> > is sorted.
> >
> > Thanks everyone!
> > Aleix
> >
> > [1] https://blog.davidedmundson.co.uk/blog/xwaylandvideobridge/

Addressed, thanks!

https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/10

Aleix


Re: XWayland Video Bridge in kdereview

2023-06-16 Thread Albert Astals Cid
El divendres, 16 de juny de 2023, a les 18:43:59 (CEST), Aleix Pol va 
escriure:
> Hi Everyone,
> As discussed by david in his blog post [1], we worked on the component
> in question to solve the problem of X11 applications that want to use
> the screencasting services on Wayland.
> 
> https://invent.kde.org/system/xwaylandvideobridge
> 
> While there's some corners to polish here and there, it's ready to be
> used by everyone who needs it. In fact, people are using it from the
> flatpak and distros want to ship it so we should probably make it
> easier and regulated through a proper release.

CI seems borked 

https://invent.kde.org/system/xwaylandvideobridge/-/merge_requests/9

Cheers,
  Albert

> 
> First it should go through kdereview, then I'd suggest starting to
> release it with Plasma 5.27 in one of the minor releases. I don't
> think it's something we have done before but it doesn't add new
> dependencies and it has an impact in our product offering which is,
> incidentally, frozen due to the switch to *6.
> 
> I would appreciate the review of the module, I guess the "how to
> release it" is something we can discuss in a couple of weeks once this
> is sorted.
> 
> Thanks everyone!
> Aleix
> 
> [1] https://blog.davidedmundson.co.uk/blog/xwaylandvideobridge/