Well, a few issues with this.
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.
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.
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.
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.
Greg Hellings <greg.helli...@gmail.com> wrote:
>On Tue, Aug 6, 2013 at 6:29 AM, Troy A. Griffitts
><scr...@crosswire.org>wrote:
>
>> Windows frontends do not use a common packaging system to share a
>SWORD
>> devel installation.
>>
>
>Correct, but accessing this information at build time might still be
>useful. Could config.h.in 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.
>
>--Greg
>
>
>>
>> Greg Hellings <greg.helli...@gmail.com> wrote:
>>>
>>> Relying on pkgconfig is not an entirely viable solution as it is
>not
>>> feasible to use on all target platforms - most notably Windows.
>>>
>>> --Greg
>>>
>>>
>>> On Mon, Aug 5, 2013 at 9:07 PM, Troy A. Griffitts
><scr...@crosswire.org>wrote:
>>>
>>>> Jaak,
>>>>
>>>> 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.
>>>>
>>>> CONFIG += link_pkgconfig
>>>> PKGCONFIG += sword
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> On 08/05/2013 10:12 PM, Jaak Ristioja wrote:
>>>>
>>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>>> Hash: SHA1
>>>>>
>>>>> Its not simple to port to Sword 1.7.0_rc2, because the Sword test
>>>>> magic in BibleTime #includes <ftptrans.h>, which is not available
>in
>>>>> 1.7. Hence compilation of that file fails. There's no way to do
>stuff
>>>>> like:
>>>>>
>>>>> #include <sword/version.h>
>>>>> #if SWORD_VERSION >= 0x010690
>>>>> #include <sword/remotetrans.h>
>>>>> typedef sword::FTPTransport SwordRemoteRransport;
>>>>> #else
>>>>> #include <sword/ftptrans.h>
>>>>> typedef sword::RemoteTransport SwordRemoteRransport;
>>>>> #endif
>>>>>
>>>>> in the .cpp file to fix this easily. So I have to write A LOT of
>CMake
>>>>> scripts to -D a SWORD_VERSION macro to pass to the compiler, or
>worse.
>>>>>
>>>>> To sum up, it is currently not reasonably feasible to write C++
>source
>>>>> code which compiles with both 1.6 and 1.7 versions of Sword. One
>needs
>>>>> a lot of support from the build system :(
>>>>>
>>>>> I again urge you to save us all some time and define a
>preprocessable
>>>>> SWORD_VERSION macro alike the *_VERSION macros boost or Qt define!
>If
>>>>> properly defined, this would at minimum halve the time needed to
>port
>>>>> existing versions of BibleTime to work with both 1.6 and 1.7.
>>>>>
>>>>> Blessings,
>>>>> Jaak
>>>>>
>>>>> PS: It would be wise to install a ftptrans.h file anyway, which
>>>>> includes a deprecation #warning, an #include "remotetrans.h" and a
>>>>> typedef aliasing sword::FTPTransport to sword::RemoteTransport.
>>>>>
>>>>> On 05.08.2013 22:01, Troy A. Griffitts wrote:
>>>>>
>>>>>> Well, It's been quiet since the release of RC2. Does this mean
>>>>>> things are working as expected? Could I get a few positives from
>>>>>> frontends if you've tried the bundle? Thank you.
>>>>>>
>>>>>> Troy
>>>>>>
>>>>>>
>>>>>>
>>>>>> On 08/03/2013 12:01 PM, Troy A. Griffitts wrote:
>>>>>>
>>>>>>> Thanks for all the testing and reports against RC1. Things have
>>>>>>> quieted down, so hopefully we have all the remaining items
>>>>>>> polished off. RC2 is available for your weekend pleasure.
>>>>>>>
>>>>>>>
>http://crosswire.org/sword/**alpha/alpha/sword-1.7.0RC2.**tar.gz<http://crosswire.org/sword/alpha/alpha/sword-1.7.0RC2.tar.gz>
>>>>>>>
>>>>>>> ______________________________**_________________ sword-devel
>>>>>>> mailing list: sword-devel@crosswire.org
>>>>>>>
>http://www.crosswire.org/**mailman/listinfo/sword-devel<http://www.crosswire.org/mailman/listinfo/sword-devel>
>>>>>>> Instructions to unsubscribe/change your settings at above page
>>>>>>>
>>>>>>
>>>>>> ______________________________**_________________ sword-devel
>mailing
>>>>>> list: sword-devel@crosswire.org
>>>>>>
>http://www.crosswire.org/**mailman/listinfo/sword-devel<http://www.crosswire.org/mailman/listinfo/sword-devel>Instructions
>>>>>> to unsubscribe/change your settings at above page
>>>>>>
>>>>> -----BEGIN PGP SIGNATURE-----
>>>>> Version: GnuPG v2.0.20 (GNU/Linux)
>>>>>
>>>>>
>iQgcBAEBAgAGBQJSAAcUAAoJEEqsYm**Et1rCO0Rs//**0uSHdvVI8C5VEWfgodW9uoW
>>>>>
>5XCRHUVR8IMMInNEv5rXXyNnZc+**UdViN4QJNYg5tjEOzxVtL6NTTR/**WWY0Uola2w
>>>>>
>Qj8NU3g6DgqVMXo7uFtEOuoQ6/**oyrKFLfiodMOcF3zMQZkBPWNc1KGPL**8OMUe7a9
>>>>>
>c5oiVqyg3LLrxGpgjTGKZWnEWawF9S**v9KgIwNyf4qILaStrDuF//**3AqYt4IHwdNL
>>>>>
>oHoZiZo7wtY4PCddJ0umEo0uF323Q0**xNX9iwM2bBNwFnGPSTpY7i3Oi5Tksb**7OJF
>>>>>
>HFnDMzdvo8ivVKEnwbnk3MUFCSBFL6**i3iyMeP6RCrPZw3wrFfMEbW6ufaghQ**5pqX
>>>>>
>MndwchusgJyN0KTTeEdMydd9xZ8i18**tUaiQZyYyPuiPwZ2uSf2vFIDrjt9TI**DuiU
>>>>>
>3uyk5ToWiWw9crc35PETh12iyaO7n9**ZVorYJwHNAYpQpCMkyJrdyt5Pvj18p**Eytt
>>>>>
>0Z8c9lxBn/+**m6yhe1ZnU6egZJ2cRyVAJon0LuJBdB**vJES0mWpG6Eaxc/uamTCB8E
>>>>>
>othxv6y4MH0CTefaF2Mrl1nOgJTaQ8**ZCuKk+**g7j5IdgOoALc9mGLpxmOqaF2YJQO
>>>>>
>NtgEVgAKMRqh/**gX3qTrFbovgOsuilXASAnDOo9KelmG**Nd553Iy4iMDz0RCE4Nb88
>>>>>
>njlbzD2q3LhCvtCKZnMH0mBiGuEpeY**gEDTVumrx9fHb9vTi66UF99YKcADe9**nphA
>>>>>
>T6OKZjK2EBJndTLDjSSqigYY49jXKr**KAGjMjSRnIfq41EFHYYezrZhjpFnq3**+Xnu
>>>>>
>pCucw0K18CPdRJZYPlkEd23928Yk/**9bZr2go/SKQVmIfvDfef+**qWfbolnQv0TU/K
>>>>>
>KC/**PNbawTgztBDvffbLpvaj51wO6Xqnwe**4FqmBn6xx7hytDQngYwsvntbO7qqg0**l
>>>>>
>QUoUol+6bqIlYtoUD3v+**XdiIjADQ7Bf23QzUyxLwsSqV4CWhO7**3gRq1+btOzoHnI
>>>>>
>+DZei5Ch7iV1zT1s/**UtS1Lau2yPUiY+**phAYysnobXzWneYfgATzebNIFa26te**Yhv
>>>>>
>c9iwBg7MKYgZJIsy5vHtqzZ8zkfk2b**v8mh1WEEtCaMidLYSQMBIt/**A6O7krL5JYp
>>>>>
>hCGMdWgM3QrxMwQ0YdjoMTy9mnzPM9**WGtucHaPUcqeniam6n3LRgBdlMoR8T**Rb+4
>>>>>
>ocywXsvEklhqdgYXoEKcNTSKzqGe1q**sBr+sPYp3x5+**JMuSVcYLGxlPZEgQVHKfkZ
>>>>>
>XyI+WYxd+**jcUti0dGTbuIroCKxZK2HdBAG9xALg**7F9RW8k4sOoCKBox2D13FTBEI
>>>>>
>Y5rPvxsQyN8N+**7r9NgujITnI3QYO73MsuRlzBp6g9mb**T0uVncUdjRYPCtGtXfO5J
>>>>>
>tfu3KOZuFgIQgDbtnbF+**CLG3jFs7orLYTM3J0qw6uZqJA0MGoa**da9/AtgEti1dEX
>>>>>
>3K1XktBKZzUZp3aL4nwP3NKOcA+**5gn8PO8Q6Vx8eHsfywVVJoy6tk5NkO**VW8GTO2
>>>>>
>TfM4PHNQFqTEJ2whTgi1SlHGUhykpG**7eF1arxGfsFdl8Z0jzqEhsPoptOuaT**I+Nn
>>>>>
>E88Bsd4d6Use4jQPtclvuXY6k6OVce**lrcCS4DEQjIkHIGIJfYug1uqlf577b**vJDj
>>>>>
>k9GnC20Dt4AzQiQv/**VEkj2xpbmjVLf9HTThHIKiIKDXpioG**v5lABu8sZO1VQgbLQ
>>>>>
>1ZGJgVtSkcxbo/nlHl+**Ksx7JdlS9F80yhl9k6eyUmnbvAlAN2**eKFiIRMhnT1oNyg
>>>>>
>Z3dAL759z0Z0M059lRT8pZjkS1UhHQ**Xvr2Qc8CjKGqzzY9wuWOW8yDKZERHJ**EjiU
>>>>>
>E5NVO7CtHX9FwS4f24Ehbemn5/**E4qWNQQfjAQVbvFiNulfR8w8ymhyiD**J0ES7at8
>>>>>
>LApzAaqPVnobkkiPihI70JcbhMaFwR**iScsWe54+**leGIkVDw3Zjh0YohhE23quuat
>>>>>
>Qd9A7WYxtYmv6ASKT7iCumFTmA2iie**2wxWCXU0M4CG7pGnB2Qn2Qnrl4VU1B**5KPz
>>>>>
>Eo3KlR0ugAFwhISBsPBNuZRm/**q5YS8plyejfq4W8+**zt3UcXYtQGFBfk8ziJGygvi
>>>>>
>7PL+EjwdeygTxyFutfx309cU+**HFKeHdNf0ElzTlKsNVZH71G0I5xCR4**Erqj4C0v1
>>>>>
>/**ow6N78XvOIYfAccsl7QXGtSeaKD4Zj**8mg2VV1bJg5h/P6uFZz3L+**plBuqZE5HEv
>>>>>
>Nne5LLQMoWO6EQ1cGW5LfMm6KGoaUP**AvNjKb1f3qUrENqUoZk+X+5SuO4Ek/**5Ivv
>>>>>
>ngu9Kgjx2QpeXAFTx7M5v4Sb/**ELulraVy+**CkQlHi4ocPQHlO1Duxi179Sn7jkR1w
>>>>>
>rvcGe7PDvI2FxY50c2QgPGOqPvAsLl**2ahwCS0OMsS8WnJpIqYCzlGh+hZ/**crAxL5
>>>>>
>DTvANBhuUwaAANV2m8eoYd36R1naJy**P8TjngwOLNL6eIbIXFXylmWyjYMKt8**281S
>>>>>
>Qrod5OJNzLEhm5AW27NAi+**au5XjO4wH8/JU7f+**iCP6AikmDWFxzn6l2ozvFeuP0C
>>>>>
>o76ghOmieLFHulMHi5MKpb57bC2vDQ**MMIpU7SvzwcLDiDenZbaP3dIu3sj25**TUI5
>>>>>
>Lf7pM6pYBqsO0P6BtStCU3REtkcvOm**pv1js/**jQ9YyZRFnNEHOWQAzA8eSxpNWW3b
>>>>>
>gbJ44+**iIQ7npfPlEhhbPP0Wcdaps0EHRuiwn**MitSPM/5mW2VmuY3GzA728n2DhSh
>>>>> k+umEB2KHXSboFDxZJPW
>>>>> =31JW
>>>>> -----END PGP SIGNATURE-----
>>>>>
>>>>> ______________________________**_________________
>>>>> sword-devel mailing list: sword-devel@crosswire.org
>>>>>
>http://www.crosswire.org/**mailman/listinfo/sword-devel<http://www.crosswire.org/mailman/listinfo/sword-devel>
>>>>> Instructions to unsubscribe/change your settings at above page
>>>>>
>>>>
>>>>
>>>> ______________________________**_________________
>>>> sword-devel mailing list: sword-devel@crosswire.org
>>>>
>http://www.crosswire.org/**mailman/listinfo/sword-devel<http://www.crosswire.org/mailman/listinfo/sword-devel>
>>>> Instructions to unsubscribe/change your settings at above page
>>>>
>>>
>>> ------------------------------
>>>
>>> sword-devel mailing list: sword-devel@crosswire.org
>>> http://www.crosswire.org/mailman/listinfo/sword-devel
>>> Instructions to unsubscribe/change your settings at above page
>>>
>>>
>> --
>> Sent from my Android phone with K-9 Mail. Please excuse my brevity.
>>
>
>
>------------------------------------------------------------------------
>
>_______________________________________________
>sword-devel mailing list: sword-devel@crosswire.org
>http://www.crosswire.org/mailman/listinfo/sword-devel
>Instructions to unsubscribe/change your settings at above page
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
_______________________________________________
sword-devel mailing list: sword-devel@crosswire.org
http://www.crosswire.org/mailman/listinfo/sword-devel
Instructions to unsubscribe/change your settings at above page
