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

Bug 107430

Summary: [gcc4] pam and shadow not compiling due to pam headers problem
Product: Gentoo Linux Reporter: Markus Rothe (RETIRED) <corsair>
Component: [OLD] GCC PortingAssignee: PAM Gentoo Team (OBSOLETE) <pam-bugs+disabled>
Severity: enhancement CC: halcy0n, ppc64
Priority: High    
Version: unspecified   
Hardware: PPC64   
OS: Linux   
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 117482    
Attachments: pam-0.78-r2.txt
pam fix
shadow fix
new and improved pam fix :)

Description Markus Rothe (RETIRED) gentoo-dev 2005-09-27 11:57:44 UTC

pam-0.78-r2 and shadow-4.0.7-r3 do not compile using gcc-4.0.1. output follows.

Comment 1 Markus Rothe (RETIRED) gentoo-dev 2005-09-27 11:58:08 UTC
Created attachment 69360 [details]
Comment 2 Markus Rothe (RETIRED) gentoo-dev 2005-09-27 11:58:25 UTC
Created attachment 69361 [details]
Comment 3 Markus Rothe (RETIRED) gentoo-dev 2005-09-27 11:59:41 UTC
Created attachment 69362 [details]
Comment 4 Tim Yamin (RETIRED) gentoo-dev 2005-12-22 15:16:31 UTC
Want to try this again with a newer gcc-4 (4.1.0_beta, maybe)?
Comment 5 Markus Rothe (RETIRED) gentoo-dev 2005-12-23 02:09:29 UTC
same result using 4.1.0-beta20051216
Comment 6 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 17:30:16 UTC
Both of these occur for the same reason.  A user space header is defining __u8 before asm/types.h is pulled in.  Just need to reorder the includes so this happens.
Comment 7 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 17:31:30 UTC
Created attachment 76871 [details, diff]
pam fix
Comment 8 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 17:31:54 UTC
Created attachment 76872 [details, diff]
shadow fix
Comment 9 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 17:32:27 UTC
base-system and pam-bugs, please apply the patch for your app.  Thanks :)
Comment 10 SpanKY gentoo-dev 2006-01-11 17:56:31 UTC
i dont like it at all
Comment 11 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 19:36:50 UTC
Yea, this way does suck.  I'll just fix the pam header.  
Comment 12 Mark Loeser (RETIRED) gentoo-dev 2006-01-11 20:12:04 UTC
Created attachment 76877 [details, diff]
new and improved pam fix :)

Since our pam is linux only, I can go ahead and just include asm/types.h, as far as I know.
Comment 13 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-01-12 00:10:49 UTC
Actually, pam works fine on gcc4 on amd64.
Can someone point why ppc64 and amd64 differs?
Comment 14 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-01-12 01:10:57 UTC
Okay added a patch that should fix this.