Opened 11 years ago

Closed 11 years ago

#42848 closed defect (fixed)

textmate2: build fails, "Frameworks/file/src/reader.cc:71:4: error: constructor 'reader_t' mu st not return void expression"

Reported by: lpsinger (Leo Singer) Owned by: neverpanic (Clemens Lang)
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc: SickTeddyBear, ryandesign (Ryan Carsten Schmidt)
Port: textmate2

Description

textmate2 build fails on Mavericks. The immediate cause is:

st not return void expression
:info:build                         return io_error("open");
:info:build                         ^      ~~~~~~~~~~~~~~~~
:info:build Frameworks/file/src/reader.cc:76:4: error: constructor 'reader_t' mu
st not return void expression
:info:build                         return io_error("read");
:info:build                         ^      ~~~~~~~~~~~~~~~~
:info:build Frameworks/file/src/reader.cc:84:5: error: constructor 'reader_t' mu
st not return void expression
:info:build                                 return io_error("iconv_open");
:info:build                                 ^      ~~~~~~~~~~~~~~~~~~~~~~

Build log attached.

Attachments (1)

main.log (469.3 KB) - added by lpsinger (Leo Singer) 11 years ago.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by lpsinger (Leo Singer)

Attachment: main.log added

comment:1 Changed 11 years ago by neverpanic (Clemens Lang)

I'm aware of the issue, and it's already fixed upstream. The patch needs to be added to the Portfile, but I currently have no time to do so. If you have some time, please go right ahead and commit the change.

comment:2 Changed 11 years ago by SickTeddyBear

Cc: amcgee@… added

Cc Me!

comment:3 in reply to:  1 Changed 11 years ago by lpsinger (Leo Singer)

Replying to cal@…:

I'm aware of the issue, and it's already fixed upstream. The patch needs to be added to the Portfile, but I currently have no time to do so.

A new release appears to be available on GitHub:

https://github.com/textmate/textmate/releases/tag/v2.0-alpha.9525

Would updating the port resolve this issue?

comment:4 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added

The port was updated to a9529 in r118452. Did that help?

comment:5 Changed 11 years ago by lpsinger (Leo Singer)

Resolution: fixed
Status: newclosed

Yes, it builds now. Thanks!

Note: See TracTickets for help on using tickets.