Merge lp:~andrikos/ubuntu/quantal/fglrx-installer/fix-switch-to-igpu into lp:ubuntu/quantal/fglrx-installer

Proposed by Nick Andrik
Status: Merged
Merge reported by: Colin Watson
Merged at revision: not available
Proposed branch: lp:~andrikos/ubuntu/quantal/fglrx-installer/fix-switch-to-igpu
Merge into: lp:ubuntu/quantal/fglrx-installer
Diff against target: 27 lines (+8/-1)
2 files modified
debian/10fglrx.in (+1/-1)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~andrikos/ubuntu/quantal/fglrx-installer/fix-switch-to-igpu
Reviewer Review Type Date Requested Status
Alberto Milone Approve
penalvch (community) suitability Abstain
Ubuntu-X Pending
Ubuntu branches Pending
Review via email: mp+132962@code.launchpad.net

Description of the change

Fixes a problem when users would use amdcccle to switch to the integrated GPU.

For ATI/intel hybrid systems, this fix has no point unless this problem is addressed first:
https://code.launchpad.net/~andrikos/ubuntu/quantal/xserver-xorg-video-intel/fix-ati-hybrid

To post a comment you must log in.
Revision history for this message
penalvch (penalvch) wrote :

This patch, based on Nick's PPA, has not been shown to work for the original reporter as per http://ati.cchtml.com/show_bug.cgi?id=624 .

Review -> Disapprove

review: Disapprove (suitability)
Revision history for this message
penalvch (penalvch) wrote :

The previously mentioned Review: Disapprove was assessed for a different bug report https://bugs.launchpad.net/fglrx/+bug/1068661 . However, because the same branch was linked to two different bug reports, my intending Disapprove for only one report, flowed to both.

Review -> Abstain

review: Abstain (suitability)
Revision history for this message
Alberto Milone (albertomilone) wrote :
Revision history for this message
Nick Andrik (andrikos) wrote :

Fine then. Any possibility that it will be included in the precise
fglrx packages (at least the -updates ones)?
This is necessary for switching GPU in hybrid intel/fglrx systems

Thanks,
Nick

--
=Do-
N.AND

Revision history for this message
Alberto Milone (albertomilone) wrote :

Sure

Revision history for this message
Nick Andrik (andrikos) wrote :

Thanks!

Revision history for this message
Alberto Milone (albertomilone) wrote :

This is also merged in raring. I'll upload to precise-proposed soon. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/10fglrx.in'
2--- debian/10fglrx.in 2011-06-23 18:42:40 +0000
3+++ debian/10fglrx.in 2012-11-05 20:37:21 +0000
4@@ -1,7 +1,7 @@
5 LIBGL_DRIVERS_PATH=/#PKGLIBDIR#/dri
6 if [ `uname -m` = 'x86_64' ]; then
7 if [ -d /#PKGLIBDIR32#/dri ]; then
8- LIBGL_DRIVERS_PATH=${LIBGL_DRIVERS_PATH}:/#PKGLIBDIR32#/dri
9+ LIBGL_DRIVERS_PATH=${LIBGL_DRIVERS_PATH}:/#PKGLIBDIR32#/dri:/usr/lib/#DEB_HOST_MULTIARCH#/dri
10 if [ ! -z $LD_LIBRARY_PATH ]; then
11 LD_LIBRARY_PATH=$LD_LIBRARY_PATH:
12 fi
13
14=== modified file 'debian/changelog'
15--- debian/changelog 2012-09-29 20:47:10 +0000
16+++ debian/changelog 2012-11-05 20:37:21 +0000
17@@ -1,3 +1,10 @@
18+fglrx-installer (2:9.000-0ubuntu3+andrik1) quantal; urgency=low
19+
20+ * debian/10fglrx.in:
21+ - Added missing path which prevented hybrid operation
22+
23+ -- Nick Andrik <nick.andrik@gmail.com> Tue, 30 Oct 2012 03:36:41 +0100
24+
25 fglrx-installer (2:9.000-0ubuntu3) quantal; urgency=low
26
27 * debian/rules:

Subscribers

People subscribed via source and target branches

to all changes: