Bug 156765 – dev-java/jdbc-mysql-5.0.5 (version bump)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Stephane Bonnell 2006-12-01 00:50:28 UTC]

New version of JDBC connector for MySQL is available.

[Comment 1 Johan Bergström 2007-02-23 09:38:56 UTC]

As MySQL recommends this for use with 4.1 and forward, and the current stable version of MySQL in the tree is 5.0.26 - switching over to the jdbc 5-series maybe isn't a bad thing?

5.x series seems to have same build deps as 3.1x, so a bump should be rather straightforward.  

[Comment 2 Georgi Chulkov 2007-02-23 20:34:59 UTC]

I concur :). It would be great if we had a current jdbc-mysql connector.

[Comment 3 William L. Thomson Jr. (RETIRED) 2007-03-13 18:22:03 UTC]

Unfortunately not a straight forward bump, seems to have a dependency on some jboss stuff.

compile-testsuite:
    [javac] Compiling 38 source files to /usr/portage/tmp/portage/dev-java/jdbc-mysql-5.0.5/work/mysql-connector-java-5.0.5/build/mysql-connector-java-5.0.5
    [javac] /usr/portage/tmp/portage/dev-java/jdbc-mysql-5.0.5/work/mysql-connector-java-5.0.5/build/mysql-connector-java-5.0.5/com/mysql/jdbc/integration/jboss/MysqlValidConnectionChecker.java:31: package org.jboss.resource.adapter.jdbc does not exist
    [javac] import org.jboss.resource.adapter.jdbc.ValidConnectionChecker;

So looks like some jboss components will need to be packaged and available first.

[Comment 4 Josh Nichols (RETIRED) 2007-03-13 20:49:33 UTC]

Could also look into how JBoss is being used. If it is definately optional, we could remove it from our package. Having a modular JBoss is still aways a way, and depending on the entirety of JBoss is pretty unreasonable for a JDBC driver.

[Comment 5 Johan Bergström 2007-03-21 21:01:11 UTC]

JBoss is definitely optional. This test case just checks the mysql-jdbc integration bug found a while ago. 
Snip from test case:

         * Tests fix for BUG#20242 - MysqlValidConnectionChecker for JBoss doesn't
         * work with MySQLXADataSources.
         *
         * @throws Exception if the test fails.
         */
        public void testBug20242() throws Exception {
                if (versionMeetsMinimum(5, 0)) {
                        try {
                                Class.forName("org.jboss.resource.adapter.jdbc.ValidConnectionChecker");
                        } catch (Exception ex) {
                                return; // class not available for testing
                        }

                ...

                }
        }

[Comment 6 Johan Bergström 2007-04-11 16:39:43 UTC]

Is there anything i can do to help? How would we attack the jboss test dependency - is a fail ok, or should we patch away that test?

[Comment 7 Petteri Räty (RETIRED) 2007-04-11 16:42:03 UTC]

(In reply to comment #6)
> Is there anything i can do to help? How would we attack the jboss test
> dependency - is a fail ok, or should we patch away that test?
> 

Coming up with a patch to remove it would help, yes.

[Comment 8 Johan Bergström 2007-04-12 07:56:24 UTC]

Created attachment 116048 [details, diff]
removes jboss regression tests

OK, here's a stab at that patch. Didn't know what scope you want it in, let me know if i should create another one. The test suite now works without jboss (but failed for me in some other places - probably weird setup). If it should hassle for you - I'm sorry; kinda new to all of this.

[Comment 9 Johan Bergström 2007-04-12 07:57:08 UTC]

Ouch, probably should mention it's for 5.0.5 - not 5.0.4 (I thought the filename would show in bugzilla)

[Comment 10 Johan Bergström 2007-04-23 19:48:08 UTC]

Sorry for making this look like pointless bumping - but i'd really want to see this in the tree; is there anything else i can do? (last post, promise)

[Comment 11 Petteri Räty (RETIRED) 2007-05-06 23:03:28 UTC]

(In reply to comment #10)
> Sorry for making this look like pointless bumping - but i'd really want to see
> this in the tree; is there anything else i can do? (last post, promise)
> 

Don't worry about making noise when it's not pointless.

[Comment 12 Petteri Räty (RETIRED) 2007-05-07 21:10:56 UTC]

Committed. Thanks for reporting and waiting.