new non-free icons in handbrake

[Reinhard Tartler]

On Thu, Mar 14, 2013 at 9:56 PM, Paul Gevers <elbrus at debian.org> wrote:
> On 14-03-13 07:11, Reinhard Tartler wrote:
>> On Wed, Mar 13, 2013 at 10:17 PM, Paul Gevers <elbrus at debian.org> wrote:
>>> Yes, well, they replaced their (first) liba52 patch by a different one.
>>>
>>> So, how to handle this? (My C knowledge is not so good that I can see
>>> the solution).
>>> - I have no clue if their changes are reasonable and could be (asked
>>> for) inclusion in liba52
>>> - Reverse the functionality used by handbrake, but that would mean we
>>> have to carry (and keep updating) that patch basically forever.
>>
>>
>> Can you please publish your work-in-progress branch? I would like to
>> have a look at the patch.
>
> I uploaded my WIP on https://github.com/paulgevers/handbrake_debian (no
> previous experience with github though). However, that branch is the
> debian branch, so it lacks the patch by Handbrake, which is in the
> contrib directory that got purged.
>
> You can find the commit of the patch here:
> https://github.com/HandBrake/HandBrakeMirror/commit/f773c4d347ca46904de6545df127a68132e4765f

Thanks for your changes, they have been very helpful to me.

My analysis of the issue shows that the new upstream version requires
further adaptations to allow the compilation against the debian system
copies of the libraries liba52 and libmkv. I've therefore updated to
patches and pushed everything to:

http://anonscm.debian.org/gitweb/?p=pkg-multimedia/handbrake.git;a=summary

This whole beast did at least compile for me, but I have not tested it
yet at all, so testers would be more than welcome. If it helps, I
could publish packages for test, but compiling from source should be
really straight-forward.


-- 
regards,
    Reinhard