[Commits] Rev 3657: MDEV-4127 : Export additional symbols when building RPM
wlad at montyprogram.com
Sun Feb 3 19:38:24 EET 2013
> -----Original Message-----
> From: commits-bounces at mariadb.org [mailto:commits-
> bounces at mariadb.org] On Behalf Of Sergei Golubchik
> Sent: Sonntag, 3. Februar 2013 18:17
> To: maria-developers at lists.launchpad.net
> Cc: commits at mariadb.org
> Subject: Re: [Commits] Rev 3657: MDEV-4127 : Export additional symbols
> when building RPM
> Hi, Vladislav!
> May be you'd rather rename symbols with #define, just as redhat does?
It is possible.
However, unless it is done conditionally (e.g #ifdef
BUILD_MARIADB_AS_RPM), it is likely to break something (e.g odbc) on other
So I see 2 variants
1. Introduce BUILD_MARIADB_AS_RPM preprocessor constant, set to 1 if build
with RPM. If BUILD_MARIADB_AS_RPM is set, do rename functions via #define
, Redhat-style .
2. Export both original and renamed symbols, as in my patch.
Both are ok to me. Which one would be better, what do you think?
> On Feb 03, Vladislav Vaintroub wrote:
> > At file:///H:/bzr/5.5/
> > ------------------------------------------------------------
> > revno: 3657
> > revision-id: wlad at montyprogram.com-20130203164516-ex2ftmyca5b507g3
> > parent: elenst at ubuntu11.home-20130202225357-za9bglcf2ngv36xj
> > committer: Vladislav Vaintroub <wlad at montyprogram.com>
> > branch nick: 5.5
> > timestamp: Sun 2013-02-03 17:45:16 +0100
> > message:
> > MDEV-4127 : Export additional symbols when building RPM,
> > to enable both recompiling mysqli or odbc from sources in addition
> > to drop-in replacement functionality.
> > The case in question is compiling mysqli from sources, that needs
> > client_errors via ER() macro.
> > Previously, we exported it as mysql_client_errors (compatibly to
> > Fedora's style symbol renaming, see MDEV-3842).
> > However, if MariaDB header files are used when compiling mysqli,
> > client_errors needs to be exported with its original name.
> commits mailing list
> commits at mariadb.org
More information about the commits