Opened 8 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 8 years ago.

Download all attachments as: .zip

Change History (4)

by Dave Yeo, 8 years ago

Attachment: cairo_h.patch added

comment:1 by Dave Yeo, 8 years ago

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

comment:2 by Dave Yeo, 8 years ago

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 by Silvan Scherrer, 5 years ago

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.