Opened 12 years ago

Closed 9 years ago

#37543 closed defect (fixed)

syck: error: conflicting types for 'syckparse'

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: lharple@…
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: haspatch Cc: kurthindenburg (Kurt Hindenburg)
Port: syck

Description

gram.h:107:5: error: conflicting types for 'syckparse'
int syckparse (void);
    ^
./syck.h:471:5: note: previous declaration is here
int syckparse( void * );
    ^
1 error generated.
make[2]: *** [gram.o] Error 1

Attachments (5)

main.log (14.1 KB) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.
syck-bison.diff (402 bytes) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.
patch-configure.diff (373 bytes) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.
syck-bison.2.diff (379 bytes) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.
patch-bison-2.6.diff (2.1 KB) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: main.log added

comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Keywords: haspatch added

This happens if bison @2.7_0 is installed. Attached is a patch to fix this by forcing the use of the system's copy of bison, which on Mountain Lion is still 2.3. The configure script provides the variable "YACC" to override the automatic detection, unfortunately the configure script then checks to make sure it has not been overridden and bails out if it has, therefore the patch for the configure script is needed to remove that check. May I commit these patches? Unless you know a way to make syck work with newer bison, which would be preferable in case future OS X includes a newer bison.

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: syck-bison.diff added

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: patch-configure.diff added

comment:2 in reply to:  1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ryandesign@…:

Unless you know a way to make syck work with newer bison, which would be preferable in case future OS X includes a newer bison.

In fact here is the upstream bug report: https://github.com/indeyets/syck/issues/10

And the fix that was committed to make it build with bison 2.6 and newer: https://github.com/indeyets/syck/commit/4c0e9b5e2e9fa93f18c850f4a9c2cfe0e835b7d1

New patches attached.

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: syck-bison.2.diff added

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: patch-bison-2.6.diff added

comment:3 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:4 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

done r144941

Note: See TracTickets for help on using tickets.