<div dir="ltr">If memory serves, XZ and BZip2 only made it into 1.7 because I goofed. Working with branches in SVN is such a travesty that it led to a few accidental commits moved from trunk to the release branch despite not being intended as such. This is why I've been reluctant to make further releases on 1.7, because there is some pollution from trunk. When the current effort towards 1.8 is finalized, I'm going to put in automation around copying commits, so that it is both easier to maintain the branch as well as more difficult to accidentally pull commits from master to the release branch that are breaking commits. That should streamline release branch management as much as I know how to streamline it with SVN.<div><br></div><div>(In git this wouldn't have been a problem as taking commits out of a branch is straightforward, as is copying a commit from one branch to the next. Likewise, if there were discrete development branches so that experimental code didn't end up landing in trunk without being intended for release.)<br><div><br></div><div>--Greg</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 3, 2016 at 10:14 AM, Peter von Kaehne <span dir="ltr"><<a href="mailto:refdoc@gmx.net" target="_blank">refdoc@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Experimental in the sense that there is no use found for them at this moment. We will not publish any modules until there is a) consensus and b) a valid preferential use case over the existing system.<br>
<div class="HOEnZb"><div class="h5"><br>
Sent from my phone. Apologies for brevity and typos.On 3 Nov 2016 1:52 pm, DM Smith <<a href="mailto:dmsmith@crosswire.org">dmsmith@crosswire.org</a>> wrote:<br>
><br>
> lzss is an early compression supported by SWORD and JSword. There is no need, other than policy, to refrain from using it.<br>
><br>
> xz and bzip2 are fully supported by JSword. But not all frontends in the wild have it.<br>
><br>
> I worked with Chris Little on both xz and bzip2 support. Him adding it to SWORD lib, me to JSword and osis2mod. I’m not sure in what way they are experimental. AFAICT, they are complete and work.<br>
><br>
> DM<br>
><br>
> > On Nov 3, 2016, at 3:57 AM, Jaak Ristioja <<a href="mailto:jaak@ristioja.ee">jaak@ristioja.ee</a>> wrote:<br>
> ><br>
> > Thanks you, Peter! Ok, so XzCompress is experimental and the most common<br>
> > ZipCompress is considered stable. But what about LZSSCompress and<br>
> > Bzip2Compress?<br>
> ><br>
> > Will the experimental compression methods be removed from stable Sword<br>
> > releases in the future?<br>
> ><br>
> > J<br>
> ><br>
> > On 03.11.2016 09:45, Peter von Kaehne wrote:<br>
> >> I can categorically say that there are no released CW modules using any compression other than zip. There might be some in our experimental repo. But they would not be released so.<br>
> >><br>
> >> I would think the same can be confirmed rapidly with all other official repos, Xiphos, IBT and eBible. I am convinced they will say the same.<br>
> >><br>
> >> It is unlikely that anyone else has done so, but who knows? I would not worry about it.<br>
> >><br>
> >> Peter<br>
> >><br>
> >> Sent from my phone. Apologies for brevity and typos.On 3 Nov 2016 07:31, Jaak Ristioja <<a href="mailto:jaak@ristioja.ee">jaak@ristioja.ee</a>> wrote:<br>
> >>><br>
> >>> I just tried `./configure && make` Sword 1.7.4 and it did compile in<br>
> >>> xzcomprs.cpp. So I guess the stable releases DO include the experimental<br>
> >>> code.<br>
> >>><br>
> >>> Usually it helps when experimental features have their own feature<br>
> >>> branches or similar. This not being a common practice in the Sword<br>
> >>> project, is probably one cause for other experimental code reaching<br>
> >>> trunk as well (see the "Infinite loop in SWModule multilemma window<br>
> >>> search" thread on this mailing list).<br>
> >>><br>
> >>> Anyway, are you saying that removing/disabling this code will not break<br>
> >>> anything for end-users? Because there are no (known?) modules which have<br>
> >>> XZ compression? I think there's a risk it has already been used for<br>
> >>> production. Can you please elaborate?<br>
> >>><br>
> >>> Best regards,<br>
> >>> J<br>
> >>><br>
> >>> PS: Unrelated to the issue at hand, but this article might be of<br>
> >>> interest to the Sword project: "Xz format inadequate for long-term<br>
> >>> archiving" <a href="http://www.nongnu.org/lzip/xz_inadequate.html" rel="noreferrer" target="_blank">http://www.nongnu.org/lzip/xz_<wbr>inadequate.html</a><br>
> >>><br>
> >>> On 03.11.2016 08:51, Peter von Kaehne wrote:<br>
> >>>> 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.<br>
> >>>><br>
> >>>> At least under autotools you need to ask to include it.<br>
> >>>><br>
> >>>> We have no modules created with it and would currently not allow any either. The only currently used and acceptable compression is zip.<br>
> >>>><br>
> >>>> HTH<br>
> >>>><br>
> >>>> Peter<br>
> >>>> Sent from my phone. Apologies for brevity and typos.On 3 Nov 2016 02:50, TS <<a href="mailto:outofthecube@icloud.com">outofthecube@icloud.com</a>> wrote:<br>
> >>>>><br>
> >>>>> In xzcomprs.cpp, at line 30, there’s the code:<br>
> >>>>><br>
> >>>>> #include <lzma.h><br>
> >>>>><br>
> >>>>> 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?<br>
> >>>>><br>
> >>>>> I couldn’t find any reference regarding this matter in specific in the wiki or mailing list.<br>
> >>>>><br>
> >>>>><br>
> >>>>> -TS<br>
> >>>> ______________________________<wbr>_________________<br>
> >>>> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> >>>> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
> >>>> Instructions to unsubscribe/change your settings at above page<br>
> >>>><br>
> >>><br>
> >>> ______________________________<wbr>_________________<br>
> >>> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> >>> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
> >>> Instructions to unsubscribe/change your settings at above page<br>
> >> ______________________________<wbr>_________________<br>
> >> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> >> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
> >> Instructions to unsubscribe/change your settings at above page<br>
> >><br>
> ><br>
> ><br>
> > ______________________________<wbr>_________________<br>
> > sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> > <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
> > Instructions to unsubscribe/change your settings at above page<br>
><br>
> ______________________________<wbr>_________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
______________________________<wbr>_________________<br>
sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/sword-devel</a><br>
Instructions to unsubscribe/change your settings at above page</div></div></blockquote></div><br></div>