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)

main.log (9.7 KB) - added by zzanderr 9 years ago.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by zzanderr

Attachment: main.log added

comment:1 Changed 9 years ago by mf2k (Frank Schima)

Cc: devans@… removed
Owner: changed from macports-tickets@… to devans@…

comment:2 in reply to:  description Changed 9 years ago by dbevans (David B. Evans)

Status: newassigned

Replying to zanderpower@…:

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?

No it's specific to leopard and earlier. Will take a look.

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: assignedclosed

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.

Note: See TracTickets for help on using tickets.