You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by di...@apache.org on 2002/10/25 18:07:49 UTC

cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

dims        2002/10/25 09:07:49

  Modified:    java/src/org/apache/axis/wsdl Java2WSDL.java
               java/src/org/apache/axis/wsdl/fromJava Namespaces.java
  Log:
  Fix one more "ant clean all-tests" failure.
  
  Revision  Changes    Path
  1.34      +1 -1      xml-axis/java/src/org/apache/axis/wsdl/Java2WSDL.java
  
  Index: Java2WSDL.java
  ===================================================================
  RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/Java2WSDL.java,v
  retrieving revision 1.33
  retrieving revision 1.34
  diff -u -r1.33 -r1.34
  --- Java2WSDL.java	18 Oct 2002 20:54:28 -0000	1.33
  +++ Java2WSDL.java	25 Oct 2002 16:07:48 -0000	1.34
  @@ -316,7 +316,7 @@
           case PACKAGE_OPT:
               String packageName = option.getArgument(0);
               String namespace = option.getArgument(1);
  -            namespaceMap.put(packageName, namespace);
  +            namespaceMap.put(namespace, packageName);
               break;
               
           case NAMESPACE_OPT:
  
  
  
  1.4       +1 -1      xml-axis/java/src/org/apache/axis/wsdl/fromJava/Namespaces.java
  
  Index: Namespaces.java
  ===================================================================
  RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/fromJava/Namespaces.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- Namespaces.java	9 Apr 2002 15:41:30 -0000	1.3
  +++ Namespaces.java	25 Oct 2002 16:07:49 -0000	1.4
  @@ -131,7 +131,7 @@
           Iterator i = map.entrySet().iterator();
           while (i.hasNext()) {
               Map.Entry entry = (Map.Entry) i.next();
  -            put(entry.getKey(), entry.getValue(), null);
  +            put(entry.getValue(), entry.getKey(), null);
           }
       }
   
  
  
  

Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

Posted by Steve Loughran <st...@iseran.com>.
----- Original Message -----
From: "Davanum Srinivas" <di...@yahoo.com>
To: <ax...@xml.apache.org>
Sent: Friday, October 25, 2002 12:59
Subject: Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
Namespaces.java


> Thanks Steve.

no, thank you for putting up with the break. I'm going to have to sit down
and work out why the tests dont work properly with http proxy settings
defined; I guess its potentially forking jvms or something. I think the
fundamental cause is that the windows JVM doesnt default to getting its
settings from IE, even though webstart and the applet runtime somehow manage
to do it. Instead we need to hack it by hand and have it propagate.

- Who has to set up their systems to run functional tests through proxies?
- How do they do it? ANT_OPTS?, some other trick.
- Ant 1.5has <setproxy> to set socks or http proxy, but it doesnt handle
authenticated proxies (yet). It does the right thing across all known ant
jvms tho'. We could insert proxy setting that way if it would work for
everyone.

The other trick would be to split online and offline tests, have a
<condition> to probe something like apache.com for being reachable, and skip
all online tests if we are offline. there are hints of a plan for this
somewhere, with test.functional.remote being the switch -it just isnt active
any more.





Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

Posted by Davanum Srinivas <di...@yahoo.com>.
Thanks Steve.

--- Steve Loughran <st...@iseran.com> wrote:
> 
> ----- Original Message -----
> From: "Davanum Srinivas" <di...@yahoo.com>
> To: <ax...@xml.apache.org>
> Sent: Friday, October 25, 2002 10:15 AM
> Subject: Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
> Namespaces.java
> 
> 
> > Steve,
> >
> > Am just patching up your commit....Here's the problem: WSDL2Java's emitter
> needs Key=URI and
> > Value=PackageName, whereas Java2WSDL's emitter needs Key=PackageName and
> Value=URI. See putAll in
> > the fromJava's Namespaces.java and toJava's Namespaces.java. Your
> check-in's this morning broke
> > the build. First WSDL2Java did not work which was fixed by
> > http://marc.theaimsgroup.com/?l=axis-dev&m=103555190010662&w=2. Then
> Java2WSDL in the
> > interop3.GroupE would not work which was fixed by
> > http://marc.theaimsgroup.com/?l=axis-dev&m=103556207822987&w=2.
> >
> > To tell you the truth, I am really patching up without thinking too much.
> As i have already spent
> > 4-5 hours debugging your check-in. Please feel free to reverse my changes
> if you feel my "fixes"
> > are bad.
> 
> I have rolled them back and fixed the fundamental cause.
> 
> >
> > *********** PLEASE DON'T CHECK-IN CODE WITHOUT RUNNING "ant clean
> all-tests" ************
> 
> I have, but the tests bail out with connectivity problems, despite all I
> have done to try and set up the proxy:
> ANT_OPTS= -Dhttp.proxyHost=web-proxy -Dhttp.proxyPort=8088
> 
> 
>      [java] [wsdl2java] java.net.ConnectException: Connection timed out:
> connect
> 
>      [java] [java2wsdl] Java2WSDL
> test.wsdl.interop3.import1.definitions.SoapInt
> eropImport1PortType
>      [java] [java2wsdl] java.lang.ClassNotFoundException:
> test.wsdl.interop3.imp
> ort1.definitions.SoapInteropImport1PortType
>      [java] [java2wsdl]         at
> java.net.URLClassLoader$1.run(URLClassLoader.
> java:198)
> 
> 1. we need tests to have a proper off-line mode
> 
> 2. how do people set up their proxies so they work properly?
> 


=====
Davanum Srinivas - http://xml.apache.org/~dims/

__________________________________________________
Do you Yahoo!?
Y! Web Hosting - Let the expert host your web site
http://webhosting.yahoo.com/

Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

Posted by Steve Loughran <st...@iseran.com>.
----- Original Message -----
From: "Davanum Srinivas" <di...@yahoo.com>
To: <ax...@xml.apache.org>
Sent: Friday, October 25, 2002 10:15 AM
Subject: Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
Namespaces.java


> Steve,
>
> Am just patching up your commit....Here's the problem: WSDL2Java's emitter
needs Key=URI and
> Value=PackageName, whereas Java2WSDL's emitter needs Key=PackageName and
Value=URI. See putAll in
> the fromJava's Namespaces.java and toJava's Namespaces.java. Your
check-in's this morning broke
> the build. First WSDL2Java did not work which was fixed by
> http://marc.theaimsgroup.com/?l=axis-dev&m=103555190010662&w=2. Then
Java2WSDL in the
> interop3.GroupE would not work which was fixed by
> http://marc.theaimsgroup.com/?l=axis-dev&m=103556207822987&w=2.
>
> To tell you the truth, I am really patching up without thinking too much.
As i have already spent
> 4-5 hours debugging your check-in. Please feel free to reverse my changes
if you feel my "fixes"
> are bad.

I have rolled them back and fixed the fundamental cause.

>
> *********** PLEASE DON'T CHECK-IN CODE WITHOUT RUNNING "ant clean
all-tests" ************

I have, but the tests bail out with connectivity problems, despite all I
have done to try and set up the proxy:
ANT_OPTS= -Dhttp.proxyHost=web-proxy -Dhttp.proxyPort=8088


     [java] [wsdl2java] java.net.ConnectException: Connection timed out:
connect

     [java] [java2wsdl] Java2WSDL
test.wsdl.interop3.import1.definitions.SoapInt
eropImport1PortType
     [java] [java2wsdl] java.lang.ClassNotFoundException:
test.wsdl.interop3.imp
ort1.definitions.SoapInteropImport1PortType
     [java] [java2wsdl]         at
java.net.URLClassLoader$1.run(URLClassLoader.
java:198)

1. we need tests to have a proper off-line mode

2. how do people set up their proxies so they work properly?


Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

Posted by Davanum Srinivas <di...@yahoo.com>.
Steve,

Am just patching up your commit....Here's the problem: WSDL2Java's emitter needs Key=URI and
Value=PackageName, whereas Java2WSDL's emitter needs Key=PackageName and Value=URI. See putAll in
the fromJava's Namespaces.java and toJava's Namespaces.java. Your check-in's this morning broke
the build. First WSDL2Java did not work which was fixed by
http://marc.theaimsgroup.com/?l=axis-dev&m=103555190010662&w=2. Then Java2WSDL in the
interop3.GroupE would not work which was fixed by
http://marc.theaimsgroup.com/?l=axis-dev&m=103556207822987&w=2. 

To tell you the truth, I am really patching up without thinking too much. As i have already spent
4-5 hours debugging your check-in. Please feel free to reverse my changes if you feel my "fixes"
are bad. 

*********** PLEASE DON'T CHECK-IN CODE WITHOUT RUNNING "ant clean all-tests" ************

Thanks,
dims




The first check-in rectified this problem.
http://marc.theaimsgroup.com/?l=axis-dev&m=103555190010662&w=2

--- Steve Loughran <st...@iseran.com> wrote:
> 
> ----- Original Message -----
> From: <di...@apache.org>
> To: <xm...@apache.org>
> Sent: Friday, October 25, 2002 9:07 AM
> Subject: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
> Namespaces.java
> 
> 
> > dims        2002/10/25 09:07:49
> >
> >   Modified:    java/src/org/apache/axis/wsdl Java2WSDL.java
> >                java/src/org/apache/axis/wsdl/fromJava Namespaces.java
> >   Log:
> >   Fix one more "ant clean all-tests" failure.
> 
> now I'm really confused.
> This inversion is in code that I didnt touch, which implies the real defect
> was actually elsewhere in my <mapping> class, which has mixed up key and
> value .
> 
> 


=====
Davanum Srinivas - http://xml.apache.org/~dims/

__________________________________________________
Do you Yahoo!?
Y! Web Hosting - Let the expert host your web site
http://webhosting.yahoo.com/

Re: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava Namespaces.java

Posted by Steve Loughran <st...@iseran.com>.
----- Original Message -----
From: <di...@apache.org>
To: <xm...@apache.org>
Sent: Friday, October 25, 2002 9:07 AM
Subject: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
Namespaces.java


> dims        2002/10/25 09:07:49
>
>   Modified:    java/src/org/apache/axis/wsdl Java2WSDL.java
>                java/src/org/apache/axis/wsdl/fromJava Namespaces.java
>   Log:
>   Fix one more "ant clean all-tests" failure.

now I'm really confused.
This inversion is in code that I didnt touch, which implies the real defect
was actually elsewhere in my <mapping> class, which has mixed up key and
value .