I think that should be part of the name change work.

On Nov 9, 2008, at 7:07 PM, Dennis Reedy wrote:
Although not documented on RIVER-261, should the com.artima.lookup.util package be changed to org.apache.river.lookup.util?

On Nov 9, 2008, at 236PM, Dennis Reedy wrote:

I noticed something in the IiopExporter, starting on line 206:

// workaround for 4632973
        ArrayList names = new ArrayList(2);
        names.add(tieClassName);
        if (tieClassName.startsWith("java.") ||
            tieClassName.startsWith("com.sun.") ||
            tieClassName.startsWith("net.jini.") ||
            tieClassName.startsWith("jini.") ||
            tieClassName.startsWith("javax."))
        {
            names.add("org.omg.stub." + tieClassName);
        }


What is the issue that 4632973 referring to? Does an additional

tieClassName.startsWith("org.apache.river")

Need to be added for the namespace change, or does the "net.jini" test cover this?

Dennis

On Nov 9, 2008, at 136PM, Dennis Reedy wrote:

Hi,

I've made the changes locally for http://issues.apache.org/jira/browse/RIVER-261 , where com.sun.jini.* has been converted to org.apache.river.* Would anyone like this as a patch? Its such a big change in the codebase, what would the right way be to proceed?

Also, I'd like to make sure the testing framework gets modified as well, where can I find that?

Regards

Dennis



Reply via email to