Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 409293 - eutils.eclass - epatch ignores files
Summary: eutils.eclass - epatch ignores files
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-22 09:36 UTC by Toralf Förster
Modified: 2012-03-23 02:33 UTC (History)
2 users (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 Toralf Förster gentoo-dev 2012-03-22 09:36:47 UTC
I tried to back port a KGpg krash and therefore I putted the KDEs diff file into the directory /etc/portage/patches/kde-base/kgpg-4.8.1/ :
changeset_r42f8b1b51b09cb4b64fadf7e6cdd5ce359b0fa74.diff
But it was ignored. From the log file of portage :

>>> Preparing source in /var/tmp/portage/kde-base/kgpg-4.8.1/work/kgpg-4.8.1 ...
 * Applying user patches from /etc/portage/patches//kde-base/kgpg-4.8.1 ...
 * Done with patching
>>> Source prepared.


After I renamed it to 42f8b1b5.patch epatch worked well :

>>> Preparing source in /var/tmp/portage/kde-base/kgpg-4.8.1/work/kgpg-4.8.1 ...
 * Applying user patches from /etc/portage/patches//kde-base/kgpg-4.8.1 ...
 *   42f8b1b5.patch ...
 [ ok ]
 * Done with patching
>>> Source prepared.



Reproducible: Always
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2012-03-22 13:06:18 UTC
I don't think the filename length is important here, but its format is, combined with some KDE related eclasses.
Comment 2 Richard Grenville 2012-03-22 13:21:24 UTC
epatch_user() in eutils.eclass only applies patches whose names end with ".patch" (EPATCH_SUFFIX="patch"), so *.diff don't work.
Comment 3 Toralf Förster gentoo-dev 2012-03-22 13:42:56 UTC
(In reply to comment #2)
> epatch_user() in eutils.eclass only applies patches whose names end with
> ".patch" (EPATCH_SUFFIX="patch"), so *.diff don't work.

Argh - is this documented (I didn't found anythign related to suffix restrictions here : http://www.gentoo.org/doc/en/handbook/handbook-amd64.xml?style=printable&part=3&chap=6#doc_chap6) ?
Nevertheless what's about adding ".diff" too to the suffix list - it is a common file suffix for such things, isn't it ?