Opened 12 years ago

Closed 12 years ago

Last modified 8 months ago

#8763 closed patch

BUILD: Plugin building cleanup

Reported by: lordhoto Owned by: sev-
Priority: normal Component: --Other--
Keywords: Cc:
Game:

Description

This patch aims at improving our plugin building.
It changes some things:

- Remove dependency on $(EXECUTABLE) while building plugins on Linux and FreeBSD, this prevents relinking of plugins when changing sources inside the main ScummVM codebase. For changes on the API (that means: in header files included in the engines code), the plugins are rebuild anyway, so it didn't look reasonable to relink the plugins when none of those had been touched but some other code inside the main ScummVM sources.

- Added 'wrapper' API for zlib (check common/zlib.[h/cpp]), this allows us to remove the need for plugins to link against libz, which was needed on some platforms (Win32/MinGW for example). Since I don't have access to a system with MinGW currently someone with MinGW should test it, but theoretically it should work.

I'm assigning this to Fingolfin since he's in charge of our build system. The part about MinGW should be more interesting for Kirben as builder for the Win32 binaries though. Still I would like to know what you think of adding such a 'wrapper' for zlib.

Ticket imported from: #1858158. Ticket imported from: patches/868.

Attachments (1)

plugin-v1.patch (6.0 KB) - added by lordhoto 12 years ago.

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by lordhoto

Attachment: plugin-v1.patch added

comment:1 Changed 12 years ago by lordhoto

A little update:
As Kirben told me now it works fine with MinGW.

comment:2 Changed 12 years ago by fingolfin

Owner: changed from fingolfin to sev-

comment:3 Changed 12 years ago by fingolfin

Addin a zlib wrapper API is the way to go, I am all for it. The one you added is minimal, which is good, too -- we should only add more stuff as (if) we need it.

So this patch looks good to me. But we are in a feature freeze. I'd say it should be OK to let this slip by, but our release manager Eugene should say yay or nay ;)

comment:4 Changed 12 years ago by fingolfin

Owner: changed from sev- to fingolfin

comment:5 Changed 12 years ago by sev-

Owner: changed from fingolfin to sev-
Status: newclosed

comment:6 Changed 12 years ago by sev-

Sigh, another patch which I noticed too late. Committed to both branch and trunk

comment:7 Changed 8 months ago by digitall

Component: --Other--
Note: See TracTickets for help on using tickets.