Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 546758 - net-misc/youtube-dl: add epatch_user
Summary: net-misc/youtube-dl: add epatch_user
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement
Assignee: Jeroen Roovers (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-16 10:05 UTC by Nikoli
Modified: 2015-05-02 10:51 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nikoli 2015-04-16 10:05:05 UTC 
Please add support for /etc/portage/patches/
Comment 1 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2015-05-01 15:43:08 UTC 
+  01 May 2015; Mikle Kolyada <zlogene@gentoo.org> youtube-dl-2015.04.03.ebuild,
+  youtube-dl-2015.04.26.ebuild, youtube-dl-2015.04.28.ebuild:
+  Add epatch_user suppot (bug #546758)
+
Comment 2 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2015-05-01 15:44:27 UTC 
A typo fixed
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2015-05-02 09:12:53 UTC 
???
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2015-05-02 09:14:33 UTC 
I guess I fixed the uninvited commits.
Comment 5 Jeroen Roovers (RETIRED) gentoo-dev 2015-05-02 09:18:59 UTC 
@zlogene: If you have been doing more random stuff like this, then please review those commits too. epatch_user is supposed to run /after/ the ebuild's patching, including through sed.
Comment 6 Nikoli 2015-05-02 10:51:09 UTC 
"epatch_user is supposed to run /after/ the ebuild's patching, including through sed."
I think in case of youtube-dl ebuild it is not safe to run epatch_user after all other commands in src_prepare, because they remove a lot files if offensive USE flag is disabled: if upstream patch modifies all or several extractors then it will fail. So it is more safe to run it first.