Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 157916 - x11-apps/sessreg-1.0.2 fails to build on fbsd
Summary: x11-apps/sessreg-1.0.2 fails to build on fbsd
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All FreeBSD
: High critical (vote)
Assignee: Gentoo X packagers
URL: http://bugs.freedesktop.org/show_bug....
Whiteboard:
Keywords: Inclusion
: 173590 182264 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-12-12 06:45 UTC by Roy Marples (RETIRED)
Modified: 2007-06-18 01:07 UTC (History)
4 users (show)

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

Attachments
Include types.h (sessreg-fbsd.patch,282 bytes, patch)
2006-12-12 06:46 UTC, Roy Marples (RETIRED) 		Details | Diff
Include sys/types.h if we have it (sessreg-fbsd.patch,229 bytes, patch)
2007-02-06 12:36 UTC, Roy Marples (RETIRED) 		Details | Diff
Show Obsolete (1) 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 Roy Marples (RETIRED) gentoo-dev 2006-12-12 06:45:21 UTC 
/usr/include/utmp.h:63: error: expected specifier-qualifier-list before ‘int32_t’
sessreg.c: In function ‘set_utmp’:
sessreg.c:421: error: ‘struct utmp’ has no member named ‘ut_time’
gmake[1]: *** [sessreg-sessreg.o] Error 1
gmake[1]: Leaving directory `/var/tmp/portage/x11-apps/sessreg-1.0.2/work/sessreg-1.0.2'
gmake: *** [all] Error 2

Patch to follow
Comment 1 Roy Marples (RETIRED) gentoo-dev 2006-12-12 06:46:01 UTC 
Created attachment 103862 [details, diff]
Include types.h
Comment 2 Donnie Berkholz (RETIRED) gentoo-dev 2006-12-12 08:45:20 UTC 
Is this required for a vanilla fbsd system as well?
Comment 3 Roy Marples (RETIRED) gentoo-dev 2006-12-12 10:29:02 UTC 
No idea as I don't have one.

drizzt does, so CC'ing him
Comment 4 Timothy Redaelli (RETIRED) gentoo-dev 2006-12-12 12:52:51 UTC 
vanilla fbsd does not use (yet) modular xorg, but theorically it needs it.
btw I suggest the following implementation:
+#ifdef HAVE_SYS_TYPES_H
+# include <sys/types.h>
+#endif
Comment 5 Roy Marples (RETIRED) gentoo-dev 2007-02-06 12:36:14 UTC 
Created attachment 109323 [details, diff]
Include sys/types.h if we have it
Comment 6 Christoph Mende (RETIRED) gentoo-dev 2007-04-06 19:45:02 UTC 
*** Bug 173590 has been marked as a duplicate of this bug. ***
Comment 7 Donnie Berkholz (RETIRED) gentoo-dev 2007-05-02 07:40:28 UTC 
Please reopen when this is committed upstream.
Comment 8 Joe Peterson (RETIRED) gentoo-dev 2007-06-12 23:02:33 UTC 
I just rant into this one too - should we not unkeyword this in 1.0.2 until fixed?
Comment 9 Joshua Baergen (RETIRED) gentoo-dev 2007-06-18 00:37:51 UTC 
This has supposedly been fixed upstream.
Comment 10 Joshua Baergen (RETIRED) gentoo-dev 2007-06-18 00:38:25 UTC 
*** Bug 182264 has been marked as a duplicate of this bug. ***
Comment 11 Joshua Baergen (RETIRED) gentoo-dev 2007-06-18 01:07:14 UTC 
Upstream's patch has been applied to sessreg-1.0.2.  They don't have the conditional, and they explain why in the upstream bug.  If this is a problem, let me know.