Re: Phonon Backends

2023-07-21 Thread Jonathan Riddell
Indeed, we seem to have the capitalised headers but lower-case "phonon"
includes back with the changes Harald has recently made, phonon-vlc and
dolphin etc now compile again.  Qt 5 and Qt 6 now seem to need separate
compiles.

Jonathan


On Fri, 21 Jul 2023 at 10:20, Volker Krause  wrote:

> On Wednesday, 19 July 2023 16:46:55 CEST Jonathan Riddell wrote:
> > Hola, I see Volker has stopped installing the capitalised headers for
> > Phonon.
> >
> > This breaks the build of phonon-vlc.  Are there any plans to update it?
>
> phonon-vlc uses lower-case namespace includes, so I rather suspect this is
> unintended fallout from https://invent.kde.org/libraries/phonon/-/
> merge_requests/14
>  which
> should have been fixed meanwhile.
>
> Adding a Qt6 CI job for phonon-vlc would also help I guess.
>
> Regards,
> Volker


Re: Phonon Backends

2023-07-21 Thread Volker Krause
On Wednesday, 19 July 2023 16:46:55 CEST Jonathan Riddell wrote:
> Hola, I see Volker has stopped installing the capitalised headers for
> Phonon.
>
> This breaks the build of phonon-vlc.  Are there any plans to update it?

phonon-vlc uses lower-case namespace includes, so I rather suspect this is 
unintended fallout from https://invent.kde.org/libraries/phonon/-/
merge_requests/14 which should have been fixed meanwhile.

Adding a Qt6 CI job for phonon-vlc would also help I guess.

Regards,
Volker

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