Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 302387 - [PATCH] dev-util/perf doesn't patch rc? versions
Summary: [PATCH] dev-util/perf doesn't patch rc? versions
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Diego Elio Pettenò (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-27 02:12 UTC by Zeev Tarantov
Modified: 2010-01-31 13:24 UTC (History)
0 users

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


Attachments
fixes the ebuild (perf_ebuild.patch,1023 bytes, patch)
2010-01-27 02:14 UTC, Zeev Tarantov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zeev Tarantov 2010-01-27 02:12:35 UTC
The source is bundled with the linux kernel tarball. During unpack, the source has to be extracted from the right tarball and then patched to the latest rc or patch level. Only the needed parts of the tree are extracted, and only they need to be patched. Thus, the patch is filtered using filterdiff. Only, the patch is made using -p1 style and filterdiff produces an empty patch, which is not applied.

Because the ebuild was never tested with a non-empty patch, the epatch is invoked in the wrong directory and after sed changes the source in a way that prevents any patch from applying cleanly.

I attach a patch to the ebuild that fixes those issues.

Reproducible: Always
Comment 1 Zeev Tarantov 2010-01-27 02:14:18 UTC
Created attachment 217530 [details, diff]
fixes the ebuild
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2010-01-27 13:05:10 UTC
Uhm in which situation the patch could be empty at all? I've made it a rule to add RCs only for stuff that gets actually changes in perf (see [1]), so there should be no way for this to happen.

[1] http://blog.flameeyes.eu/2010/01/19/splitting-packages-when-to-bump
Comment 3 Zeev Tarantov 2010-01-27 15:16:00 UTC
# ebuild /usr/portage/dev-util/perf/perf-2.6.33_rc5.ebuild unpack

# ls -laF /var/tmp/portage/dev-util/perf-2.6.33_rc5/work/perf-2.6.33_rc5.patch
-rw-r--r-- 1 root root 0 Jan 27 17:11 /var/tmp/portage/dev-util/perf-2.6.33_rc5/work/perf-2.6.33_rc5.patch

because the ebuild runs:

$ filterdiff -i tools/perf/* -i include/* -i lib/* -i arch/*/include/* -z /usr/portage/distfiles/patch-2.6.33-rc5.bz2 | wc -l

0  <---  EMPTY PATCH !!!

instead of:

$ filterdiff -p1 -i tools/perf/* -i include/* -i lib/* -i arch/*/include/* -z /usr/portage/distfiles/patch-2.6.33-rc5.bz2 | wc -l

133716
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2010-01-31 13:24:02 UTC
Thanks Zeev, I fixed this now, sorry for the wait.