| Summary: | Patch to add support for building on Haiku | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Portable OpenSSH | Reporter: | Scott McCreary <scottmc2> | ||||
| Component: | Build system | Assignee: | Assigned to nobody <unassigned-bugs> | ||||
| Status: | CLOSED FIXED | ||||||
| Severity: | normal | CC: | djm, dtucker | ||||
| Priority: | P2 | ||||||
| Version: | 5.4p1 | ||||||
| Hardware: | ix86 | ||||||
| OS: | Other | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 1743 | ||||||
| Attachments: |
|
||||||
|
Description
Scott McCreary
2010-03-25 06:07:18 AEDT
Comment on attachment 1819 [details]
configure.ac patch to allow for building on Haiku
looks pretty safe, include for 5.5p1?
Comment on attachment 1819 [details] configure.ac patch to allow for building on Haiku >+if test -z "$have_u_int64_t" ; then >+ AC_MSG_CHECKING([for u_int64_t type in posix/stdint.h]) >+ AC_TRY_COMPILE( >+ [ #include <posix/stdint.h> ], >+ [ u_int64_t a; a = 1], This looks OK, but we don't actually check for the posix/stdint.h header nor do we include it so we actually pick up the u_int64_t definition in a central place. So perhaps just add AC_DEFINE(HAVE_U_INT64_T) to that first section for the *-*-haiku*) case, and remove the second half of that patch? (In reply to comment #3) > So perhaps just add > AC_DEFINE(HAVE_U_INT64_T) > to that first section for the *-*-haiku*) case, and remove the second > half of that patch? Sounds good, committed as suggested, it will be in 5.5p1. Thanks. Mass move of bugs RESOLVED->CLOSED following the release of openssh-5.5p1 |