Opened 7 years ago

Closed 5 years ago

#214 closed enhancement (wontfix)

Consider installing OMF type import libs along side aout style ones.

Reported by: Dave Yeo Owned by:
Priority: major Milestone:
Component: other Version:
Severity: low Keywords:
Cc: dave.r.yeo@…, lgrosenthal@…

Description

I've updated Doodles ScreenSaver? so the cairo based modules are linked to the RPM/YUM installed cairo2.dll.
Needed changes are having an OMF type export lib, cairo_dll.lib and a small patch to cairo/cairo.h to force cdecl calling convention, attached.

Attachments (1)

cairo_h.patch (418 bytes) - added by Dave Yeo 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by Dave Yeo

Attachment: cairo_h.patch added

comment:1 Changed 7 years ago by Dave Yeo

Summary: Doodles ScreenSaver can not use the installed Cairo as isDoodles ScreenSaver can not link to installed Cairo as is

comment:2 Changed 7 years ago by Dave Yeo

Summary: Doodles ScreenSaver can not link to installed Cairo as isConsider installing OMF type import libs along side aout style ones.

Simpler just to define before including cairo.h so patch dropped.
There is still the question of whether OMF type import libs should be installed along side the aout import libs.
Not only does it simplify linking for compilers besides GCC, but they have traditionally been included or been easily generated to speed up GCC linking when using -Zomf.
I'll change the title to reflect this

comment:3 Changed 5 years ago by Silvan Scherrer

Resolution: wontfix
Status: newclosed

it's true linking is a bit faster. but given the overhead of having .a and .lib doesn't qualify it. We would have to change to many tools to do it while building. And to be honest we don't have the resources to do so.

Note: See TracTickets for help on using tickets.