#8763 closed patch
BUILD: Plugin building cleanup
Reported by: | lordhoto | Owned by: | sev- |
---|---|---|---|
Priority: | normal | Component: | --Other-- |
Version: | 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)
Change History (8)
by , 17 years ago
Attachment: | plugin-v1.patch added |
---|
comment:1 by , 17 years ago
comment:2 by , 17 years ago
Owner: | changed from | to
---|
comment:3 by , 17 years ago
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 by , 17 years ago
Owner: | changed from | to
---|
comment:5 by , 17 years ago
Owner: | changed from | to
---|---|
Status: | new → closed |
comment:6 by , 17 years ago
Sigh, another patch which I noticed too late. Committed to both branch and trunk
comment:7 by , 6 years ago
Component: | → --Other-- |
---|
A little update: As Kirben told me now it works fine with MinGW.