1 |
Donnie Berkholz <dberkholz@g.o> said: |
2 |
> On 17:26 Sat 29 Mar , Mike Frysinger (vapier) wrote: |
3 |
> > 1.1 sys-apps/iproute2/iproute2-2.6.24.20080108.ebuild |
4 |
> > |
5 |
> > file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/sys-apps/iproute2/iproute2-2.6.24.20080108.ebuild?rev=1.1&view=markup |
6 |
> > plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/sys-apps/iproute2/iproute2-2.6.24.20080108.ebuild?rev=1.1&content-type=text/plain |
7 |
> |
8 |
> > local check base=${PORTAGE_CONFIGROOT}/etc/portage/patches |
9 |
> > for check in {${CATEGORY}/${PF},${CATEGORY}/${P},${CATEGORY}/${PN}}; do |
10 |
> > EPATCH_SOURCE=${base}/${CTARGET}/${check} |
11 |
> > [[ -r ${EPATCH_SOURCE} ]] || EPATCH_SOURCE=${base}/${CHOST}/${check} |
12 |
> > [[ -r ${EPATCH_SOURCE} ]] || EPATCH_SOURCE=${base}/${check} |
13 |
> > if [[ -d ${EPATCH_SOURCE} ]] ; then |
14 |
> > EPATCH_SUFFIX="patch" |
15 |
> > EPATCH_FORCE="yes" \ |
16 |
> > EPATCH_MULTI_MSG="Applying user patches from ${EPATCH_SOURCE} ..." \ |
17 |
> > epatch |
18 |
> > break |
19 |
> > fi |
20 |
> > done |
21 |
> |
22 |
> This looks like it should be generic code somewhere else. |
23 |
|
24 |
Actually, I'd say this should just be removed. If a user wants to apply |
25 |
a patch, they can put their own ebuild into an overlay and do it |
26 |
themselves (presumably if they want to patch something, they'll know how |
27 |
to make the simple modifications to an ebuild). By allowing the user to |
28 |
arbitrarily patch something means we have no idea what the user has |
29 |
built and is filing a bug about. If they installed an ebuild from an |
30 |
overlay it is a lot easier to identify what they built. Sure, they |
31 |
could patch the ebuild in their tree, but by supporting user supplied |
32 |
patches easily in this way, we are encouraging them to patch things |
33 |
without our knowledge. If we start supporting this across the board, I |
34 |
can see bugs being filed when their patches break and they don't |
35 |
understand what is happening. |
36 |
|
37 |
Thanks, |
38 |
|
39 |
-- |
40 |
Mark Loeser |
41 |
email - halcy0n AT gentoo DOT org |
42 |
email - mark AT halcy0n DOT com |
43 |
web - http://www.halcy0n.com |