Re: phonon-backend-vlc for EL7

2017-02-27 Thread Nicolas Chauvet
2017-02-27 12:14 GMT+01:00 Kevin Kofler :
> Nicolas Chauvet wrote:
>> The split with vlc-core isn't about binaries vs libraries, but with
>> headless/server/core vs wayland/Xorg/display.
>
> … which is a split whose usefulness I doubt. All the applications with
> associated library are split into app and app-libs, not app (GUI) and
> app-core.
I don't buy this, so I'm not reading this message any further based on
your false premises.
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


Re: phonon-backend-vlc for EL7

2017-02-27 Thread Kevin Kofler
Nicolas Chauvet wrote:
> The split with vlc-core isn't about binaries vs libraries, but with
> headless/server/core vs wayland/Xorg/display.

… which is a split whose usefulness I doubt. All the applications with 
associated library are split into app and app-libs, not app (GUI) and
app-core.

> What you suggest is to add libvlc and libvlc-core to vlc and vlc-core.

Yes, if you want to keep the -core split, we need a 4-way split, I have been 
saying that for months.

See how other applications that can be embedded as a library are split, 
e.g.:

http://pkgs.fedoraproject.org/cgit/rpms/avogadro.git/tree/avogadro.spec
(avogadro-libs)

http://pkgs.fedoraproject.org/cgit/rpms/marble.git/tree/marble.spec
(marble-widget-qt5)

http://pkgs.fedoraproject.org/cgit/rpms/konsole5.git/tree/konsole5.spec
(konsole5-part)

Even if the main package is small, it still makes sense to not drag in end 
user applications as dependencies of other applications.

> That's not implemented, and IMO it doesn't worth either. What would be
> the benefit not to rely on vlc itself ?
> - not having the vlc desktop file?
> - not having vlc binaries ?
> - not having the qt4/qt5/skin2 vlc gui interfaces ?

All of the above. A Phonon backend should not drag in a media player UI. 
Kaffeine should also not drag in the VLC application that serves the same 
purpose and clutters the menu.

> I all theses cases, it could be to minimize the dependencies cost more
> than additional size of few unneeded binaries. And speaking about
> phonon, you should already have the qt4/qt5 libs.
> But dependencies size matters most for embedeed/server use case, not
> workstation. So it's doesn't bring any value and just make things more
> complex to deal with.

It brings the value of clean packaging and uncluttered menus.

Kevin Kofler
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


Re: phonon-backend-vlc for EL7

2017-02-27 Thread Nicolas Chauvet
2017-02-24 1:20 GMT+01:00 Orion Poplawski <or...@cora.nwra.com>:
> On 02/23/2017 04:38 PM, Orion Poplawski wrote:
>> I've taken a look at building phonon-backend-vlc for EL7.  The version that 
>> is
>> in the el7 branch in git is 0.7.1-1, which does not build because it requires
>> phonon 4.7.0, and EL7 has phonon 4.6.0.  I was able to build 0.6.2-1 fine.
>>
>> Is there any objections to me reverting to 0.6.2-1 in git?  (I suppose 
>> someone
>> with right access to the git repo could reset the el7 branch to that, but not
>> sure it's worth it).
>>
>> Also, while trying it out via dragon player in an X2Go session I'm getting:
>>
>> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
>> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
>> [7f21a80a84b8] core video output error: video output creation failed
>> [7f21c0019cc8] core decoder error: failed to create video output
>>
>> If anyone has any ideas that, that would be helpful.
>>
>
> Okay, I figured out these errors - it's because vlc needs the basic libxcb_*
> video_output plugins, but for some reason they are in the main vlc package
> rather than vlc-core.  This doesn't make since to me since vlc requires
> vlc-core.  I don't want the vlc binaries, but the libraries/plugins needed for
> playback.
The split with vlc-core isn't about binaries vs libraries, but with
headless/server/core vs wayland/Xorg/display.
What you suggest is to add libvlc and libvlc-core to vlc and vlc-core.

That's not implemented, and IMO it doesn't worth either. What would be
the benefit not to rely on vlc itself ?
- not having the vlc desktop file?
- not having vlc binaries ?
- not having the qt4/qt5/skin2 vlc gui interfaces ?
I all theses cases, it could be to minimize the dependencies cost more
than additional size of few unneeded binaries. And speaking about
phonon, you should already have the qt4/qt5 libs.
But dependencies size matters most for embedeed/server use case, not
workstation. So it's doesn't bring any value and just make things more
complex to deal with.

As far as I'm concerned as vlc maintainer, I won't create this change,
but I don't want to prevent anyone to work on such patch.
I might or might not apply it based on either it will be much more
complex to maintain.

Thx for your understanding.

-- 
-

Nicolas (kwizart)
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


Re: phonon-backend-vlc for EL7

2017-02-27 Thread Nicolas Chauvet
2017-02-24 0:38 GMT+01:00 Orion Poplawski <or...@cora.nwra.com>:
> I've taken a look at building phonon-backend-vlc for EL7.  The version that is
> in the el7 branch in git is 0.7.1-1, which does not build because it requires
> phonon 4.7.0, and EL7 has phonon 4.6.0.  I was able to build 0.6.2-1 fine.
>
> Is there any objections to me reverting to 0.6.2-1 in git?  (I suppose someone
> with right access to the git repo could reset the el7 branch to that, but not
> sure it's worth it).

Best way is to cut-off the changelog and revert back to the version
you need (along with cherry-pick the commit from master), then commit
for el7.
Rewriting history doesn't worth it. None (from pkgdb based acl) is
allowed to do that (in either RPM Fusion or even Fedora ). This would
requires a manual tweak of the auto-generated gitolite.conf to do that
along with tweaking the bare git module configuration file.

It's disallowed in Fedora, we don't permit that in RPM Fusion either.

-- 
-

Nicolas (kwizart)
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


Re: phonon-backend-vlc for EL7

2017-02-24 Thread Xavier Bachelot
On 24/02/2017 01:20, Orion Poplawski wrote:
> On 02/23/2017 04:38 PM, Orion Poplawski wrote:
>> I've taken a look at building phonon-backend-vlc for EL7.  The version that 
>> is
>> in the el7 branch in git is 0.7.1-1, which does not build because it requires
>> phonon 4.7.0, and EL7 has phonon 4.6.0.  I was able to build 0.6.2-1 fine.
>>
>> Is there any objections to me reverting to 0.6.2-1 in git?  (I suppose 
>> someone
>> with right access to the git repo could reset the el7 branch to that, but not
>> sure it's worth it).
>>
>> Also, while trying it out via dragon player in an X2Go session I'm getting:
>>
>> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
>> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
>> [7f21a80a84b8] core video output error: video output creation failed
>> [7f21c0019cc8] core decoder error: failed to create video output
>>
>> If anyone has any ideas that, that would be helpful.
>>
> 
> Okay, I figured out these errors - it's because vlc needs the basic libxcb_*
> video_output plugins, but for some reason they are in the main vlc package
> rather than vlc-core.  This doesn't make since to me since vlc requires
> vlc-core.  I don't want the vlc binaries, but the libraries/plugins needed for
> playback.
> 
This very much looks like what is discussed here :
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4394
The vlc packaging could probably be arranged to better fit packages that
only depends on the vlc libs.

Regards,
Xavier
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


Re: phonon-backend-vlc for EL7

2017-02-23 Thread Orion Poplawski
On 02/23/2017 04:38 PM, Orion Poplawski wrote:
> I've taken a look at building phonon-backend-vlc for EL7.  The version that is
> in the el7 branch in git is 0.7.1-1, which does not build because it requires
> phonon 4.7.0, and EL7 has phonon 4.6.0.  I was able to build 0.6.2-1 fine.
> 
> Is there any objections to me reverting to 0.6.2-1 in git?  (I suppose someone
> with right access to the git repo could reset the el7 branch to that, but not
> sure it's worth it).
> 
> Also, while trying it out via dragon player in an X2Go session I'm getting:
> 
> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
> [7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
> [7f21a80a84b8] core video output error: video output creation failed
> [7f21c0019cc8] core decoder error: failed to create video output
> 
> If anyone has any ideas that, that would be helpful.
> 

Okay, I figured out these errors - it's because vlc needs the basic libxcb_*
video_output plugins, but for some reason they are in the main vlc package
rather than vlc-core.  This doesn't make since to me since vlc requires
vlc-core.  I don't want the vlc binaries, but the libraries/plugins needed for
playback.

-- 
Orion Poplawski
Technical Manager  720-772-5637
NWRA, Boulder/CoRA Office FAX: 303-415-9702
3380 Mitchell Lane   or...@nwra.com
Boulder, CO 80301   http://www.nwra.com
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org


phonon-backend-vlc for EL7

2017-02-23 Thread Orion Poplawski
I've taken a look at building phonon-backend-vlc for EL7.  The version that is
in the el7 branch in git is 0.7.1-1, which does not build because it requires
phonon 4.7.0, and EL7 has phonon 4.6.0.  I was able to build 0.6.2-1 fine.

Is there any objections to me reverting to 0.6.2-1 in git?  (I suppose someone
with right access to the git repo could reset the el7 branch to that, but not
sure it's worth it).

Also, while trying it out via dragon player in an X2Go session I'm getting:

[7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
[7f21a8054578] vdpau_avcodec generic error: Xlib is required for VDPAU
[7f21a80a84b8] core video output error: video output creation failed
[7f21c0019cc8] core decoder error: failed to create video output

If anyone has any ideas that, that would be helpful.

-- 
Orion Poplawski
Technical Manager  720-772-5637
NWRA, Boulder/CoRA Office FAX: 303-415-9702
3380 Mitchell Lane   or...@nwra.com
Boulder, CO 80301   http://www.nwra.com
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org