#13020 closed defect (fixed)
fortune doesn't build under Mac OS X 10.5 Leopard
Reported by: | afb@… | Owned by: | afb@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.5.0 |
Keywords: | Cc: | afb@…, mordeth@… | |
Port: |
Description
Attachments (1)
Change History (8)
comment:1 Changed 17 years ago by mordeth@…
comment:2 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | afb@… mordeth@… added |
---|---|
Owner: | changed from macports-dev@… to afb@… |
Summary: | fortune doesn't build under leopard → fortune doesn't build under Mac OS X 10.5 Leopard |
comment:3 Changed 17 years ago by rmcintosh@…
I concur fortune won't build with Leopard on my Intel MBP & a PPC PowerBook as well. Shouldn't the priority of this bug be raised above "normal"?
comment:4 Changed 17 years ago by afb@…
Depends on how vital "fortune" is to your every-day functionality :-)
It build and works OK on Tiger, so it's Yet Another Leopard Issue...
comment:5 Changed 17 years ago by seanrees@…
I consider 'fortune' to be pretty vital... :)
Made a few fixes; strfile/strfile.h --> #include <stdint.h> [strfile/strfile.c, unstr/unstr.c, fortune/fortune.c] --> OSSwapHostToBigInt32 -> htonl (from machine/endian.h)
I decided to make new patches for the OSSwapHostToBigInt32 -> htonl on for darwin 9 instead of patching the patch (though I just edited the existing patch files). Not sure of the standard way to approach this :)
Cheers.
comment:6 Changed 17 years ago by afb@…
Resolution: | → fixed |
---|---|
Status: | new → closed |
Thanks, <stdint.h> is needed on all platforms and patch should have used "htonl" instead of the Tiger-specific internal function (it was probably broken on Panther as well as on Leopard...)
Applied in r31206.
I can confirm this (Intel Core Duo MacBook Pro). Here is the error text: