MIR for libproxool-java (and avalon-framework)

[Thierry Carrez]

Binary package hint: libproxool-java

libproxool-java is a new dependency of eucalyptus 1.6.2, replacing libc3p0-java as the connection pooling layer.
See MIR report: https://wiki.ubuntu.com/ProxoolMIR

libavalon-framework-java is a build-dependency of libproxool-java.
See MIR report: https://wiki.ubuntu.com/AvalonFrameworkMIR

[Dustin Kirkland  (kirkland)]

Elevating the priority to High.

This is blocking our upload of Eucalyptus 1.6.2 to Lucid.

[Kees Cook (kees)]

+1 on libavalon-framework-java.

How much effort is needed to implement the testsuite in libproxool-java? Given what the software does, it does not sound like it would be trivial to test it during SRUs or security updates. I would be much more comfortable if the tests were working.

[Thierry Carrez (ttx)]

OK, once I got past the funny implementation (testsuite doesn't use the junit ant task), I realized that in a typical java fashion, the testsuite brings a lot of new dependencies that are either in universe or non-packaged yet, so I've disabled those tests (requiring missing tyrex naming (?) and excalibur component manager libraries, + hibernate/spring which are in universe).

Some other tests fail because the debian packaging does not repackage the cglib classes inside the proxool JAR. I also disabled them.

What's left is working and passing tests, so I just uploaded a new version that should satisfy your requirements.

 libproxool-java (0.9.1-2ubuntu4) lucid; urgency=low
 .
   * Enable test suite:
     * debian/ant.properties, debian/build.xml: Add ant rules to run tests
     * debian/control: Added missing build-depends necessary for tests
     * debian/rules: Enable tests, add missing JARs to classpath
     * debian/patches/cglib.diff: updated so that src/java-test is also patched
     * debian/patches/testsuite.diff: Disable some tests that require
       non-packaged or universe dependencies, or fail due to CGLIB not being
       repackaged in the proxool jar.
   * debian/build.xml: Properly remove doc build directory on clean target

[Kees Cook (kees)]

Thanks! +1 on this.

[Martin Pitt (pitti)]

in component-mismatches, promoted.