Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 549174 - net-wireless/wpa_supplicant: add epatch_user support
Summary: net-wireless/wpa_supplicant: add epatch_user support
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bjarke Istrup Pedersen (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-11 12:28 UTC by charles17
Modified: 2015-05-14 08:06 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description charles17 2015-05-11 12:28:03 UTC
Please add epatch_user support.
Comment 1 Adam Feldman gentoo-dev 2015-05-12 20:50:05 UTC
Just curious, what's the reasoning behind the request?
Comment 2 charles17 2015-05-13 06:25:55 UTC
(In reply to NP-Hardass from comment #1)
> Just curious, what's the reasoning behind the request?

epatch_user is very common to many packages. Having it in the ebuild would simply avoid the need for putting the /etc/portage/bashrc stuff as described on https://wiki.gentoo.org/wiki//etc/portage/patches#Enabling_.2Fetc.2Fportage.2Fpatches_for_all_ebuilds in case a user wants to use patches from http://w1.fi/security/ which are not yet in the tree or if a user wants to use his/her own patches.
Comment 3 Bjarke Istrup Pedersen (RETIRED) gentoo-dev 2015-05-13 07:43:47 UTC
I haven't looked at epatch_user yet, but it might be an idea, as long as it is possible to show a big, fat warning telling the user, that they should be able to reproduce the issue without user patches before reporting them.

It is going to be a nightmare to debug issues caused by a users own patches :-)
Comment 4 Adam Feldman gentoo-dev 2015-05-14 06:15:46 UTC
(In reply to charles17 from comment #2)
> epatch_user is very common to many packages. Having it in the ebuild would
> simply avoid the need for putting the /etc/portage/bashrc stuff as described
> on
> https://wiki.gentoo.org/wiki//etc/portage/patches#Enabling_.2Fetc.2Fportage.
> 2Fpatches_for_all_ebuilds in case a user wants to use patches from
> http://w1.fi/security/ which are not yet in the tree or if a user wants to
> use his/her own patches.

Yeah, I am aware of what it is.  I was just having trouble thinking of a use case of why this package would be likely to have users applying patches.  Regarding the link that you submitted of security patches, I think those would be better handled by Gentoo and the maintainers as security bugs, but that is just my opinion.

(In reply to Bjarke Istrup Pedersen from comment #3)
> I haven't looked at epatch_user yet, but it might be an idea, as long as it
> is possible to show a big, fat warning telling the user, that they should be
> able to reproduce the issue without user patches before reporting them.
> 
> It is going to be a nightmare to debug issues caused by a users own patches
> :-)

The build.log should note that user patches were applied.  Even has a nice internal function name for the inclusion of the warning "epatch_user_death_notice" which ewarns "!!! User patches were applied to this build!" so I think you'll be okay on that front if you decide to add the support.
Comment 5 Bjarke Istrup Pedersen (RETIRED) gentoo-dev 2015-05-14 08:06:31 UTC
Okay, so the use-case is adding security patches when they arrive, I'm not going to do this - sorry.

If any new security patches are released, please submit bug report about this instead, so we can get them implemented.

I much better like the idea of people submitting the patches they have, so we all can benefit from it.