Skip to content

media-libs/x264: LTO patch - #55

Merged
InBetweenNames merged 3 commits into
InBetweenNames:masterfrom
pchome:lto-patch-x264
Nov 7, 2017
Merged

media-libs/x264: LTO patch#55
InBetweenNames merged 3 commits into
InBetweenNames:masterfrom
pchome:lto-patch-x264

Conversation

@pchome

@pchome pchome commented Oct 24, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@pchome pchome changed the title media-libs/h264: LTO patch media-libs/x264: LTO patch Oct 24, 2017
@InBetweenNames

Copy link
Copy Markdown
Owner

Nice! Just curious, does the patch only work on the snapshot version or does it work on the latest one in the /usr/portage tree as well?

@pchome

pchome commented Oct 24, 2017

Copy link
Copy Markdown
Contributor Author

(~)0.0.20170701(0/152) is the latest unstable snapshot in tree.

And due to http://git.videolan.org/?p=x264.git;a=blob;f=configure;h=f7b14d9d9730d9830e132e5d8ab50b7a3c066400;hb=HEAD#l961 this patch should work even for 9999 and maybe for prev versions too.

@InBetweenNames

Copy link
Copy Markdown
Owner

I'll give it a try when I boot into Gentoo later today. If it works on the latest testing x264, we may as well just apply your patch to that instead and that way there's less to maintain. Nice work! With all the user patches we've been getting, I may look into a better mechanism to merge them into /etc/portage/patches.

@InBetweenNames

Copy link
Copy Markdown
Owner

Alrighty, finally got a chance to look at it. So I see now why you included the ebuild itself: you needed the epatch_user functionality. However, since we're supplying the ebuild ourselves, we could actually just add the patch to the PATCHES variable and forego epatch_user altogether. What do you think about that change? We could make it policy that if an ebuild is supplied, then the LTO patches must be applied in the ebuild itself. Otherwise, it may be applied with eapply_patch (for out of tree ebuilds).

@pchome

pchome commented Oct 25, 2017

Copy link
Copy Markdown
Contributor Author

Ok, you can change this PRs on your needs and I will follow policy in future ones.

@InBetweenNames

Copy link
Copy Markdown
Owner

Sounds good! Next time I'm doing my Gentoo stuff I will amend this as needed and merge it.

@InBetweenNames
InBetweenNames merged commit 00870d1 into InBetweenNames:master Nov 7, 2017
@pchome
pchome deleted the lto-patch-x264 branch July 1, 2018 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants