<html><head></head><body>Well, a few issues with this.<br>
<br>
Currently, I don't believe best practice is to install config.h because it defines so many general things like VERSION and PACKAGE_NAME. Image including two of these from different packages.<br>
<br>
I was going to add the new version define to swversion.h, but considered it yet another redundant place we need to change the package version when we release.<br>
<br>
I believe pkg-config is still the ubiquitously supported means for checking version, cflags, and libs (which are just as important as version for shared devel packages) on just about every package installed on your box.<br>
<br>
If we could agree to a way to generate this between your cmake build system and auto tools then I would consider swapping out config.h (already packaged for non-UNIX builds and included in swversion.cpp) for whatever mechanism we agree on. But this still doesn't replace the necessary work of pkg-config to query SWORD build flags. Remember, just because you compile and link doesn't mean you've successfully build your app against the exact state of the headers when the library was compiled and this can be dangerous.<br>
<br>
<br><br><div class="gmail_quote">Greg Hellings <greg.hellings@gmail.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div dir="ltr"><br /><div class="gmail_extra"><br /><br /><div class="gmail_quote">On Tue, Aug 6, 2013 at 6:29 AM, Troy A. Griffitts <span dir="ltr"><<a href="mailto:scribe@crosswire.org" target="_blank">scribe@crosswire.org</a>></span> wrote:<br />
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Windows frontends do not use a common packaging system to share a SWORD devel installation.<br /></div></blockquote>
<div><br /></div><div>Correct, but accessing this information at build time might still be useful. Could <a href="http://config.h.in" target="_blank">config.h.in</a> be updated to include this information in addition to sword.pc? I have seen multiple libraries that will give defines either the way you've specified it or by using SWORD_MAJOR_VERSION, SWORD_MINOR_VERSION and SWORD_PATCH_VERSION. Then the information would be available to anyone who included the header files.</div>
<div><br /></div><div>--Greg</div><div>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br /><div class="gmail_quote">Greg Hellings <<a href="mailto:greg.hellings@gmail.com" target="_blank">greg.hellings@gmail.com</a>> wrote:<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div><div>
<div dir="ltr">Relying on pkgconfig is not an entirely viable solution as it is not feasible to use on all target platforms - most notably Windows.<div><br /></div><div>--Greg</div></div><div class="gmail_extra"><br /><br /><div class="gmail_quote">
On Mon, Aug 5, 2013 at 9:07 PM, Troy A. Griffitts <span dir="ltr"><<a href="mailto:scribe@crosswire.org" target="_blank">scribe@crosswire.org</a>></span> wrote:<br /><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Jaak,<br />
<br />
I've include -DSWORD_NUMVERSION as a pkg-config cflag directive. I hope this is good enough for you. You can get this defined for you by including this in your .pro file.<br />
<br />
CONFIG += link_pkgconfig<br />
PKGCONFIG += sword<div><div><br />
<br />
<br />
<br />
<br />
On 08/05/2013 10:12 PM, Jaak Ristioja wrote:<br />
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-----BEGIN PGP SIGNED MESSAGE-----<br />
Hash: SHA1<br />
<br />
Its not simple to port to Sword 1.7.0_rc2, because the Sword test<br />
magic in BibleTime #includes <ftptrans.h>, which is not available in<br />
1.7. Hence compilation of that file fails. There's no way to do stuff<br />
like:<br />
<br />
#include <sword/version.h><br />
#if SWORD_VERSION >= 0x010690<br />
#include <sword/remotetrans.h><br />
typedef sword::FTPTransport SwordRemoteRransport;<br />
#else<br />
#include <sword/ftptrans.h><br />
typedef sword::RemoteTransport SwordRemoteRransport;<br />
#endif<br />
<br />
in the .cpp file to fix this easily. So I have to write A LOT of CMake<br />
scripts to -D a SWORD_VERSION macro to pass to the compiler, or worse.<br />
<br />
To sum up, it is currently not reasonably feasible to write C++ source<br />
code which compiles with both 1.6 and 1.7 versions of Sword. One needs<br />
a lot of support from the build system :(<br />
<br />
I again urge you to save us all some time and define a preprocessable<br />
SWORD_VERSION macro alike the *_VERSION macros boost or Qt define! If<br />
properly defined, this would at minimum halve the time needed to port<br />
existing versions of BibleTime to work with both 1.6 and 1.7.<br />
<br />
Blessings,<br />
Jaak<br />
<br />
PS: It would be wise to install a ftptrans.h file anyway, which<br />
includes a deprecation #warning, an #include "remotetrans.h" and a<br />
typedef aliasing sword::FTPTransport to sword::RemoteTransport.<br />
<br />
On 05.08.2013 22:01, Troy A. Griffitts wrote:<br />
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Well, It's been quiet since the release of RC2. Does this mean<br />
things are working as expected? Could I get a few positives from<br />
frontends if you've tried the bundle? Thank you.<br />
<br />
Troy<br />
<br />
<br />
<br />
On 08/03/2013 12:01 PM, Troy A. Griffitts wrote:<br />
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks for all the testing and reports against RC1. Things have<br />
quieted down, so hopefully we have all the remaining items<br />
polished off. RC2 is available for your weekend pleasure.<br />
<br />
<a href="http://crosswire.org/sword/alpha/alpha/sword-1.7.0RC2.tar.gz" target="_blank">http://crosswire.org/sword/<u></u>alpha/alpha/sword-1.7.0RC2.<u></u>tar.gz</a><br />
<br />
______________________________<u></u>_________________ sword-devel<br />
mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br />
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a><br />
Instructions to unsubscribe/change your settings at above page<br />
</blockquote>
<br />
______________________________<u></u>_________________ sword-devel mailing<br />
list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br />
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a> Instructions<br />
to unsubscribe/change your settings at above page<br />
</blockquote>
-----BEGIN PGP SIGNATURE-----<br />
Version: GnuPG v2.0.20 (GNU/Linux)<br />
<br />
iQgcBAEBAgAGBQJSAAcUAAoJEEqsYm<u></u>Et1rCO0Rs//<u></u>0uSHdvVI8C5VEWfgodW9uoW<br />
5XCRHUVR8IMMInNEv5rXXyNnZc+<u></u>UdViN4QJNYg5tjEOzxVtL6NTTR/<u></u>WWY0Uola2w<br />
Qj8NU3g6DgqVMXo7uFtEOuoQ6/<u></u>oyrKFLfiodMOcF3zMQZkBPWNc1KGPL<u></u>8OMUe7a9<br />
c5oiVqyg3LLrxGpgjTGKZWnEWawF9S<u></u>v9KgIwNyf4qILaStrDuF//<u></u>3AqYt4IHwdNL<br />
oHoZiZo7wtY4PCddJ0umEo0uF323Q0<u></u>xNX9iwM2bBNwFnGPSTpY7i3Oi5Tksb<u></u>7OJF<br />
HFnDMzdvo8ivVKEnwbnk3MUFCSBFL6<u></u>i3iyMeP6RCrPZw3wrFfMEbW6ufaghQ<u></u>5pqX<br />
MndwchusgJyN0KTTeEdMydd9xZ8i18<u></u>tUaiQZyYyPuiPwZ2uSf2vFIDrjt9TI<u></u>DuiU<br />
3uyk5ToWiWw9crc35PETh12iyaO7n9<u></u>ZVorYJwHNAYpQpCMkyJrdyt5Pvj18p<u></u>Eytt<br />
0Z8c9lxBn/+<u></u>m6yhe1ZnU6egZJ2cRyVAJon0LuJBdB<u></u>vJES0mWpG6Eaxc/uamTCB8E<br />
othxv6y4MH0CTefaF2Mrl1nOgJTaQ8<u></u>ZCuKk+<u></u>g7j5IdgOoALc9mGLpxmOqaF2YJQO<br />
NtgEVgAKMRqh/<u></u>gX3qTrFbovgOsuilXASAnDOo9KelmG<u></u>Nd553Iy4iMDz0RCE4Nb88<br />
njlbzD2q3LhCvtCKZnMH0mBiGuEpeY<u></u>gEDTVumrx9fHb9vTi66UF99YKcADe9<u></u>nphA<br />
T6OKZjK2EBJndTLDjSSqigYY49jXKr<u></u>KAGjMjSRnIfq41EFHYYezrZhjpFnq3<u></u>+Xnu<br />
pCucw0K18CPdRJZYPlkEd23928Yk/<u></u>9bZr2go/SKQVmIfvDfef+<u></u>qWfbolnQv0TU/K<br />
KC/<u></u>PNbawTgztBDvffbLpvaj51wO6Xqnwe<u></u>4FqmBn6xx7hytDQngYwsvntbO7qqg0<u></u>l<br />
QUoUol+6bqIlYtoUD3v+<u></u>XdiIjADQ7Bf23QzUyxLwsSqV4CWhO7<u></u>3gRq1+btOzoHnI<br />
+DZei5Ch7iV1zT1s/<u></u>UtS1Lau2yPUiY+<u></u>phAYysnobXzWneYfgATzebNIFa26te<u></u>Yhv<br />
c9iwBg7MKYgZJIsy5vHtqzZ8zkfk2b<u></u>v8mh1WEEtCaMidLYSQMBIt/<u></u>A6O7krL5JYp<br />
hCGMdWgM3QrxMwQ0YdjoMTy9mnzPM9<u></u>WGtucHaPUcqeniam6n3LRgBdlMoR8T<u></u>Rb+4<br />
ocywXsvEklhqdgYXoEKcNTSKzqGe1q<u></u>sBr+sPYp3x5+<u></u>JMuSVcYLGxlPZEgQVHKfkZ<br />
XyI+WYxd+<u></u>jcUti0dGTbuIroCKxZK2HdBAG9xALg<u></u>7F9RW8k4sOoCKBox2D13FTBEI<br />
Y5rPvxsQyN8N+<u></u>7r9NgujITnI3QYO73MsuRlzBp6g9mb<u></u>T0uVncUdjRYPCtGtXfO5J<br />
tfu3KOZuFgIQgDbtnbF+<u></u>CLG3jFs7orLYTM3J0qw6uZqJA0MGoa<u></u>da9/AtgEti1dEX<br />
3K1XktBKZzUZp3aL4nwP3NKOcA+<u></u>5gn8PO8Q6Vx8eHsfywVVJoy6tk5NkO<u></u>VW8GTO2<br />
TfM4PHNQFqTEJ2whTgi1SlHGUhykpG<u></u>7eF1arxGfsFdl8Z0jzqEhsPoptOuaT<u></u>I+Nn<br />
E88Bsd4d6Use4jQPtclvuXY6k6OVce<u></u>lrcCS4DEQjIkHIGIJfYug1uqlf577b<u></u>vJDj<br />
k9GnC20Dt4AzQiQv/<u></u>VEkj2xpbmjVLf9HTThHIKiIKDXpioG<u></u>v5lABu8sZO1VQgbLQ<br />
1ZGJgVtSkcxbo/nlHl+<u></u>Ksx7JdlS9F80yhl9k6eyUmnbvAlAN2<u></u>eKFiIRMhnT1oNyg<br />
Z3dAL759z0Z0M059lRT8pZjkS1UhHQ<u></u>Xvr2Qc8CjKGqzzY9wuWOW8yDKZERHJ<u></u>EjiU<br />
E5NVO7CtHX9FwS4f24Ehbemn5/<u></u>E4qWNQQfjAQVbvFiNulfR8w8ymhyiD<u></u>J0ES7at8<br />
LApzAaqPVnobkkiPihI70JcbhMaFwR<u></u>iScsWe54+<u></u>leGIkVDw3Zjh0YohhE23quuat<br />
Qd9A7WYxtYmv6ASKT7iCumFTmA2iie<u></u>2wxWCXU0M4CG7pGnB2Qn2Qnrl4VU1B<u></u>5KPz<br />
Eo3KlR0ugAFwhISBsPBNuZRm/<u></u>q5YS8plyejfq4W8+<u></u>zt3UcXYtQGFBfk8ziJGygvi<br />
7PL+EjwdeygTxyFutfx309cU+<u></u>HFKeHdNf0ElzTlKsNVZH71G0I5xCR4<u></u>Erqj4C0v1<br />
/<u></u>ow6N78XvOIYfAccsl7QXGtSeaKD4Zj<u></u>8mg2VV1bJg5h/P6uFZz3L+<u></u>plBuqZE5HEv<br />
Nne5LLQMoWO6EQ1cGW5LfMm6KGoaUP<u></u>AvNjKb1f3qUrENqUoZk+X+5SuO4Ek/<u></u>5Ivv<br />
ngu9Kgjx2QpeXAFTx7M5v4Sb/<u></u>ELulraVy+<u></u>CkQlHi4ocPQHlO1Duxi179Sn7jkR1w<br />
rvcGe7PDvI2FxY50c2QgPGOqPvAsLl<u></u>2ahwCS0OMsS8WnJpIqYCzlGh+hZ/<u></u>crAxL5<br />
DTvANBhuUwaAANV2m8eoYd36R1naJy<u></u>P8TjngwOLNL6eIbIXFXylmWyjYMKt8<u></u>281S<br />
Qrod5OJNzLEhm5AW27NAi+<u></u>au5XjO4wH8/JU7f+<u></u>iCP6AikmDWFxzn6l2ozvFeuP0C<br />
o76ghOmieLFHulMHi5MKpb57bC2vDQ<u></u>MMIpU7SvzwcLDiDenZbaP3dIu3sj25<u></u>TUI5<br />
Lf7pM6pYBqsO0P6BtStCU3REtkcvOm<u></u>pv1js/<u></u>jQ9YyZRFnNEHOWQAzA8eSxpNWW3b<br />
gbJ44+<u></u>iIQ7npfPlEhhbPP0Wcdaps0EHRuiwn<u></u>MitSPM/5mW2VmuY3GzA728n2DhSh<br />
k+umEB2KHXSboFDxZJPW<br />
=31JW<br />
-----END PGP SIGNATURE-----<br />
<br />
______________________________<u></u>_________________<br />
sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br />
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a><br />
Instructions to unsubscribe/change your settings at above page<br />
</blockquote>
<br />
<br />
______________________________<u></u>_________________<br />
sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br />
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a><br />
Instructions to unsubscribe/change your settings at above page<br />
</div></div></blockquote></div><br /></div>
<p style="margin-top:2.5em;margin-bottom:1em;border-bottom:1px solid #000"></p></div></div><pre><hr /><div><br />sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br />
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br />Instructions to unsubscribe/change your settings at above page</div></pre></blockquote>
</div><span><font color="#888888"><br />
-- <br />
Sent from my Android phone with K-9 Mail. Please excuse my brevity.</font></span></div></blockquote></div><br /></div></div>
<p style="margin-top: 2.5em; margin-bottom: 1em; border-bottom: 1px solid #000"></p><pre class="k9mail"><hr /><br />sword-devel mailing list: sword-devel@crosswire.org<br /><a href="http://www.crosswire.org/mailman/listinfo/sword-devel">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br />Instructions to unsubscribe/change your settings at above page</pre></blockquote></div><br>
-- <br>
Sent from my Android phone with K-9 Mail. Please excuse my brevity.</body></html>