Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 598236 - dev-libs/beignet-1.2.0 - add llvm 3.9 support
Summary: dev-libs/beignet-1.2.0 - add llvm 3.9 support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Marek Szuba
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2016-10-27 11:46 UTC by Andreas Steinmetz
Modified: 2016-11-10 14:30 UTC (History)
0 users

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


Attachments
llvm 3.9 support for beignet 1.2.0 (llvm39-support.patch,66.57 KB, patch)
2016-10-27 11:46 UTC, Andreas Steinmetz
Details | Diff
beignet-1.2.1.ebuild (beignet-1.2.1.ebuild,2.19 KB, text/plain)
2016-11-05 11:09 UTC, Russell Knighton
Details
beignet-1.2.1_no-hardcoded-cflags.patch (beignet-1.2.1_no-hardcoded-cflags.patch,721 bytes, patch)
2016-11-05 11:12 UTC, Russell Knighton
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Steinmetz 2016-10-27 11:46:57 UTC
Created attachment 451584 [details, diff]
llvm 3.9 support for beignet 1.2.0

Patch shamelessly ripped from:
http://ftp.debian.org/debian/pool/main/b/beignet/beignet_1.2.0-3.debian.tar.xz
Comment 1 Russell Knighton 2016-11-04 11:03:32 UTC
I think this support request can be safely superseded by a version bump request for 1.2.1 (which already includes this patch) and was just released: https://cgit.freedesktop.org/beignet/log/?h=Release_v1.2

Trying a simple ebuild bump (removing the <lvm-3.9 restriction) now.
Comment 2 Russell Knighton 2016-11-04 16:44:22 UTC
Official release announcement:
https://01.org/beignet/downloads/beignet-1.2.1-2016-11-04
Comment 3 Russell Knighton 2016-11-05 11:09:53 UTC
Created attachment 452412 [details]
beignet-1.2.1.ebuild

Working updated ebuild with the following changes applied:
--- beignet-1.2.0-r1.ebuild	2016-11-04 10:58:15.001788568 +0000
+++ beignet-1.2.1.ebuild	2016-11-05 10:08:44.799594091 +0000
@@ -29,7 +29,6 @@
 	media-libs/mesa
 	sys-devel/clang
 	>=sys-devel/llvm-3.5
-	<sys-devel/llvm-3.9
 	>=x11-libs/libdrm-2.4.70[video_cards_intel]
 	x11-libs/libXext
 	x11-libs/libXfixes"
@@ -43,7 +42,6 @@
 	"${FILESDIR}"/no-debian-multiarch.patch
 	"${FILESDIR}"/${P}_no-hardcoded-cflags.patch
 	"${FILESDIR}"/llvm-terminfo.patch
-	"${FILESDIR}"/${P}-drm_intel_get_pooled_eu.patch
 )
 
 DOCS=(
Comment 4 Russell Knighton 2016-11-05 11:12:13 UTC
Created attachment 452414 [details, diff]
beignet-1.2.1_no-hardcoded-cflags.patch

Updated patch, needed for the updated beignet-1.2.1 ebuild
Comment 5 Russell Knighton 2016-11-05 11:13:51 UTC
Haven't thoroughly tested the results, but the attached patch & ebuild for 1.2.1 cleanly compiled on my system.
Comment 6 Marek Szuba archtester gentoo-dev 2016-11-08 10:44:10 UTC
Thanks, guys. I'll have a look at this in the next couple of days.
Comment 7 Marek Szuba archtester gentoo-dev 2016-11-10 14:30:05 UTC
commit 56aa01b1e4c0f1ee4c5ec7376fd9f986e8eaf5f4
Author: Marek Szuba
Date:   Thu Nov 10 09:48:34 2016 +0000

    dev-libs/beignet: version bump to 1.2.1


Thank you for the heads-up and the testing, Russell!