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
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
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
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
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
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
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
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
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
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
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,
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
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
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
14 matches
Mail list logo