[sword-devel] SWORD 1.7.0RC2

Troy A. Griffitts scribe at crosswire.org
Tue Aug 6 04:54:49 MST 2013 

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.hellings at gmail.com> wrote:
>On Tue, Aug 6, 2013 at 6:29 AM, Troy A. Griffitts
><scribe at 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.hellings at 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
><scribe at 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 at 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 at 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 at 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 at 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 at 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 at 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.crosswire.org/pipermail/sword-devel/attachments/20130806/5c8c3f8f/attachment.html>

More information about the sword-devel mailing list