Return-Path: X-Original-To: apmail-commons-dev-archive@www.apache.org Delivered-To: apmail-commons-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 81F8E1032C for ; Fri, 28 Feb 2014 17:48:53 +0000 (UTC) Received: (qmail 96480 invoked by uid 500); 28 Feb 2014 17:48:51 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 96291 invoked by uid 500); 28 Feb 2014 17:48:51 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 96283 invoked by uid 99); 28 Feb 2014 17:48:51 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 28 Feb 2014 17:48:51 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of garydgregory@gmail.com designates 74.125.83.54 as permitted sender) Received: from [74.125.83.54] (HELO mail-ee0-f54.google.com) (74.125.83.54) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 28 Feb 2014 17:48:47 +0000 Received: by mail-ee0-f54.google.com with SMTP id c41so2395533eek.41 for ; Fri, 28 Feb 2014 09:48:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=ZdARNxU/dORg03ztHba6v7SU+nIgbppePP2jpM63zAs=; b=dIcVVvWc1Zfiim+xlX6I2t28f5UYhbDiM9rC5TricLK3Fb/1ZqV9zogQhEGfB6kTXB gTsZ7GHeIHSDumNYuEDZdPz6mQOhOawGma1pc3NXbm6q8y8x7L+haSHZQzgaEMPFRBlE Eiu56wM6PFTnNKvyyg02HwLRKW9dgjvFsdEaJJ68+lxPwdagdu5e0E26HqzSubS18Kwc 1Szqs4i2A/8J9kcyN92cQdbmFMkJu7iuXgRjpdS6E4JkLCh2pXmBsoWHDvznCI5jaikd RiwEOL4wJ+WrKozZ3lY1JnmvbtgiAY5wqQPmRbnYOI0SPFI9jl0UAiKjWXzUTPEGDhV9 8H1w== MIME-Version: 1.0 X-Received: by 10.205.74.4 with SMTP id yu4mr215bkb.97.1393609706266; Fri, 28 Feb 2014 09:48:26 -0800 (PST) Received: by 10.204.102.198 with HTTP; Fri, 28 Feb 2014 09:48:26 -0800 (PST) In-Reply-To: References: <87ha7jcil2.fsf@v35516.1blu.de> Date: Fri, 28 Feb 2014 12:48:26 -0500 Message-ID: Subject: Re: [compress] problems with 7z BCJ methods From: Gary Gregory To: Commons Developers List Content-Type: multipart/alternative; boundary=f46d0415532c7f90b704f37b0b98 X-Virus-Checked: Checked by ClamAV on apache.org --f46d0415532c7f90b704f37b0b98 Content-Type: text/plain; charset=UTF-8 Do we have any idea WRT a new release of XZ? Gary On Fri, Feb 28, 2014 at 12:33 PM, sebb wrote: > On 28 February 2014 17:02, Gary Gregory wrote: > > On Fri, Feb 28, 2014 at 11:23 AM, Stefan Bodewig > wrote: > > > >> Hi, > >> > >> I've managed to implement the major BCJ filters for 7z (special filters > >> for native executables) by simply invoking the XZ for Java > >> implementations for it. This is needed for COMPRESS-257. > >> > >> Unfortunately there is a bug in XZ for Java that has already been fixed > >> in git. When using a released version of XZ together wirh Compress > >> 1.8-SNAPSHOT you'll be greeted with an AssertionError thrown from inside > >> XZ as soon as you try to read an archive that uses BCJ. > >> > >> I see two options and I'm not quite sure which is better from a user's > >> POV: > >> > >> * document the problem in XZ so people know they'll need to upgrade XZ > >> once it becomes available - and deal with recurring questions about > >> AssertionErrors > >> > > > > I like this one because we can document it and then the user can just > drop > > in a new XZ. Otherwise you'd need BOTH XZ and [compress] to have a new > > release. > > Can Compress catch the specific Assertion Error and convert it to a > message that explains the issue? > Perhaps even provide a link to a Wiki FAQ? > > Alternatively, can Compress detect the XZ version and refuse to run > BCJ, again with suitable message? > > These work-rounds can eventually be removed. > > > Gary > > > > > >> > >> * back out the BCJ code and re-introduce it once XZ has seen a new > >> release - this way users get a "method not supported" error rather > >> than some strange AssertionError > >> > >> Stefan > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > >> For additional commands, e-mail: dev-help@commons.apache.org > >> > >> > > > > > > -- > > E-Mail: garydgregory@gmail.com | ggregory@apache.org > > Java Persistence with Hibernate, Second Edition< > http://www.manning.com/bauer3/> > > JUnit in Action, Second Edition > > Spring Batch in Action > > Blog: http://garygregory.wordpress.com > > Home: http://garygregory.com/ > > Tweet! http://twitter.com/GaryGregory > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > -- E-Mail: garydgregory@gmail.com | ggregory@apache.org Java Persistence with Hibernate, Second Edition JUnit in Action, Second Edition Spring Batch in Action Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory --f46d0415532c7f90b704f37b0b98--