Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 334135 - dev-cpp/commoncpp2 fails to build with glibc 2.12
Summary: dev-cpp/commoncpp2 fails to build with glibc 2.12
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: C++ Team [disbanded]
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: glibc-2.12
  Show dependency tree
 
Reported: 2010-08-23 19:07 UTC by Stefan Böhmann
Modified: 2010-08-24 14:13 UTC (History)
1 user (show)

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

Attachments
build log (commoncpp2-1.8.0_build.log,61.12 KB, text/plain)
2010-08-23 19:07 UTC, Stefan Böhmann 		Details
emerge --info (commoncpp2-1.8.0_emerge-info.txt,4.26 KB, text/plain)
2010-08-23 19:08 UTC, Stefan Böhmann 		Details
adds the missing sys/stat.h (1.8.0-glibc-2.12.patch,1.04 KB, patch)
2010-08-24 00:11 UTC, Arttu Valo 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Böhmann 2010-08-23 19:07:21 UTC 
commoncpp2 doesn't compile with recent glibc (I'm using glibc-2.12.1-r1).


Reproducible: Always
Comment 1 Stefan Böhmann 2010-08-23 19:07:59 UTC 
Created attachment 244277 [details]
build log
Comment 2 Stefan Böhmann 2010-08-23 19:08:34 UTC 
Created attachment 244279 [details]
emerge --info
Comment 3 Arttu Valo 2010-08-24 00:11:45 UTC 
Created attachment 244313 [details, diff]
adds the missing sys/stat.h
Comment 4 Arttu Valo 2010-08-24 00:15:50 UTC 
On a second thought ... maybe I should've put that HAVE_SYS_STAT_H into the cpp file instead, as it's not header stuff but rather implementation details. (And forgot to CC myself.)

Anyway, it compiles with the patch, on an ~x86 with glibc 2.12.1.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2010-08-24 14:13:15 UTC 
The same package has sys/stat.h everywhere and doesn't check for it beforehand, so no need for this either

Fixed now