Opened 9 years ago
Closed 9 years ago
#48245 closed defect (fixed)
policykit @0.113+docs fails in patch phase on ppc leopard (10.5.8)
Reported by: | zzanderr | Owned by: | dbevans (David B. Evans) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.3 |
Keywords: | leopard | Cc: | |
Port: | policykit |
Description
The logfile is attached. It's a bit disconcerting that no one else has reported this problem yet. Is the patch failure related to the +docs variant?
Attachments (1)
Change History (8)
Changed 9 years ago by zzanderr
comment:1 Changed 9 years ago by mf2k (Frank Schima)
Cc: | devans@… removed |
---|---|
Owner: | changed from macports-tickets@… to devans@… |
comment:2 Changed 9 years ago by dbevans (David B. Evans)
Status: | new → assigned |
---|
comment:3 Changed 9 years ago by dbevans (David B. Evans)
Fix committed in r138298. This removes the offending patch which is no longer required and fixes a minor configuration issue that prevented the API docs from being installed when using the +docs variant.
I don't have a test machine for leopard so please confirm whether this fixes your issue so I can close the ticket. Thanks for the report.
comment:4 Changed 9 years ago by josephsacco
That fix does remove the offending patch. However, there is an additional problem:
Undefined symbols for architecture ppc:
"_getline", referenced from:
_read_cookie in polkit_agent_helper_1-polkitagenthelperprivate.o
ld: symbol(s) not found for architecture ppc
-Joseph
comment:5 Changed 9 years ago by dbevans (David B. Evans)
getline() was only introduced to OS X in Lion so this will break the build on Snow Leopard and earlier.
Patch committed in r138310 that provides a private copy of getline() for platforms that need it. Again, I'm not able to test this so please let me know if it works for you. Thanks.
comment:6 Changed 9 years ago by zzanderr
policykit built and installed without a hitch. As I only use policykit because emacs depends on it, I can only assure you that emacs appears to work. If there is any specific procedure(s) you want me to try with policykit, let me know.
Thanks for your work!
comment:7 Changed 9 years ago by dbevans (David B. Evans)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Thanks for your testing as well. I also got a private email from another user that it works on 10.6.8 so I think we are good for now.
Replying to zanderpower@…:
No it's specific to leopard and earlier. Will take a look.