commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Bodewig <>
Subject Re: commons-compress git commit: [COMPRESS-392] Add Brotli decoder based on the Google Brotli library.
Date Thu, 04 May 2017 06:08:36 GMT
On 2017-05-03, Gary Gregory wrote:

> On May 2, 2017 9:19 PM, "Stefan Bodewig" <> wrote:

>> On 2017-05-02, <> wrote:

>>> @@ -245,6 +251,7 @@ jar, tar, zip, dump, 7z, arj.
>>>          <configuration>
>>>            <instructions>
>>>              <Import-Package>org.tukaani.xz;resolution:=optional</Import-Package>
>>> +            <Import-Package>org.brotli.dec;resolution:=optional</Import-Package>
>>>            </instructions>
>>>          </configuration>
>>>        </plugin>

>> org.brotli:dec is not an OSGi bundle as far as I can tell. Is this going
>> to work or will it break Compress in an OSGi environment?

Looking at what we did back with (Jukka's proposed
patch) this may actually be enough.

> Good question... I do not see use of the bundle plugin in

> Assuming I am looking in right place...

> Maybe a PR would help ;-)

Yeah, I did that for XZ for Java at that time.

dec probably doesn't really use Maven to build even though there is a
POM. I suspect it is using Bazel (see the BUILD file inside the source
directory), I'd need to figure out how to add a manifest to a Bazel
built jar first. Might move "take a closer look at Bazel" further up on
my TODO list. :-)


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message