[sword-devel] installmgr (and xiphos) crashes (svn 2831)
Troy A. Griffitts
scribe at crosswire.org
Fri Jul 5 18:51:01 MST 2013
Mark and Jaak,
Thanks for your work on this. I've applied a modified patch from the
resolved update from Mark. Please have a look and let me know if this
tests ok against your repository. The modification was to simplify the
logic using SWBuf instead of a char *.
Thanks again for the patch,
On 07/04/2013 09:41 AM, Mark Trompell wrote:
> Replying to myself again, maybe I should find a doctor :)
> one patch didn't apply right away, I ported it to current trunk and
> made one big patch file out of the 3.
> The attached one should work right away.
> Testet with my own repository and with crosswire.org http(s) repositories.
> On Thu, Jul 4, 2013 at 7:40 AM, Mark Trompell <mark at foresightlinux.org> wrote:
>> ok, just to summarize, what we need is:
>> Would be great, if someone could add them too trunk before 1.7 is out.
>> On Tue, Jul 2, 2013 at 11:25 AM, Mark Trompell <mark at foresightlinux.org> wrote:
>>> On Tue, Jul 2, 2013 at 9:50 AM, Jaak Ristioja <jaak at ristioja.ee> wrote:
>>>> Well, if pBufRes == NULL, then pBufRes < pBuf.
>>>> For reference, here is the latest version of this source file:
>>>> It currently only breaks the loop if !pBufRes.
>>>> Do you still think there's something wrong with my changes? I have
>>>> trouble trying to understand what you mean.
>>> I'm the last to say there is something wrong, I just want to
>>> understand the reasons.
>>> So if you break on !pBufRes then the assert() isn't needed anymore?
>>> Anyway, thanks for taking care, now I hope one of the sword devs
>>> applies the fix.
>>> Blessing Mark
>>> Mark Trompell
>>> Foresight Linux Xfce Edition
>>> Cause your desktop should be freaking cool
>>> (and Xfce)
>> Mark Trompell
>> Foresight Linux Xfce Edition
>> Cause your desktop should be freaking cool
>> (and Xfce)
> sword-devel mailing list: sword-devel at crosswire.org
> Instructions to unsubscribe/change your settings at above page
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the sword-devel