Re: [PATCH] get rid of DLR_MYSQL & co

Stipe Tolj <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization Wapme Systems AG
Message-ID <[email protected]>
Alexander Malysh wrote:
> 
> Hi list,
> 
> attached you can find patch that removes unneeded DLR_[MYSQL|SDB] defines and
> --with-dlr= configure option. These confuse all users and just make no sense
> because if we compile with --enable-mysql as example we anyhow have a mysql
> dependencies.
> 
> With this patch applied , user can simple compile with: --enable-mysql
> --enable-oracle --enable-sdb and has dlr for this storage build-in.
> 
> Comments or vetos please!

some notes that are not specific to the patch itself:

according to autobook-1.3 the configure switches should have the
following semantics:

  --with-foobar: include support of the *external* foobar
component/library
  
  --enable-foobar: include support of the *internal* foobar component

hence I'd like to switch this to --with-mysql. Which of course brings
us again to the problem that this switch is already used and defines
mysql instllation location. We should then use --with-mysql-dir to
allow users to locate the mysql installation if it is installed at a
foobar place.

Comments on the patch itself follow... ;)

Stipe

mailto:[email protected]
-------------------------------------------------------------------
Wapme Systems AG

Münsterstr. 248
40470 Düsseldorf, NRW, Germany

phone: +49.211.74845.0
fax: +49.211.74845.299

mailto:[email protected]
http://www.wapme-systems.de/
-------------------------------------------------------------------

-----BEGIN PGP PUBLIC KEY BLOCK-----
Version: GnuPG v1.2.2 (Cygwin)

mIsEP6mcYwEEAMDnUiUwrbb+xwTFWN6TxF2+XZu7/alwJMeCwMBRvXtPZqfjpPhS
OkBpU0F4TrVuugz1HINTSaJTYq10AzDQXp5NkyWgckqW79nPAWuOX0dicbJk+cN2
nM2TI4KaxUDe6u8hghNEnH/i2lXsUu9apnP/iixzV81VC2je3uc9hZpnAAYptEVT
dGlwZSBUb2xqIChUZWNobm9sb2d5IENlbnRlciAmIFJlc2VhcmNoIExhYikgPHRv
bGpAd2FwbWUtc3lzdGVtcy5kZT6ItAQTAQIAHgUCP6mcYwIbAwYLCQgHAwIDFQID
AxYCAQIeAQIXgAAKCRABV0w1BqPYRuSqA/wPzsQxao2YePENCtgRTrO86U6zg3sl
OcS6CJFI4FZP5h/xD3GRsNH1+MPSvZlomDdpFnr547DGz/Kq9MXuQwVvlVig5yWZ
K5dtKp1r5YLhxJQBhfirZbRFFnYmf19f18J8OoS28tuFVftDl1AIwJS3HLyBTv6H
g2HyLAEKQIp30Q==
=aYCI
-----END PGP PUBLIC KEY BLOCK-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.