Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.x
    • Fix Version/s: None
    • Component/s: None
    • Security Level: public (Regular issues)
    • Labels:
      None
    • Environment:

      MS Windows XP SP2 (although Java based should work on any OS supporting Java)

      Description

      IBM has together with in a joint effort with Covalent developed a JBoss to Geronimo conversion tool. This tool is used when converting applications from JBoss to Geronimo, and automatically converts the configuration file from one app server to the other.

      We feel that this piece of software adds value to Geronimo and users adopting Geronimo and would like to see this effort continue as part of the Geronimo project, a plugin or a sub project of Geronimo.

      The initial donation is for version 1.0 of this tool, and while a 1.1 is in the making to improve 1.0, 1.1 is not yet complete but will be donated as soon as the community feels that this tool belongs at the ASF, more specifically within the Geronimo project.

      If you'd think this tool is valuable, but believe it should go through incubation, we would hope that a Geronimo committer would step up and champion this effort.

      The tool, including IBM's CCLA, can be found at http://people.apache.org/~fhanik/j2g/j2g.html (Covalent will file the CCLA upon request)

      1. CCLA.tif
        175 kB
        Filip Hanik
      2. Covalent-J2G-Tool.pdf
        1.18 MB
        Filip Hanik
      3. J2G-Migration_2.0.0_src_20070220-1501.zip
        2.98 MB
        Jeffrey Faelnar
      4. J2G-Migration_2.0.0_src_20070302-1501.zip
        3.03 MB
        Jeffrey Faelnar
      5. J2G-Migration_2.0.0_src_20070321-1501.zip
        3.03 MB
        Jeffrey Faelnar
      6. J2G-Migration-v2_src_1.0.0.zip
        5.52 MB
        Filip Hanik

        Activity

        Filip Hanik created issue -
        Filip Hanik made changes -
        Field Original Value New Value
        Attachment CCLA.tif [ 12349047 ]
        Attachment J2G-Migration-v2_src_1.0.0.zip [ 12349046 ]
        Filip Hanik made changes -
        Attachment Covalent-J2G-Tool.pdf [ 12349176 ]
        Davanum Srinivas made changes -
        Workflow RTC Workflow [ 12394416 ] jira [ 12397008 ]
        Hide
        Jeffrey Faelnar added a comment -

        The JIRA has been updated with the copyrights fixed and packages renamed.

        Show
        Jeffrey Faelnar added a comment - The JIRA has been updated with the copyrights fixed and packages renamed.
        Jeffrey Faelnar made changes -
        Hide
        Filip Hanik added a comment -

        Current codebase is packaged as: org.apache.apps.j2gmigrator...
        we suggest that during the import, the code is restructured as: org.apache.geronimo.j2gmigrator or org.apache.geronimo.devtools.j2gmigrator

        Show
        Filip Hanik added a comment - Current codebase is packaged as: org.apache.apps.j2gmigrator... we suggest that during the import, the code is restructured as: org.apache.geronimo.j2gmigrator or org.apache.geronimo.devtools.j2gmigrator
        Hide
        Paul McMahan added a comment -

        There was some discussion about the package name on dev:
        http://www.nabble.com/-VOTE--J2G-Conversion-tool-acceptance-tf3149274.html#a9120007

        Specifically, Sachin said:

        In order to follow proper convention, the package names need to start with the plugin id and the plugin id should be organization + product + plugin so in our case...

        plugin id: org.apache.geronimo.j2g.someIdForThisSpecificPlugin

        and all packages within that plugin should start with the pluginId

        Since this probably constitutes a significant amount of refactoring (and maybe even some more discussion) I think it might be best to make the packaging adjustments after the code is in sandbox.

        Show
        Paul McMahan added a comment - There was some discussion about the package name on dev: http://www.nabble.com/-VOTE--J2G-Conversion-tool-acceptance-tf3149274.html#a9120007 Specifically, Sachin said: In order to follow proper convention, the package names need to start with the plugin id and the plugin id should be organization + product + plugin so in our case... plugin id: org.apache.geronimo.j2g.someIdForThisSpecificPlugin and all packages within that plugin should start with the pluginId Since this probably constitutes a significant amount of refactoring (and maybe even some more discussion) I think it might be best to make the packaging adjustments after the code is in sandbox.
        Hide
        Paul McMahan added a comment -

        One of the questions on the IP clearance form is:
        "Check and make sure that the files that have been donated have been updated to reflect the new ASF copyright."

        These java source files don't contain an ASF copyright header:
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/Anybody.java
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/DigestCallback.java
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/GeronimoLoginModule.java
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/GeronimoSimpleGroup.java
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/Nobody.java
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/PasswordHasher.java
        src2/ejb-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/ejb/TestEJBDescriptorTool.java
        src2/enterprise-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/app/TestApplicationDescriptorTool.java
        src2/web-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/web/TestWebDescriptorTool.java
        src2/xml-conversion-helper/test/org/apache/apps/j2gmigrator/descriptors/xmlutil/TestXMLConversionHelper.java
        src2/xml-conversion-helper/test/org/apache/apps/j2gmigrator/resources/xmlutil/test/XMLConvertionHelperTestCase.java

        Also, I noticed these non-java files don't contain an ASF copyright header:
        src/common/build.properties
        src/common/plugin.xml
        src/configurator/bin/com/ibm/j2g/Configurator.class
        src/dependence-plugin/build.properties
        src/dependence-plugin/plugin.xml
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/class_analogies.properties
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/compatible_sources.properties
        src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/default_jars.properties
        src/descriptors-tool/build.properties
        src/descriptors-tool/plugin.xml
        src/descriptors-tool/schema/tool.migrations.exsd
        src/ejb-environment-plugin/build.properties
        src/ejb-environment-plugin/plugin.xml
        src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-interfaces.properties
        src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-references.properties
        src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-types.properties
        src/jasper-plugin/build.properties
        src/jasper-plugin/plugin.xml
        src/resources-tool/build.properties
        src/resources-tool/plugin.xml
        src/resources-tool/schema/tool.migrations.exsd
        src/scripts/j2g-configure.cmd
        src/scripts/jdesc2g.cmd
        src/scripts/jres2g.cmd
        src/scripts/jsrc2g.cmd
        src/sources-tool/build.properties
        src/sources-tool/plugin.xml
        src/sources-tool/schema/migrations.exsd
        src2/data-source-plugin/build.properties
        src2/data-source-plugin/plugin.xml
        src2/data-source-plugin/src/commons-logging.properties
        src2/data-source-plugin/test-apps/ds/hsqldb-ds.xml
        src2/data-source-plugin/test-apps/ds/hsqldb-geronimo-plan.xml
        src2/data-source-plugin/test-apps/ds/mysql-ds.xml
        src2/data-source-plugin/test-apps/ds/mysql-geronimo-plan.xml
        src2/data-source-plugin/test-apps/ds/oracle-ds.xml
        src2/data-source-plugin/test-apps/ds/oracle-geronimo-plan.xml
        src2/ejb-cmp-migration-plugin/build.properties
        src2/ejb-cmp-migration-plugin/plugin.xml
        src2/ejb-cmp-migration-plugin/src/commons-logging.properties
        src2/ejb-cmp-migration-plugin/test-resources/ejb-jar.xml
        src2/ejb-cmp-migration-plugin/test-resources/jbosscmp-jdbc.xml
        src2/ejb-migration-plugin/build.properties
        src2/ejb-migration-plugin/plugin.xml
        src2/ejb-migration-plugin/src/common-logging.properties
        src2/ejb-migration-plugin/test-resources/jboss.xml
        src2/enterprise-migration-plugin/.settings/org.eclipse.ltk.core.refactoring.prefs
        src2/enterprise-migration-plugin/build.properties
        src2/enterprise-migration-plugin/plugin.xml
        src2/enterprise-migration-plugin/src/commons-logging.properties
        src2/enterprise-migration-plugin/test-resources/geronimo-application.xml
        src2/enterprise-migration-plugin/test-resources/jboss-app.xml
        src2/jms-plugin/build.properties
        src2/jms-plugin/plugin.xml
        src2/jms-plugin/src/commons-logging.properties
        src2/jms-plugin/test-apps/jms/geronimo-jms-resource-plan.xml
        src2/jms-plugin/test-apps/jms/jbossmq-destinations-service.xml
        src2/jms-plugin/test-apps/jms/jms-ds.xml
        src2/jms-plugin/test-apps/jms/jms-geronimo-plan.xml
        src2/mail-plugin/build.properties
        src2/mail-plugin/plugin.xml
        src2/mail-plugin/src/commons-logging.properties
        src2/mail-plugin/test-apps/mail/err/mail-service.xml
        src2/mail-plugin/test-apps/mail/geronimo-mail-service.xml
        src2/mail-plugin/test-apps/mail/mail-geronimo-plan.xml
        src2/mail-plugin/test-apps/mail/mail-service.xml
        src2/security-plugin/build.properties
        src2/security-plugin/plugin.xml
        src2/security-plugin/src/commons-logging.properties
        src2/security-plugin/test-apps/security/geronimo-secutiry-plan.xml
        src2/security-plugin/test-apps/security/login-config.xml
        src2/security-plugin/test-apps/security/security-geronimo-plan.xml
        src2/web-migration-plugin/build.properties
        src2/web-migration-plugin/plugin.xml
        src2/web-migration-plugin/src/commons-logging.properties
        src2/web-migration-plugin/test-resources/jboss-web.xml
        src2/xml-conversion-helper/.settings/org.eclipse.ltk.core.refactoring.prefs
        src2/xml-conversion-helper/build.properties
        src2/xml-conversion-helper/plugin.xml
        src2/xml-conversion-helper/src/commons-logging.properties

        Show
        Paul McMahan added a comment - One of the questions on the IP clearance form is: "Check and make sure that the files that have been donated have been updated to reflect the new ASF copyright." These java source files don't contain an ASF copyright header: src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/Anybody.java src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/DigestCallback.java src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/GeronimoLoginModule.java src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/GeronimoSimpleGroup.java src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/Nobody.java src/dependence-plugin/src/org/apache/apps/j2gmigrator/compatibility/PasswordHasher.java src2/ejb-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/ejb/TestEJBDescriptorTool.java src2/enterprise-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/app/TestApplicationDescriptorTool.java src2/web-migration-plugin/test/org/apache/apps/j2gmigrator/descriptors/web/TestWebDescriptorTool.java src2/xml-conversion-helper/test/org/apache/apps/j2gmigrator/descriptors/xmlutil/TestXMLConversionHelper.java src2/xml-conversion-helper/test/org/apache/apps/j2gmigrator/resources/xmlutil/test/XMLConvertionHelperTestCase.java Also, I noticed these non-java files don't contain an ASF copyright header: src/common/build.properties src/common/plugin.xml src/configurator/bin/com/ibm/j2g/Configurator.class src/dependence-plugin/build.properties src/dependence-plugin/plugin.xml src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/class_analogies.properties src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/compatible_sources.properties src/dependence-plugin/src/org/apache/apps/j2gmigrator/source/dependence/default_jars.properties src/descriptors-tool/build.properties src/descriptors-tool/plugin.xml src/descriptors-tool/schema/tool.migrations.exsd src/ejb-environment-plugin/build.properties src/ejb-environment-plugin/plugin.xml src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-interfaces.properties src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-references.properties src/ejb-environment-plugin/src/org/apache/apps/j2gmigrator/sources/env/beans-types.properties src/jasper-plugin/build.properties src/jasper-plugin/plugin.xml src/resources-tool/build.properties src/resources-tool/plugin.xml src/resources-tool/schema/tool.migrations.exsd src/scripts/j2g-configure.cmd src/scripts/jdesc2g.cmd src/scripts/jres2g.cmd src/scripts/jsrc2g.cmd src/sources-tool/build.properties src/sources-tool/plugin.xml src/sources-tool/schema/migrations.exsd src2/data-source-plugin/build.properties src2/data-source-plugin/plugin.xml src2/data-source-plugin/src/commons-logging.properties src2/data-source-plugin/test-apps/ds/hsqldb-ds.xml src2/data-source-plugin/test-apps/ds/hsqldb-geronimo-plan.xml src2/data-source-plugin/test-apps/ds/mysql-ds.xml src2/data-source-plugin/test-apps/ds/mysql-geronimo-plan.xml src2/data-source-plugin/test-apps/ds/oracle-ds.xml src2/data-source-plugin/test-apps/ds/oracle-geronimo-plan.xml src2/ejb-cmp-migration-plugin/build.properties src2/ejb-cmp-migration-plugin/plugin.xml src2/ejb-cmp-migration-plugin/src/commons-logging.properties src2/ejb-cmp-migration-plugin/test-resources/ejb-jar.xml src2/ejb-cmp-migration-plugin/test-resources/jbosscmp-jdbc.xml src2/ejb-migration-plugin/build.properties src2/ejb-migration-plugin/plugin.xml src2/ejb-migration-plugin/src/common-logging.properties src2/ejb-migration-plugin/test-resources/jboss.xml src2/enterprise-migration-plugin/.settings/org.eclipse.ltk.core.refactoring.prefs src2/enterprise-migration-plugin/build.properties src2/enterprise-migration-plugin/plugin.xml src2/enterprise-migration-plugin/src/commons-logging.properties src2/enterprise-migration-plugin/test-resources/geronimo-application.xml src2/enterprise-migration-plugin/test-resources/jboss-app.xml src2/jms-plugin/build.properties src2/jms-plugin/plugin.xml src2/jms-plugin/src/commons-logging.properties src2/jms-plugin/test-apps/jms/geronimo-jms-resource-plan.xml src2/jms-plugin/test-apps/jms/jbossmq-destinations-service.xml src2/jms-plugin/test-apps/jms/jms-ds.xml src2/jms-plugin/test-apps/jms/jms-geronimo-plan.xml src2/mail-plugin/build.properties src2/mail-plugin/plugin.xml src2/mail-plugin/src/commons-logging.properties src2/mail-plugin/test-apps/mail/err/mail-service.xml src2/mail-plugin/test-apps/mail/geronimo-mail-service.xml src2/mail-plugin/test-apps/mail/mail-geronimo-plan.xml src2/mail-plugin/test-apps/mail/mail-service.xml src2/security-plugin/build.properties src2/security-plugin/plugin.xml src2/security-plugin/src/commons-logging.properties src2/security-plugin/test-apps/security/geronimo-secutiry-plan.xml src2/security-plugin/test-apps/security/login-config.xml src2/security-plugin/test-apps/security/security-geronimo-plan.xml src2/web-migration-plugin/build.properties src2/web-migration-plugin/plugin.xml src2/web-migration-plugin/src/commons-logging.properties src2/web-migration-plugin/test-resources/jboss-web.xml src2/xml-conversion-helper/.settings/org.eclipse.ltk.core.refactoring.prefs src2/xml-conversion-helper/build.properties src2/xml-conversion-helper/plugin.xml src2/xml-conversion-helper/src/commons-logging.properties
        Hide
        Jeffrey Faelnar added a comment -

        Files listed by Paul has been updated to reflect the new ASF copyright.

        Show
        Jeffrey Faelnar added a comment - Files listed by Paul has been updated to reflect the new ASF copyright.
        Jeffrey Faelnar made changes -
        Hide
        Paul McMahan added a comment -

        committed to sandbox, but did not commit the word documents that contain the "IBM Confidential" footer. if one of the contributors wants to remove that footer and reattach then I will commit those to sandbox as well

        Show
        Paul McMahan added a comment - committed to sandbox, but did not commit the word documents that contain the "IBM Confidential" footer. if one of the contributors wants to remove that footer and reattach then I will commit those to sandbox as well
        Paul McMahan made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 1.1.x [ 12312003 ]
        Assignee Paul McMahan [ pmcmahan ]
        Status Open [ 1 ] Closed [ 6 ]
        Hide
        Paul McMahan added a comment -

        reopening the issue so the contributor can attach a zip with the COPYRIGHT.txt removed

        Show
        Paul McMahan added a comment - reopening the issue so the contributor can attach a zip with the COPYRIGHT.txt removed
        Paul McMahan made changes -
        Status Closed [ 6 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        Hide
        Jeffrey Faelnar added a comment -

        COPYRIGHT.txt file removed.

        Show
        Jeffrey Faelnar added a comment - COPYRIGHT.txt file removed.
        Jeffrey Faelnar made changes -
        Hide
        Paul McMahan added a comment -

        Thanks for the updated attachment. I ended up just removing the "IBM Confidential" line from the COPYRIGHT.txt instead of deleting the file, which I think was fine to do now that you have provided an additional attachment that does not contain the file at all.

        Show
        Paul McMahan added a comment - Thanks for the updated attachment. I ended up just removing the "IBM Confidential" line from the COPYRIGHT.txt instead of deleting the file, which I think was fine to do now that you have provided an additional attachment that does not contain the file at all.
        Paul McMahan made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Paul McMahan
            Reporter:
            Filip Hanik
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development