Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 16794

Summary: Xfree86-4.2.1-r2 unpacks savage driver into incorect place
Product: Gentoo Linux Reporter: roma1390
Component: Current packagesAssignee: Martin Schlemmer (RETIRED) <azarah>
Status: VERIFIED FIXED    
Severity: trivial    
Priority: High    
Version: unspecified   
Hardware: x86   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: i hope this patch works, now testing

Description roma1390 2003-03-04 01:38:09 UTC
i emerging Xfree86 again and again ... but all time i have that i have 1.1.20
version of savage.
after small study of ebuild i noticed that in ebuild is version: 1.1.27t.

during emerge xfree86 i visit:
/var/tmp/portage/xfree-4.2.1-r2/work/xc/programs/Xserver/hw/xfree86/drivers/savage
and saw:
# ls
Imakefile     savage.man       savage_dga.c     savage_image.c   savage_vbe.h
Makefile      savage_accel.c   savage_driver.c  savage_regs.h    savage_video.c
Makefile.bak  savage_bci.h     savage_driver.h  savage_shadow.c
savage        savage_cursor.c  savage_i2c.c     savage_vbe.c
# ls -l | grep ^d
drwxr-xr-x    2 portage  portage       552 Jan 17 03:41 savage
# grep DRIVER_VERSION savage/savage_driver.c | grep define
#define DRIVER_VERSION  "1.1.27t"
# grep DRIVER_VERSION savage_driver.c | grep define
#define DRIVER_VERSION  "1.1.20"

I think that thease files must be overwriten but not inside savage/savage directory?


Reproducible: Always
Steps to Reproduce:
1.emerge xfree

Actual Results:  
we have savage 1.1.20
# strings /usr/X11R6/lib/modules/drivers/savage_drv.o | grep version
driver (version 1.1.20) for S3 Savage chipsets

Expected Results:  
i want 1.1.27t
Comment 1 roma1390 2003-03-04 01:50:37 UTC
Created attachment 8928 [details, diff]
i hope this patch works, now testing

I hopes that this patch works corect, may be savage driver maintainer changed
internal structure of source zip package.
Comment 2 Martin Schlemmer (RETIRED) gentoo-dev 2003-03-06 16:13:39 UTC
Yep, thanks, fixed in CVS.
Comment 3 roma1390 2003-12-14 13:02:43 UTC
thanks, works. Just marking as VERIFIED