Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 255647 - x11-base/x11-drm-99999999: 002_all_misc-makefile-fixes.patch fails to apply
Summary: x11-base/x11-drm-99999999: 002_all_misc-makefile-fixes.patch fails to apply
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Bryan Stine (RETIRED)
URL:
Whiteboard:
Keywords:
: 254649 (view as bug list)
Depends on:
Blocks: 259503
  Show dependency tree
 
Reported: 2009-01-20 15:05 UTC by Jure Repinc
Modified: 2009-03-20 19:24 UTC (History)
4 users (show)

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


Attachments
002_all_misc-makefile-fixes.patch-22936.out (002_all_misc-makefile-fixes.patch-22936.out,3.18 KB, text/plain)
2009-01-20 15:07 UTC, Jure Repinc
Details
Various fixes for x11-drm, addition of x11-apps/drm-progs (0001-Various-fixes-for-x11-drm-addition-of-x11-apps-drm.patch,7.84 KB, patch)
2009-01-23 00:36 UTC, Bryan Stine (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jure Repinc 2009-01-20 15:05:13 UTC
When I try to emerge x11-base/x11-drm from x11 overlay It fails with this message:

 * Applying various patches (bugfixes/updates) ...
 *   002_all_misc-makefile-fixes.patch ...

 * Failed Patch: 002_all_misc-makefile-fixes.patch !
 *  ( /var/tmp/portage/x11-base/x11-drm-99999999/work/patch/002_all_misc-makefile-fixes.patch )
 *
 * Include in your bugreport the contents of:
 *
 *   /var/tmp/portage/x11-base/x11-drm-99999999/temp/002_all_misc-makefile-fixes.patch-22936.out

 *
 * ERROR: x11-base/x11-drm-99999999 failed.
 * Call stack:
 *               ebuild.sh, line   49:  Called src_unpack
 *             environment, line 4048:  Called epatch '/var/tmp/portage/x11-base/x11-drm-99999999/work/patch'
 *             environment, line 2053:  Called die
 * The specific snippet of code:
 *                   die "Failed Patch: ${patchname}!";
 *  The die message:
 *   Failed Patch: 002_all_misc-makefile-fixes.patch!

Reproducible: Always

Steps to Reproduce:
Comment 1 Jure Repinc 2009-01-20 15:07:06 UTC
Created attachment 179078 [details]
002_all_misc-makefile-fixes.patch-22936.out
Comment 2 Donnie Berkholz (RETIRED) gentoo-dev 2009-01-21 19:21:07 UTC
I think they recently changed their makefile to not build dr[im]stat in the same cases. That probably broke our patch.
Comment 3 Hong Hao 2009-01-22 09:55:03 UTC
It seems to work if everything related to dr[im]stat and epatch was removed from ebuild.
Comment 4 Bryan Stine (RETIRED) gentoo-dev 2009-01-22 17:34:35 UTC
Reassigning to me for whenever I get overlay access...
Comment 5 Donnie Berkholz (RETIRED) gentoo-dev 2009-01-22 18:58:21 UTC
Feel free to generate a git-formatted patch (commit message etc included) with `git format-patch HEAD^..HEAD` or so and post it here for me to apply.
Comment 6 Bryan Stine (RETIRED) gentoo-dev 2009-01-23 00:36:39 UTC
Created attachment 179399 [details, diff]
Various fixes for x11-drm, addition of x11-apps/drm-progs

git formatted patch to x11 overlay.

This patch covers two issues. This bug, as Donnie said, is a result of upstream changes to the linux-core Makefile. The test programs have their own happy home now, so the Makefiles are completely independent. Because of this, it's now possible to build x11-drm without any patches.

Since we don't need patches, I've made the SRC_URI and patch processing conditional on the presence of PATCHVER in the ebuild. No point doing all that stuff (and distributing an empty patchball) if there's nothing there.

I've fixed a minor issue with the MODULESD_... variables, which needed the .${KV_OBJ} stripped off the module names when being eval'ed.

Finally for x11-drm, I've removed the dr[im]stat programs. 

Since those programs might be useful, they've been added as a new package, x11-apps/drm-progs (Donnie's awesome package name). This ebuild performs some minor source and autotools modifications to build the programs against the system-provided libdrm (for the most part... dristat is mildly problematic).

Included in this git patch is a live ebuild for drm-progs. It almost certainly makes most sense to package as snapshots, since they don't change much and there is some source modification involved. Distfiles for the package will be almost the same as x11-drm's source tarball, with the *-core directories ripped out, since they aren't needed.

Thanks.
Comment 7 Xake 2009-01-26 23:19:05 UTC
I think Bryan has a good point in his approach.
dristat/drmstat may be as intresting for people building modules from the kernel as it is for people building from x11-drm. And building libdrm again just to build dri/drmstat in x11-drm?

As I see it either follow Bryans proposal or let libdrm build and install dri/drmstat and let x11-drm only build and install the modules.
Oh, and drm-progs should maybe follow libdrm instead of git-snapshots? 
Comment 8 Andreas Sturmlechner gentoo-dev 2009-02-28 12:36:14 UTC
That patch doesn't work anymore after a recent sync:

error: patch failed: x11-base/x11-drm/Manifest:1
error: x11-base/x11-drm/Manifest: patch does not apply
error: patch failed: x11-base/x11-drm/x11-drm-99999999.ebuild:28
error: x11-base/x11-drm/x11-drm-99999999.ebuild: patch does not apply
Comment 9 Tomáš Chvátal (RETIRED) gentoo-dev 2009-03-11 13:25:45 UTC
*** Bug 254649 has been marked as a duplicate of this bug. ***
Comment 10 Bryan Stine (RETIRED) gentoo-dev 2009-03-20 19:24:31 UTC
I've been given access to the overlay now, so I pushed these changes in.

Marking as fixed. Thanks.