Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Eeli Kaikkonen
Quoting Matthew Talbert : We also use InstallMgr without subclassing. Frankly, I don't see how the virtual method will make it any more likely that frontends will prompt the user. We already warn the user, so having this method at all seems redundant. I think the idea is that the library can

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Eeli Kaikkonen
Quoting Manfred Bergmann : I didn't actually have seen a reason yet to subclass it. Why do you subclass InstallMgr? Because I can :) To be serious, it's nothing more than a design decision which could have been done otherwise. Maybe it will be changed one day. BibleTime doesn't have one g

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Manfred Bergmann
Am 23.04.2009 um 11:14 schrieb Eeli Kaikkonen: Quoting Manfred Bergmann : So I need to subclass InstallMgr now? Hmm, not sure I like it, means more code an an additional class just to return a true here. This is what I tried to explain in an earlier post. I didn't know that people use

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Manfred Bergmann
Am 23.04.2009 um 11:08 schrieb Troy A. Griffitts: Manfred Bergmann wrote: I noticed the setUserDisclaimerConfirmed(bool) has been removed. How can I set this now? So I need to subclass InstallMgr now? Hmm, not sure I like it, means more code an an additional class just to return a true here

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Eeli Kaikkonen
Quoting "Troy A. Griffitts" : I understand it is easier to just setUserDislaimerConfirmed(true), but this better assures that the frontend developers actually present something to the user-- or they are really going out of their way to NOT present something to the user if they overload this meth

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Matthew Talbert
On Thu, Apr 23, 2009 at 6:14 AM, Eeli Kaikkonen wrote: > Quoting Manfred Bergmann : > >> So I need to subclass InstallMgr now? >> Hmm, not sure I like it, means more code an an additional class just to >> return a true here. > > This is what I tried to explain in an earlier post. I didn't know tha

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Eeli Kaikkonen
Quoting Manfred Bergmann : So I need to subclass InstallMgr now? Hmm, not sure I like it, means more code an an additional class just to return a true here. This is what I tried to explain in an earlier post. I didn't know that people use instmgr without subclassing. Now when they do, it's b

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Troy A. Griffitts
Manfred Bergmann wrote: I noticed the setUserDisclaimerConfirmed(bool) has been removed. How can I set this now? So I need to subclass InstallMgr now? Hmm, not sure I like it, means more code an an additional class just to return a true here. Dear Manfred, My apologies for the inconvenience

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Manfred Bergmann
Am 23.04.2009 um 10:29 schrieb Manfred Bergmann: Am 21.04.2009 um 16:24 schrieb Troy A. Griffitts: Eeli Kaikkonen wrote: Unfortunately I have to destroy your hope here: we have only the static warning which doesn't have to be agreed. But I have already planned replacing it with a dynamic

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-23 Thread Manfred Bergmann
Am 21.04.2009 um 16:24 schrieb Troy A. Griffitts: Eeli Kaikkonen wrote: Unfortunately I have to destroy your hope here: we have only the static warning which doesn't have to be agreed. But I have already planned replacing it with a dynamic warning. There is virtual int refreshRemoteSource

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-21 Thread Troy A. Griffitts
Eeli Kaikkonen wrote: Unfortunately I have to destroy your hope here: we have only the static warning which doesn't have to be agreed. But I have already planned replacing it with a dynamic warning. There is virtual int refreshRemoteSourceConfiguration(). Would it be enough if I override it,

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-18 Thread Eeli Kaikkonen
Troy A. Griffitts wrote: Did you see the other #ifdefs we have for av11n? I believe we define 2 things in the sword cmake file to let you know if you are using the newer version of the lib. Maybe they should just be changed to something like SWORD_1_6 now that we've committed to a 1.6.x bran

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-17 Thread Troy A. Griffitts
Dear Eeli, Eeli Kaikkonen wrote: For several reason I just updated to, or actually installed from scratch, Kubuntu Jauntu (soon to be released 9.04). Therefore I could also install sword 1.6rc package. I have fixed the above mentioned problem, though not committed it yet. Did you see the oth

Re: [sword-devel] [bt-devel] Bibletime 2.0.alpha3 FTBFS with sword 1.6.0RC1 library

2009-04-17 Thread Eeli Kaikkonen
Eeli Kaikkonen wrote: Troy A. Griffitts wrote: Jonathan Marsden wrote: I'm seeing a problem building BT 2.0.alpha3 with the new 1.6.0RC1 SWORD /home/jonathan/packages/sword/bibletime/export-rc1/src/backend/managers/cswordbackend.cpp:498: error: ‘sysconfig’ was not declared in this scope My