Opened 13 years ago

Closed 13 years ago

#29770 closed enhancement (fixed)

cmake: remove platform darwin block

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: haspatch Cc:
Port: cmake

Description

The entire platform darwin block in the cmake port appears to no longer be needed. See r70407 (in which the platform darwin block was added to do a post-configure reinplace), r70553 (in which the universal variant was fixed by adding code to make and use a list of directories the reinplace would have to be done in), and r73737 (in which the post-configure reinplace was removed but the list-of-directories code was not deleted).

Attachments (1)

cmake.diff (627 bytes) - added by ryandesign (Ryan Carsten Schmidt) 13 years ago.
proposed patch

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: cmake.diff added

proposed patch

comment:1 Changed 13 years ago by cssdev

Status: newassigned

I'll check the build results using otool. I need to make sure cmake doesn't pull in the wrong thing.

comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

How do you mean? The entirety of the platform darwin block is a post-configure block whose sole purpose is to set a variable called dirs, which is never used again in the portfile. I've shown above the reasons why this block came into existence, how that reason no longer exists, and how it was neglected to be removed in a previous revision. Surely it can simply be deleted now.

comment:3 Changed 13 years ago by cssdev

Resolution: fixed
Status: assignedclosed

fixed in r79569

Note: See TracTickets for help on using tickets.