Bug 749195 - [PATCH] nfs-utils-1.2.5-2.fc16 fails to build, PAGE_SIZE undefined
Summary: [PATCH] nfs-utils-1.2.5-2.fc16 fails to build, PAGE_SIZE undefined
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: nfs-utils
Version: 16
Hardware: powerpc
OS: Linux
high
high
Target Milestone: ---
Assignee: Steve Dickson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F16Betappc
TreeView+ depends on / blocked
 
Reported: 2011-10-26 13:14 UTC by Karsten Hopp
Modified: 2012-03-14 13:10 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-11-28 14:18:59 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
fix for undefined PAGE_SIZE (2.22 KB, patch)
2011-10-26 13:14 UTC, Karsten Hopp
no flags Details | Diff

Description Karsten Hopp 2011-10-26 13:14:18 UTC
Created attachment 530281 [details]
fix for undefined PAGE_SIZE

Description of problem:
ppc and ppc64 don't have PAGE_SIZE defined in the glibc headers, so nfs-utils fails to build:
http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=306011

Version-Release number of selected component (if applicable):
nfs-utils-1.2.5-2.fc16 and nfs-utils-1.2.5-1.fc16

How reproducible:
always

Steps to Reproduce:
1. ppc-koji build --scratch dist-f16 nfs-utils-1.2.5-2.fc16.src.rpm
  
Actual results:
http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=306011

Comment 1 Steve Dickson 2011-11-14 12:35:37 UTC
It appears glibc-headers was removed from the default build root... 

Instead the patch that adds the define, wouldn't be simpler
and better to add glibc-headers to the BuildRequires list?

Comment 2 Karsten Hopp 2011-11-15 20:09:31 UTC
no, root.log of http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=306011 has this:

DEBUG util.py:247:   glibc                     ppc       2.14.90-13                  build    4.6 M
DEBUG util.py:247:   glibc-common              ppc       2.14.90-13                  build     11 M
DEBUG util.py:247:   glibc-devel               ppc       2.14.90-13                  build    989 k
DEBUG util.py:247:   glibc-headers             ppc       2.14.90-13                  build    602 k

(The root.log is from PPC, but I've reproduced this on PPC64)

Comment 3 Jeff Layton 2011-11-15 20:15:07 UTC
Why is PAGE_SIZE not defined in the glibc headers on those arches? I'd imagine that that would break a lot of code, not just nfs-utils...

Comment 4 Jiri Skala 2011-11-22 15:06:28 UTC
searching on ppc64:

/usr/include/common.h:#define PAGE_SIZE 4096

Comment 5 Karsten Hopp 2011-11-22 19:19:36 UTC
re: comment #4: neither nfs-utils nor any of its build requirements pull in librtas-devel (which provides /usr/include/common.h). In addition common.h doesn't get included and has a wrong value on ppc where we have a 64k pagesize.

re: comment #3: actually there only a few packages (<5) that use PAGE_SIZE instead of getpagesize(), at least that's my impression after rebuilding > 10000 packages on ppc.

Comment 6 Steve Dickson 2011-11-22 20:28:09 UTC
This appears to be fixed in nfs-utils-1.2.5-3.fc16 which has been pushed to stable.

Comment 7 Steve Dickson 2011-11-28 14:18:59 UTC
(In reply to comment #6)
> This appears to be fixed in nfs-utils-1.2.5-3.fc16 which has been pushed to
> stable.
The latest nfs-utils version does indeed build:
   http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=320376


Note You need to log in before you can comment on or make changes to this bug.