Opened 6 years ago

Closed 6 years ago

#38588 closed defect (fixed)

tmux 1.8 build failure (osdep-darwin.c problems)

Reported by: frozencemetery (Robbie Harwood) Owned by: yeled@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: haspatch Cc: nonstop.server@…, kenyon@…, cooljeanius (Eric Gallager)
Port: tmux

Description

I believe the relevant part of the log is:

:info:build osdep-darwin.c: In function ‘osdep_get_name’:
:info:build osdep-darwin.c:36: error: storage size of ‘bsdinfo’ isn’t known
:info:build osdep-darwin.c:43: error: ‘PROC_PIDT_SHORTBSDINFO’ undeclared (first use in this function)
:info:build osdep-darwin.c:43: error: (Each undeclared identifier is reported only once
:info:build osdep-darwin.c:43: error: for each function it appears in.)
:info:build make: *** [osdep-darwin.o] Error 1

System is x86-64 running 10.6. Build was attempted using gcc47 to upgrade from 1.7 to 1.8. Please let me know if there's any other information I can provide.

Attachments (1)

osdep-darwin.c.diff (736 bytes) - added by jmroot (Joshua Root) 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by larryv (Lawrence Velázquez)

Owner: changed from macports-tickets@… to yeled@…

Thanks for the ticket. In the future, please Cc relevant port maintainers.

comment:2 Changed 6 years ago by cooljeanius (Eric Gallager)

Hm, this error looks like #38506, although that was just a Tiger/PPC issue...

comment:4 Changed 6 years ago by jmroot (Joshua Root)

The info in struct proc_bsdshortinfo is a subset of that in struct proc_bsdinfo, so upstream should be able to just use that and PROC_PIDTBSDINFO instead of PROC_PIDT_SHORTBSDINFO and be backward compatible.

Changed 6 years ago by jmroot (Joshua Root)

Attachment: osdep-darwin.c.diff added

comment:5 Changed 6 years ago by jmroot (Joshua Root)

Keywords: haspatch added

Like so. Patch should be OK to apply unconditionally, and to send upstream.

comment:6 Changed 6 years ago by nonstop.server@…

Cc: nonstop.server@… added

Cc Me!

comment:7 Changed 6 years ago by cooljeanius (Eric Gallager)

Patch was applied in r104682 and then reverted in r104683...

comment:8 Changed 6 years ago by jmroot (Joshua Root)

No, it wasn't.

comment:9 Changed 6 years ago by kenyon@…

Cc: kenyon@… added

Cc Me!

comment:10 Changed 6 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:11 Changed 6 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

r105099 apparently

Note: See TracTickets for help on using tickets.