[sword-devel] #include <lzma.h> in xzcomprs.cpp - is this a bug or ?

[Jaak Ristioja]

I just tried `./configure && make` Sword 1.7.4 and it did compile in
xzcomprs.cpp. So I guess the stable releases DO include the experimental
code.

Usually it helps when experimental features have their own feature
branches or similar. This not being a common practice in the Sword
project, is probably one cause for other experimental code reaching
trunk as well (see the "Infinite loop in SWModule multilemma window
search" thread on this mailing list).

Anyway, are you saying that removing/disabling this code will not break
anything for end-users? Because there are no (known?) modules which have
XZ compression? I think there's a risk it has already been used for
production. Can you please elaborate?

Best regards,
J

PS: Unrelated to the issue at hand, but this article might be of
interest to the Sword project: "Xz format inadequate for long-term
archiving" http://www.nongnu.org/lzip/xz_inadequate.html

On 03.11.2016 08:51, Peter von Kaehne wrote:
> Leaving aside the question of a bug, xzcompress.cpp is experimental code. It is not included in any releases so far AFAIK and should not be compiled into releases or used by frontends. 
> 
> At least under autotools you need to ask to include it. 
> 
> We have no modules created with it and would currently not allow any either. The only currently used and acceptable compression is zip.
> 
> HTH
> 
> Peter
> Sent from my phone. Apologies for brevity and typos.On 3 Nov 2016 02:50, TS <outofthecube at icloud.com> wrote:
>>
>> In xzcomprs.cpp, at line 30, there’s the code:
>>
>> #include <lzma.h>
>>
>> Xcode was not happy about this since there is no header named this. It seems that in the previous version of PocketSword, the fix was just to set Xcode as to not include this file when compiling so that’s what I’m doing too. Since the SWORD framework does not include lzma.h as a file, this seems like a bug to me, but perhaps there's a reason for it?
>>
>> I couldn’t find any reference regarding this matter in specific in the wiki or mailing list.
>>
>>
>> -TS
> _______________________________________________
> 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
>