Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [gemini-dev] Patch for Gemini JPA Bug 330979

Changes checked in after review by Tom Ware.

    Shaun

On 24/11/2010 9:58 AM, Shaun Smith wrote:
I've run all the Gemini JPA tests and my own smoke tests.  The only place I can find where JPAInitializer.BUNDLE_RESOURCE_PREFIX is used is MetadataAsmFactory.buildClassMetadata().  The tests never go down the branch that leads to it's use.  Given that the constant is defined as "" I don't see how it's useful anyway.

    Shaun

On 24/11/2010 9:19 AM, Tom Ware wrote:
Hi Shaun,

  I am surprised this works without maintaining the JPAInitializer.BUNDLE_RESOURCE_PREFIX string.  There are a couple of places in the EclipseLink metadata processing code where we use this string to find xml files in the jar.  How much testing have you run?

-Tom

Shaun Smith wrote:
  A patch is ready for review for Bug 330979 <https://bugs.eclipse.org/bugs/show_bug.cgi?id=330979> - Support getEntry on installed bundles in PDE dev mode.
https://bugs.eclipse.org/bugs/show_bug.cgi?id=330979

Tom,

    I updated your recently contributed BundleArchive class to also make it pick up the bin path specified in the Eclipse .classpath file.  Can you review the patch to make sure you're happy with it?

        Shaun

 --
Oracle <http://www.oracle.com>
Shaun Smith | Principal Product Manager
Phone: +1.905.502.3094 | Mobile: +1.416.558.6244
Oracle Fusion Middleware, TopLink
ORACLE Canada | 110 Matheson Boulevard West, Suite 100, Mississauga, Ontario | L5R 3P4
Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment

--
Oracle
Shaun Smith | Principal Product Manager
Phone: +1.905.502.3094 | Mobile: +1.416.558.6244
Oracle Fusion Middleware, TopLink
ORACLE Canada | 110 Matheson Boulevard West, Suite 100, Mississauga, Ontario | L5R 3P4

Green
            Oracle Oracle is committed to developing practices and products that help protect the environment
_______________________________________________ gemini-dev mailing list gemini-dev@xxxxxxxxxxx https://dev.eclipse.org/mailman/listinfo/gemini-dev

--
Oracle
Shaun Smith | Principal Product Manager
Phone: +1.905.502.3094 | Mobile: +1.416.558.6244
Oracle Fusion Middleware, TopLink
ORACLE Canada | 110 Matheson Boulevard West, Suite 100, Mississauga, Ontario | L5R 3P4

Green
          Oracle Oracle is committed to developing practices and products that help protect the environment

Back to the top