Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.maint.java > #12714

Re: Bug#1054361: ITP: jruby-jzlib

From Jérôme Charaoui <jerome@riseup.net>
Newsgroups linux.debian.maint.java
Subject Re: Bug#1054361: ITP: jruby-jzlib
Date 2023-10-24 16:30 +0200
Message-ID <HsqzT-xeA-1@gated-at.bofh.it> (permalink)
References <HrHW9-4fN-1@gated-at.bofh.it> <Hspu9-wA9-1@gated-at.bofh.it> <Hsqgx-x7A-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Emmanuel,

Le 2023-10-24 à 10 h 02, Emmanuel Bourg a écrit :
> Hi Jérôme,
> 
> Le 24/10/2023 à 15:18, Jérôme Charaoui a écrit :
> 
>> I've prepared the jruby-jzlib package, but I'm uncertain about the 
>> relationship it should have with jzlib.
>>
>> Since the module itself is still using the same "com.jcraft" 
>> namespace, I'm thinking of using a Conflicts: and Provides: on 
>> libjzlib-java.
> 
> No that's not necessary, because the libjzlib-java and 
> libjruby-jzlib-java packages do not conflict at the filesystem level. 
> The Conflicts field doesn't cover classpath conflicts.

Right, my thinking was to use the same path usj/jzlib.jar to signal the 
classpath conflict. Otherwise, we can install it to usj/jruby-jzlib.jar 
and not make the packages conflict, but I'm not sure what would happen 
if both were installed at the same time, at the JVM-level.


>> You can check out the package here: 
>> https://salsa.debian.org/lavamind/jruby-jzlib
> 
> I got a quick look at the jzlib fork [1] and there are very few changes 
> compared to the original project, it just adds a JPMS auto module name. 
> We could simply patch our existing jzlib package instead of introducing 
> a new one. On the jruby side, that would mean patching the Maven 
> coordinates of jzlib (org.jruby:jzlib -> com.jcraft:jzlib).

There are some (small) code changes as well, here is a pkgdiff report: 
https://paste.lib3.net/lavamind/2023-10-24-gBV6KdXXUJ4R0DlxXjjnjz0RmA9OCJ6goNYKux5c03M/changes_report.html

In addition, I believe there may be more substantive changes in the 
future since there are zlib-related bugs reported against JRuby which 
may lead to further changes in jruby-jzlib, see 
https://github.com/jruby/jruby/issues/6613

Thanks,

-- Jérôme

Back to linux.debian.maint.java | Previous | NextPrevious in thread | Next in thread | Find similar


Thread

Fwd: Bug#1054361: ITP: jruby-jzlib Jérôme Charaoui <jerome@riseup.net> - 2023-10-24 15:20 +0200
  Re: Bug#1054361: ITP: jruby-jzlib Emmanuel Bourg <ebourg@apache.org> - 2023-10-24 16:10 +0200
    Re: Bug#1054361: ITP: jruby-jzlib Jérôme Charaoui <jerome@riseup.net> - 2023-10-24 16:30 +0200
      Re: Bug#1054361: ITP: jruby-jzlib Emmanuel Bourg <ebourg@apache.org> - 2023-10-24 17:10 +0200
        Re: Bug#1054361: ITP: jruby-jzlib Jérôme Charaoui <jerome@riseup.net> - 2023-10-24 17:30 +0200

csiph-web