No subject
Wed Jan 8 00:05:46 CET 2014
exception? . A complete diff between the two versions, if only just to see
how big it actually is, might be helpful here - the git history is not
that easy to read.
--alec
[1] https://github.com/mpv-player/mpv/
On Sat, Jan 25, 2014 at 11:33 AM, Ben Reedy <thebenj88 at gmail.com> wrote:
> On 16/01/14 21:14, Ben Reedy wrote:
>
>> Sorry Miro, I forgot to add that.
>>
>> >Has the library behaviour been modified?
>> The MPV back end to be bundled with CMPlayer has been patched for use
>> with CMPlayer.
>> For those interested, the changes made to the bundled MPV can be viewed
>> at Github [1]
>>
>> >Why haven't the changes been pushed to the upstream library?
>> The developer for CMPlayer has stated that these patches are specific to
>> CMPlayer, and are incompatible with upstream.
>>
>> >Could we make the forked version the canonical version within Fedora?
>> Same answer as above: the CMPlayer patches are incompatible with upstrea=
m
>> and are specific to CMPlayer.
>>
>> >Are the changes useful to consumers other than the bundling application=
?
>> No, these changes would only benefit CMPlayer users.
>>
>> >Is upstream keeping the base library updated or are they continuously
>> one or more versions behind the latest upstream release?
>> The CMPlayer developer is regularly merging upstream MPV commits into hi=
s
>> fork.
>>
>> >What is the attitude of upstream towards bundling?
>> The CMPlayer developer has stated that the bundling of MPV is necessary
>> for CMPlayer to successfully build and run.
>>
>> >Does the maintainer of the Fedora package of the library being bundled
>> have any comments about this?
>> The maintainer of the MPV package, Miro Hron=C4=8Dok, suggested this bun=
dling
>> exception.
>>
>> >Is there a plan for unbundling the library at a later time?
>> Not currently.
>>
>> Attached is an email reply from upstream concerning the bundling of MPV
>> with CMPlayer.
>>
>> [1] https://github.com/xylosper/mpv/compare/mpv-player:master...master
>>
>
> Hi all, would anyone like to comment on this bundling exception?
>
> Thanks,
>
> Ben.
>
--001a11331298d32d1204f0ca09b3
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
<div dir=3D"ltr"><div>Hm... Really, one of the elderly should step in here.=
In their absence: <br><br></div><div>Basically, the root problem seem to b=
e that=C2=A0 the cmplayer dev hasn't really tried to push his patches t=
o the mpv upstream. Looking at how vital the development at mpv [1] is, thi=
s looks like a mistake to me.=C2=A0 <br>
<br></div><div>However, this is the cmplayers's devs decision. Question=
is how rpmfusion should handle this. To me, merging the two versions (if a=
t all possible) is probably not a reasonable task for a packager - it's=
just looks too much (?) <br>
<br></div><div>From a more formal=C2=A0 POV: are these changes big enough t=
o warrant an exception?=C2=A0 . A complete diff between the two versions, =
if only just to see how big it actually is,=C2=A0 might be helpful here - t=
he git history is not that easy to read.<br>
</div><br><div>--alec<br></div><div><br></div><div><br>[1] <a href=3D"https=
://github.com/mpv-player/mpv/">https://github.com/mpv-player/mpv/</a><br></=
div></div><div class=3D"gmail_extra"><br><br><div class=3D"gmail_quote">On =
Sat, Jan 25, 2014 at 11:33 AM, Ben Reedy <span dir=3D"ltr"><<a href=3D"m=
ailto:thebenj88 at gmail.com" target=3D"_blank">thebenj88 at gmail.com</a>></s=
pan> wrote:<br>
<blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=
x #ccc solid;padding-left:1ex">On 16/01/14 21:14, Ben Reedy wrote:<br>
<blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=
x #ccc solid;padding-left:1ex">
Sorry Miro, I forgot to add that.<br>
<br>
>Has the library behaviour been modified?<br>
The MPV back end to be bundled with CMPlayer has been patched for use with =
CMPlayer.<br>
For those interested, the changes made to the bundled MPV can be viewed at =
Github [1]<br>
<br>
>Why haven't the changes been pushed to the upstream library?<br>
The developer for CMPlayer has stated that these patches are specific to CM=
Player, and are incompatible with upstream.<br>
<br>
>Could we make the forked version the canonical version within Fedora?<b=
r>
Same answer as above: the CMPlayer patches are incompatible with upstream a=
nd are specific to CMPlayer.<br>
<br>
>Are the changes useful to consumers other than the bundling application=
?<br>
No, these changes would only benefit CMPlayer users.<br>
<br>
>Is upstream keeping the base library updated or are they continuously o=
ne or more versions behind the latest upstream release?<br>
The CMPlayer developer is regularly merging upstream MPV commits into his f=
ork.<br>
<br>
>What is the attitude of upstream towards bundling?<br>
The CMPlayer developer has stated that the bundling of MPV is necessary for=
CMPlayer to successfully build and run.<br>
<br>
>Does the maintainer of the Fedora package of the library being bundled =
have any comments about this?<br>
The maintainer of the MPV package, Miro Hron=C4=8Dok, suggested this bundli=
ng exception.<br>
<br>
>Is there a plan for unbundling the library at a later time?<br>
Not currently.<br>
<br>
Attached is an email reply from upstream concerning the bundling of MPV wit=
h CMPlayer.<br>
<br>
[1] <a href=3D"https://github.com/xylosper/mpv/compare/mpv-player:master...=
master" target=3D"_blank">https://github.com/xylosper/<u></u>mpv/compare/mp=
v-player:master.<u></u>..master</a><br>
</blockquote>
<br>
Hi all, would anyone like to comment on this bundling exception?<br>
<br>
Thanks,<br>
<br>
Ben.<br>
</blockquote></div><br></div>
--001a11331298d32d1204f0ca09b3--
More information about the rpmfusion-developers
mailing list