You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by Tony Wu <wu...@gmail.com> on 2007/02/28 04:11:01 UTC
[classlib][jndi]anyone see this compile error?
I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
then I got following compile error.
[javac] 1. ERROR in
/home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
[javac] (at line 49)
[javac] suite.addTest(new TestSuite(TestBasicControl.class));
[javac] ^^^^^^^^^^^^^^^^
[javac] TestBasicControl cannot be resolved to a type
[javac] ----------
[javac] 1 problem (1 error)
I did not find the TestBasicControl.java in jndi src.
--
Tony Wu
China Software Development Lab, IBM
Re: [classlib][jndi]anyone see this compile error?
Posted by "Geir Magnusson Jr." <ge...@pobox.com>.
(I can't believe I'm defending them :)
*I* thought that some people were using them w/ their IDEs.
I'd be happy to leave them alone, but just as happy to see them go.
If no one stands up and says they use them, we should probably remove.
geir
On Mar 1, 2007, at 2:45 AM, Mikhail Loenko wrote:
> I'll remove them from jndi module
>
> Thanks,
> Mikhail
>
> 2007/3/1, Richard Liang <ri...@gmail.com>:
>> On 3/1/07, Nathan Beyer <nd...@apache.org> wrote:
>> > I'd vote for deleting any extraneous AllTests files.
>>
>> +1
>>
>> >
>> > On 2/28/07, Alexei Zakharov <al...@gmail.com> wrote:
>> > > > I think the AllTest makes sense only when some one want to
>> run test
>> > > > suites partly. I had never used it but I'm not sure if there is
>> > > > somebody who perfer that way.
>> > >
>> > > Well, seems AllTest is a history now. It doesn't make use of
>> exclude
>> > > lists and etc. Moreover, some modules does not support it -
>> i.e. does
>> > > not have AllTest.. IIRC we have agreed that storing grouping
>> > > information in java source is (generally) not a very good idea.
>> > >
>> > > Thanks,
>> > >
>> > > 2007/2/28, Tony Wu <wu...@gmail.com>:
>> > > > On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
>> > > > > Thanks, Stepan
>> > > > >
>> > > > > Seems like the key is "cleaned my working copy" here.
>> Because I saw no
>> > > > > problems when tested my commit.
>> > > > >
>> > > > yes, there might be a TestBasicControl.class in classpath if
>> did not clean.
>> > > >
>> > > > > BTW, do these "AllTest" make sense? Does anybody use them?
>> > > > > We do'nt have all the tests included into "AllTest", so it
>> might make
>> > > > > sense to get rid of them
>> > > > >
>> > > >
>> > > > I think the AllTest makes sense only when some one want to
>> run test
>> > > > suites partly. I had never used it but I'm not sure if there is
>> > > > somebody who perfer that way.
>> > > >
>> > > > > Thanks,
>> > > > > Mikhail
>> > > > >
>> > > > > 2007/2/28, Stepan Mishura <st...@gmail.com>:
>> > > > > > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
>> > > > > > >
>> > > > > > > I cleaned my working copy and ran "ant -
>> Dbuild.module=jndi test",
>> > > > > > > then I got following compile error.
>> > > > > >
>> > > > > >
>> > > > > > Fixed at r512588.
>> > > > > >
>> > > > > > -Stepan.
>> > > > > >
>> > > > > > [javac] 1. ERROR in
>> > > > > > >
>> > > > > > > /home/clear/trunk/modules/jndi/src/test/java/org/
>> apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
>> > > > > > > [javac] (at line 49)
>> > > > > > > [javac] suite.addTest(new TestSuite
>> (TestBasicControl.class));
>> > > > > > > [javac]
>> ^^^^^^^^^^^^^^^^
>> > > > > > > [javac] TestBasicControl cannot be resolved to a type
>> > > > > > > [javac] ----------
>> > > > > > > [javac] 1 problem (1 error)
>> > > > > > >
>> > > > > > > I did not find the TestBasicControl.java in jndi src.
>> > > > > > >
>> > >
>> > > --
>> > > Alexei Zakharov,
>> > > Intel ESSD
>> > >
>> >
>>
>>
>> --
>> Richard Liang
>> China Development Lab, IBM
>>
Re: [classlib][jndi]anyone see this compile error?
Posted by Mikhail Loenko <ml...@gmail.com>.
I'll remove them from jndi module
Thanks,
Mikhail
2007/3/1, Richard Liang <ri...@gmail.com>:
> On 3/1/07, Nathan Beyer <nd...@apache.org> wrote:
> > I'd vote for deleting any extraneous AllTests files.
>
> +1
>
> >
> > On 2/28/07, Alexei Zakharov <al...@gmail.com> wrote:
> > > > I think the AllTest makes sense only when some one want to run test
> > > > suites partly. I had never used it but I'm not sure if there is
> > > > somebody who perfer that way.
> > >
> > > Well, seems AllTest is a history now. It doesn't make use of exclude
> > > lists and etc. Moreover, some modules does not support it - i.e. does
> > > not have AllTest.. IIRC we have agreed that storing grouping
> > > information in java source is (generally) not a very good idea.
> > >
> > > Thanks,
> > >
> > > 2007/2/28, Tony Wu <wu...@gmail.com>:
> > > > On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
> > > > > Thanks, Stepan
> > > > >
> > > > > Seems like the key is "cleaned my working copy" here. Because I saw no
> > > > > problems when tested my commit.
> > > > >
> > > > yes, there might be a TestBasicControl.class in classpath if did not clean.
> > > >
> > > > > BTW, do these "AllTest" make sense? Does anybody use them?
> > > > > We do'nt have all the tests included into "AllTest", so it might make
> > > > > sense to get rid of them
> > > > >
> > > >
> > > > I think the AllTest makes sense only when some one want to run test
> > > > suites partly. I had never used it but I'm not sure if there is
> > > > somebody who perfer that way.
> > > >
> > > > > Thanks,
> > > > > Mikhail
> > > > >
> > > > > 2007/2/28, Stepan Mishura <st...@gmail.com>:
> > > > > > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> > > > > > >
> > > > > > > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > > > > > > then I got following compile error.
> > > > > >
> > > > > >
> > > > > > Fixed at r512588.
> > > > > >
> > > > > > -Stepan.
> > > > > >
> > > > > > [javac] 1. ERROR in
> > > > > > >
> > > > > > > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > > > > > > [javac] (at line 49)
> > > > > > > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > > > > > > [javac] ^^^^^^^^^^^^^^^^
> > > > > > > [javac] TestBasicControl cannot be resolved to a type
> > > > > > > [javac] ----------
> > > > > > > [javac] 1 problem (1 error)
> > > > > > >
> > > > > > > I did not find the TestBasicControl.java in jndi src.
> > > > > > >
> > >
> > > --
> > > Alexei Zakharov,
> > > Intel ESSD
> > >
> >
>
>
> --
> Richard Liang
> China Development Lab, IBM
>
Re: [classlib][jndi]anyone see this compile error?
Posted by Richard Liang <ri...@gmail.com>.
On 3/1/07, Nathan Beyer <nd...@apache.org> wrote:
> I'd vote for deleting any extraneous AllTests files.
+1
>
> On 2/28/07, Alexei Zakharov <al...@gmail.com> wrote:
> > > I think the AllTest makes sense only when some one want to run test
> > > suites partly. I had never used it but I'm not sure if there is
> > > somebody who perfer that way.
> >
> > Well, seems AllTest is a history now. It doesn't make use of exclude
> > lists and etc. Moreover, some modules does not support it - i.e. does
> > not have AllTest.. IIRC we have agreed that storing grouping
> > information in java source is (generally) not a very good idea.
> >
> > Thanks,
> >
> > 2007/2/28, Tony Wu <wu...@gmail.com>:
> > > On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
> > > > Thanks, Stepan
> > > >
> > > > Seems like the key is "cleaned my working copy" here. Because I saw no
> > > > problems when tested my commit.
> > > >
> > > yes, there might be a TestBasicControl.class in classpath if did not clean.
> > >
> > > > BTW, do these "AllTest" make sense? Does anybody use them?
> > > > We do'nt have all the tests included into "AllTest", so it might make
> > > > sense to get rid of them
> > > >
> > >
> > > I think the AllTest makes sense only when some one want to run test
> > > suites partly. I had never used it but I'm not sure if there is
> > > somebody who perfer that way.
> > >
> > > > Thanks,
> > > > Mikhail
> > > >
> > > > 2007/2/28, Stepan Mishura <st...@gmail.com>:
> > > > > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> > > > > >
> > > > > > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > > > > > then I got following compile error.
> > > > >
> > > > >
> > > > > Fixed at r512588.
> > > > >
> > > > > -Stepan.
> > > > >
> > > > > [javac] 1. ERROR in
> > > > > >
> > > > > > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > > > > > [javac] (at line 49)
> > > > > > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > > > > > [javac] ^^^^^^^^^^^^^^^^
> > > > > > [javac] TestBasicControl cannot be resolved to a type
> > > > > > [javac] ----------
> > > > > > [javac] 1 problem (1 error)
> > > > > >
> > > > > > I did not find the TestBasicControl.java in jndi src.
> > > > > >
> >
> > --
> > Alexei Zakharov,
> > Intel ESSD
> >
>
--
Richard Liang
China Development Lab, IBM
Re: [classlib][jndi]anyone see this compile error?
Posted by Nathan Beyer <nd...@apache.org>.
I'd vote for deleting any extraneous AllTests files.
On 2/28/07, Alexei Zakharov <al...@gmail.com> wrote:
> > I think the AllTest makes sense only when some one want to run test
> > suites partly. I had never used it but I'm not sure if there is
> > somebody who perfer that way.
>
> Well, seems AllTest is a history now. It doesn't make use of exclude
> lists and etc. Moreover, some modules does not support it - i.e. does
> not have AllTest.. IIRC we have agreed that storing grouping
> information in java source is (generally) not a very good idea.
>
> Thanks,
>
> 2007/2/28, Tony Wu <wu...@gmail.com>:
> > On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
> > > Thanks, Stepan
> > >
> > > Seems like the key is "cleaned my working copy" here. Because I saw no
> > > problems when tested my commit.
> > >
> > yes, there might be a TestBasicControl.class in classpath if did not clean.
> >
> > > BTW, do these "AllTest" make sense? Does anybody use them?
> > > We do'nt have all the tests included into "AllTest", so it might make
> > > sense to get rid of them
> > >
> >
> > I think the AllTest makes sense only when some one want to run test
> > suites partly. I had never used it but I'm not sure if there is
> > somebody who perfer that way.
> >
> > > Thanks,
> > > Mikhail
> > >
> > > 2007/2/28, Stepan Mishura <st...@gmail.com>:
> > > > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> > > > >
> > > > > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > > > > then I got following compile error.
> > > >
> > > >
> > > > Fixed at r512588.
> > > >
> > > > -Stepan.
> > > >
> > > > [javac] 1. ERROR in
> > > > >
> > > > > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > > > > [javac] (at line 49)
> > > > > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > > > > [javac] ^^^^^^^^^^^^^^^^
> > > > > [javac] TestBasicControl cannot be resolved to a type
> > > > > [javac] ----------
> > > > > [javac] 1 problem (1 error)
> > > > >
> > > > > I did not find the TestBasicControl.java in jndi src.
> > > > >
>
> --
> Alexei Zakharov,
> Intel ESSD
>
Re: [classlib][jndi]anyone see this compile error?
Posted by Alexei Zakharov <al...@gmail.com>.
> I think the AllTest makes sense only when some one want to run test
> suites partly. I had never used it but I'm not sure if there is
> somebody who perfer that way.
Well, seems AllTest is a history now. It doesn't make use of exclude
lists and etc. Moreover, some modules does not support it - i.e. does
not have AllTest.. IIRC we have agreed that storing grouping
information in java source is (generally) not a very good idea.
Thanks,
2007/2/28, Tony Wu <wu...@gmail.com>:
> On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
> > Thanks, Stepan
> >
> > Seems like the key is "cleaned my working copy" here. Because I saw no
> > problems when tested my commit.
> >
> yes, there might be a TestBasicControl.class in classpath if did not clean.
>
> > BTW, do these "AllTest" make sense? Does anybody use them?
> > We do'nt have all the tests included into "AllTest", so it might make
> > sense to get rid of them
> >
>
> I think the AllTest makes sense only when some one want to run test
> suites partly. I had never used it but I'm not sure if there is
> somebody who perfer that way.
>
> > Thanks,
> > Mikhail
> >
> > 2007/2/28, Stepan Mishura <st...@gmail.com>:
> > > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> > > >
> > > > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > > > then I got following compile error.
> > >
> > >
> > > Fixed at r512588.
> > >
> > > -Stepan.
> > >
> > > [javac] 1. ERROR in
> > > >
> > > > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > > > [javac] (at line 49)
> > > > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > > > [javac] ^^^^^^^^^^^^^^^^
> > > > [javac] TestBasicControl cannot be resolved to a type
> > > > [javac] ----------
> > > > [javac] 1 problem (1 error)
> > > >
> > > > I did not find the TestBasicControl.java in jndi src.
> > > >
--
Alexei Zakharov,
Intel ESSD
Re: [classlib][jndi]anyone see this compile error?
Posted by Tony Wu <wu...@gmail.com>.
On 2/28/07, Mikhail Loenko <ml...@gmail.com> wrote:
> Thanks, Stepan
>
> Seems like the key is "cleaned my working copy" here. Because I saw no
> problems when tested my commit.
>
yes, there might be a TestBasicControl.class in classpath if did not clean.
> BTW, do these "AllTest" make sense? Does anybody use them?
> We do'nt have all the tests included into "AllTest", so it might make
> sense to get rid of them
>
I think the AllTest makes sense only when some one want to run test
suites partly. I had never used it but I'm not sure if there is
somebody who perfer that way.
> Thanks,
> Mikhail
>
> 2007/2/28, Stepan Mishura <st...@gmail.com>:
> > On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> > >
> > > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > > then I got following compile error.
> >
> >
> > Fixed at r512588.
> >
> > -Stepan.
> >
> > [javac] 1. ERROR in
> > >
> > > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > > [javac] (at line 49)
> > > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > > [javac] ^^^^^^^^^^^^^^^^
> > > [javac] TestBasicControl cannot be resolved to a type
> > > [javac] ----------
> > > [javac] 1 problem (1 error)
> > >
> > > I did not find the TestBasicControl.java in jndi src.
> > >
> > > --
> > > Tony Wu
> > > China Software Development Lab, IBM
> > >
> >
> >
> >
> > --
> > Stepan Mishura
> > Intel Enterprise Solutions Software Division
> >
>
--
Tony Wu
China Software Development Lab, IBM
Re: [classlib][jndi]anyone see this compile error?
Posted by Mikhail Loenko <ml...@gmail.com>.
Thanks, Stepan
Seems like the key is "cleaned my working copy" here. Because I saw no
problems when tested my commit.
BTW, do these "AllTest" make sense? Does anybody use them?
We do'nt have all the tests included into "AllTest", so it might make
sense to get rid of them
Thanks,
Mikhail
2007/2/28, Stepan Mishura <st...@gmail.com>:
> On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
> >
> > I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> > then I got following compile error.
>
>
> Fixed at r512588.
>
> -Stepan.
>
> [javac] 1. ERROR in
> >
> > /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> > [javac] (at line 49)
> > [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> > [javac] ^^^^^^^^^^^^^^^^
> > [javac] TestBasicControl cannot be resolved to a type
> > [javac] ----------
> > [javac] 1 problem (1 error)
> >
> > I did not find the TestBasicControl.java in jndi src.
> >
> > --
> > Tony Wu
> > China Software Development Lab, IBM
> >
>
>
>
> --
> Stepan Mishura
> Intel Enterprise Solutions Software Division
>
Re: [classlib][jndi]anyone see this compile error?
Posted by Stepan Mishura <st...@gmail.com>.
On 2/28/07, Tony Wu <wu...@gmail.com> wrote:
>
> I cleaned my working copy and ran "ant -Dbuild.module=jndi test",
> then I got following compile error.
Fixed at r512588.
-Stepan.
[javac] 1. ERROR in
>
> /home/clear/trunk/modules/jndi/src/test/java/org/apache/harmony/jndi/tests/javax/naming/ldap/AllTestLdap.java
> [javac] (at line 49)
> [javac] suite.addTest(new TestSuite(TestBasicControl.class));
> [javac] ^^^^^^^^^^^^^^^^
> [javac] TestBasicControl cannot be resolved to a type
> [javac] ----------
> [javac] 1 problem (1 error)
>
> I did not find the TestBasicControl.java in jndi src.
>
> --
> Tony Wu
> China Software Development Lab, IBM
>
--
Stepan Mishura
Intel Enterprise Solutions Software Division