Bug 201 - Building openssh from CVS fails on AIX 4.2.1
Summary: Building openssh from CVS fails on AIX 4.2.1
Status: CLOSED FIXED
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: sshd (show other bugs)
Version: -current
Hardware: PPC AIX
: P2 normal
Assignee: OpenSSH Bugzilla mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-04-03 12:18 AEST by Darren Tucker
Modified: 2004-04-14 12:24 AEST (History)
0 users

See Also:


Attachments
Does this patch help? (515 bytes, patch)
2002-04-06 02:21 AEST, Ben Lindstrom
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darren Tucker 2002-04-03 12:18:13 AEST
While compiling from CVS, monitor_mm.c on AIX 4.2.1, gcc gives the following 
error:

monitor_mm.c: In function `mm_create':
monitor_mm.c:87: `MAP_FAILED' undeclared (first use in this function)
monitor_mm.c:87: (Each undeclared identifier is reported only once
monitor_mm.c:87: for each function it appears in.)
make: *** [monitor_mm.o] Error 1

This does not occur on AIX 4.3.3, where /usr/include/sys/mman.h contains 
"#define MAP_FAILED ((void *)-1)". Adding this #define to monitor_mm.c allows it 
to compile and run. I'm not sure what the proper fix is.

AIX 4.2.1 is no longer supported by IBM but some boxes are still running it.
Comment 1 Ben Lindstrom 2002-04-06 02:21:56 AEST
Created attachment 62 [details]
Does this patch help?
Comment 2 Ben Lindstrom 2002-04-06 02:23:42 AEST
FYI.. I'd like to avoid adding more to the current include.h, defines.h, etc.  
We were in the middle of a proposed rewrite to simplify the header hell that is 
plaguing us. 

If no other platforms require this, then I think we should fix it for AIX only 
and then come back to it after the header header rewrites.

- Ben
Comment 3 Darren Cole 2002-04-06 05:44:09 AEST
This breaks on HP-UX 10.26 also.  If I defined MAP_FAILED as in the attachment
for this bug I can complie HP-UX (took some info to get it to compile from bug
<http://bugzilla.mindrot.org/show_bug.cgi?id=184>)
Comment 4 Ben Lindstrom 2002-04-06 08:39:17 AEST
If we move it into (at the bottom) of includes.h will that solve it for both 
platforms?  If so I'll just handle it that way and we will have to ensure to 
remember it when we finish up the header cleanup.

- Ben
Comment 5 Darren Tucker 2002-04-06 11:05:27 AEST
Attached patch (#61) fixes the compile error on AIX 4.2.1. Putting the same 
#define at the bottom of includes.h instead also works.
Comment 6 Ben Lindstrom 2002-04-06 14:12:08 AEST
Commited to CVS tree.
Comment 7 Damien Miller 2004-04-14 12:24:18 AEST
Mass change of RESOLVED bugs to CLOSED