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: Proposal for using gitlab for kdereview process

2023-06-19 Thread Jonathan Riddell
Seems using issues is more intuitive so I'll ask for a global group label
for issues and update the process docs for kdereview and incubator.

Incubator suggestion is here
https://discuss.kde.org/t/proposal-kde-invent-based-incubator-process/

Jonathan


On Wed, 1 Mar 2023 at 08:27, David Redondo  wrote:

> So there are two new kdereview requests using gitlab issues now instead of
> my
> more complicated idea of merge requests. I imagined a MR to have
> advantages
> for codereview but an issue seems to be less complicated (no need to have
> an
> empty orphan branch, etc..), so maybe we should go with that.
>
> David
>
>
>


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.