Fix the unpin_decorators.patch file.

[alanhdu]

Closes #16

[conda-forge-linter]

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

[alanhdu]

cc @hmaarrfk, since I think you also seem to be a maintainer / have merge-access?

(apologies for the ping if you don't).

[alanhdu]

cc @jakirkham @hmaarrfk -- is there anything I can do here to help get this merged in? Without this, the current version of moviepy on conda is broken.

[alanhdu]

@conda-forge-admin please rerender

[jakirkham]

Thanks Alan! 🙏

It is worth noting that this is a Python 3 only package now, but AIUI this patch was for Python 2. Am wondering should we just drop the patch? Or is it still needed for some reason?

[alanhdu]

It is worth noting that this is a Python 3 only package now, but AIUI this patch was for Python 2. Am wondering should we just drop the patch? Or is it still needed for some reason?

I'm not an expert here, but IIUC ths patch correctly it's meant to allow moviepy to use decorator=5 (this seems to be a port of Zulko/moviepy#1565) -- I think that Python 2 builds are already skipped (see feffd3f where this was added), so I'm not sure we can drop the patch.

(I think the "root" problem is that moviepy hasn't cut a release in almost 2 years now, which is why some of the dependencies needed to get updated).

[jakirkham]

Gotcha. Thanks for the context! 🙏

Yeah it seems they are working on it ( Zulko/moviepy#1089 ), but it is a big release, which would take time to get out.