Bug#821181: jruby: FTBFS due to PsychParser class error

[Miguel Landaeta]

On Wed, Aug 10, 2016 at 11:58:48PM +0200, Emmanuel Bourg wrote:
> 
> [...] 
> 
> I was a bit reluctant to upgrade again bnd and maven-bundle-plugin for
> stretch but we have a good reason to do it now.

Hi Emmanuel,

I have concluded the only feasible way to get out of this hole with
jruby is to revert yecht its 1.0 behavior. This means to remove
YechtService class from yecht.jar.

I can take care of that.

This way, I can get jruby 1.7.25 building in the archive with
available versions of bnd and maven-bundle-plugin. The remaining hurdle
is to get all its tests passing.

I apologize for basically dropping the ball with jruby maintenance on
this release cycle, but I had too many stuff to deal with in real
life.

My plan here is:

* Hack/patch yecht jar and upload yecht 1.1-2 to get jruby buildable
again.

* Upload jruby 1.7.25-1 to unstable once all tests are passing, to fix
a couple of FTBFS and remaining RC bugs.

* Later (before the freeze), check if it's possible to upload the
latest 1.7.x release (1.7.26).

Unfortunately, this means "stretch" will include a jruby release with
no support from upstream [1].

Cheers,


1. https://github.com/jruby/jruby/issues/4112

-- 
Miguel Landaeta, nomadium at debian.org
secure email with PGP 0x6E608B637D8967E9 available at http://miguel.cc/key.
"Faith means not wanting to know what is true." -- Nietzsche


signature.asc
Description: Digital signature

Bug#821181: jruby: FTBFS due to PsychParser class error

[Emmanuel Bourg]

Le 23/04/2016 à 15:17, Miguel Landaeta a écrit :

> It turns out I was responsible for the second bug. When I updated yecht
> 1.1-1, the updated .jar file is now dropping a class in the default package
> (YechtService.class).
> 
> That doesn't work with maven-bundle-plugin.

I got a look at the maven-bundle-plugin issue. I haven't been able to
find a plugin configuration that works, but I tested different versions
of the bundle plugin and the versions > 3 do not fail with the error
"The default package '.' is not permitted by the Import-Package syntax".

I was a bit reluctant to upgrade again bnd and maven-bundle-plugin for
stretch but we have a good reason to do it now.

Emmanuel Bourg

Bug#821181: jruby: FTBFS due to PsychParser class error

[Miguel Landaeta]

tags 821181 + pending
thanks

On Sat, Apr 16, 2016 at 03:08:19PM +0100, Miguel Landaeta wrote:
> 
> [...]
> 
> I already fixed the issue in the git repo.
> 
> I don't upload the fixed package yet because it keeps FTBFS due to
> another issue.

It turns out I was responsible for the second bug. When I updated yecht
1.1-1, the updated .jar file is now dropping a class in the default package
(YechtService.class).

That doesn't work with maven-bundle-plugin.

I got jruby 1.7.25 building in Debian although is not ready for upload
since some tests are failing and I need to workaround the yecht issue
in similar way to what is described here:

https://techotom.wordpress.com/2014/10/21/fixing-the-default-package-is-not-permitted-by-the-import-package-syntax-with-maven-bundle-plugin/

Help is welcome as usual.

-- 
Miguel Landaeta, nomadium at debian.org
secure email with PGP 0x6E608B637D8967E9 available at http://miguel.cc/key.
"Faith means not wanting to know what is true." -- Nietzsche


signature.asc
Description: Digital signature

Bug#821181: jruby: FTBFS due to PsychParser class error

[Miguel Landaeta]

On Sat, Apr 16, 2016 at 10:04 PM, Emmanuel Bourg  wrote:
> Le 16/04/2016 16:08, Miguel Landaeta a écrit :
>
>> The bug was caused by a breaking change introduced in snakeyaml 1.17.
>
> Hi Miguel,

Hello Emmanuel,

>
> Was this caused by the removal of getVersion() from
> DumperOptions.Version? I may try to patch snakeyaml and bring back the
> method to preserve the compatibility.
>

Yes, that was the cause of the bug.

> What is the other issue you encountered?
>

After patching the getVersion() change I found the following error
with maven-bundler-plugin:

[...]
[WARNING] Manifest org.jruby:jruby-complete:bundle:1.7.25 : Export
org.jruby.util.io,  has 4,  private references [jnr.posix,
jnr.unixsocket, org.jcodings, jnr.constants.platform],

[ERROR] Manifest org.jruby:jruby-complete:bundle:1.7.25 : The default
package '.' is not permitted by the Import-Package syntax.

 This can be caused by compile errors in Eclipse because Eclipse
creates

valid class files regardless of compile errors.

The following package(s) import from the default package null

[ERROR] Error(s) found in manifest configuration

[INFO] 

[INFO] Reactor Summary:

[INFO]

[INFO] JRuby .. SUCCESS [
0.119 s][INFO] JRuby Core .
SUCCESS [01:00 min]

   [INFO] JRuby Ext .. SUCCESS
[  0.004 s][INFO] JRuby Readline .
SUCCESS [  3.436 s]

[INFO] JRuby Ripper ... SUCCESS [  4.680 s]

[INFO] JRuby Lib Setup  SUCCESS [
0.003 s]

[INFO] JRuby Artifacts  SUCCESS [  0.003 s]

[INFO] JRuby Stdlib ... SUCCESS [
1.777 s]

[INFO] JRuby Complete . FAILURE [  5.236 s]

[INFO] 

[INFO] BUILD FAILURE

[INFO] 

[INFO] Total time: 01:17 min

[INFO] Finished at: 2016-04-16T13:12:11+00:00

[INFO] Final Memory: 110M/751M

[INFO] 

[ERROR] Failed to execute goal
org.apache.felix:maven-bundle-plugin:2.5.4:manifest (default) on
project jruby-complete: Error(s) found in manifest configuration ->
[Help 1]

[ERROR]

[ERROR] To see the full stack trace of the errors, re-run Maven with
the -e switch.

[...]


I have not found yet the root cause of that.

When I have some time available I will keep debugging if nobody beats me to it.

Cheers,

Bug#821181: jruby: FTBFS due to PsychParser class error

[Emmanuel Bourg]

Le 16/04/2016 16:08, Miguel Landaeta a écrit :

> The bug was caused by a breaking change introduced in snakeyaml 1.17.

Hi Miguel,

Was this caused by the removal of getVersion() from
DumperOptions.Version? I may try to patch snakeyaml and bring back the
method to preserve the compatibility.

What is the other issue you encountered?

Emmanuel Bourg

Bug#821181: jruby: FTBFS due to PsychParser class error

[Miguel Landaeta]

tags 821181 + pending
thanks

On Sat, Apr 16, 2016 at 12:19:01PM +0100, Miguel Landaeta wrote:
> Package: src:jruby
> Version: 1.7.22-2
> Severity: serious
> Justification: fails to build from source (but built successfully in the past)
> Owner: Miguel Landaeta 
> 

The bug was caused by a breaking change introduced in snakeyaml 1.17.

I already fixed the issue in the git repo.

I don't upload the fixed package yet because it keeps FTBFS due to
another issue.

Help is welcome.

-- 
Miguel Landaeta, nomadium at debian.org
secure email with PGP 0x6E608B637D8967E9 available at http://miguel.cc/key.
"Faith means not wanting to know what is true." -- Nietzsche


signature.asc
Description: Digital signature

Bug#821181: jruby: FTBFS due to PsychParser class error

[Miguel Landaeta]

Package: src:jruby
Version: 1.7.22-2
Severity: serious
Justification: fails to build from source (but built successfully in the past)
Owner: Miguel Landaeta 

Relevant error message:

[...]
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
/build/jruby-1.7.22/core/src/main/java/org/jruby/ext/psych/PsychParser.java:[232,66]
 error: cannot find symbol
[INFO] 1 error
[INFO] -
[...]

-- System Information:
Debian Release: 8.4
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.0.0-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=UTF-8 (charmap=UTF-8) (ignored: LC_ALL set 
to en_US.UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

-- 
Miguel Landaeta, nomadium at debian.org
secure email with PGP 0x6E608B637D8967E9 available at http://miguel.cc/key.
"Faith means not wanting to know what is true." -- Nietzsche


jruby_1.7.22-2_amd64.build.gz
Description: application/gzip


signature.asc
Description: Digital signature