Re: [classlib] Build failure messages
I looked at adding the nowarn arg for the tests, but 'build-test.xml' doesn't have a single large compile statement like 'buid-java.xml' does. The arg would need to be added to the compile-test of every module's build, which can be done, I was just feeling lazy at the moment, sorry. :( Also, I had the thought of creating a compile indirection in the build scripts using an Ant macro. This way any global compiler tweaks could be performed in the macro. I haven't convinced myself of the value of that though. -Nathan On 10/20/06, Paulex Yang <[EMAIL PROTECTED]> wrote: Nathan Beyer wrote: > I added a compiler arg to the main 'build-java.xml' compile to disable > all of the warnings. Thanks a lot, Nathan, but how about also add that arg to build-test.xml? I think the ECJ probably feels more unhappy with the test cases:). > > This should take care of the warnings. > > -Nathan > > On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: >> Can we just shut them off for now? it's a good project for someone to >> go and clean those up, but there's no need for all of us to see them. >> >> geir >> >> Mark Hindess wrote: >> > These messages aren't getting through on the commits list on due to >> the >> > large number of warnings since switching to ecj. I've modified the >> > summary process so that it ignores these for the moment. >> > >> > Hopefully this means that the failure messages that haven't been >> making >> > the commits list will start getting through. >> > >> > Regards, >> > Mark. >> > >> > >> > >> > >> > - >> > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > For additional commands, e-mail: [EMAIL PROTECTED] >> > >> >> - >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Paulex Yang China Software Development Lab IBM - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Nathan Beyer wrote: I added a compiler arg to the main 'build-java.xml' compile to disable all of the warnings. Thanks a lot, Nathan, but how about also add that arg to build-test.xml? I think the ECJ probably feels more unhappy with the test cases:). This should take care of the warnings. -Nathan On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: Can we just shut them off for now? it's a good project for someone to go and clean those up, but there's no need for all of us to see them. geir Mark Hindess wrote: > These messages aren't getting through on the commits list on due to the > large number of warnings since switching to ecj. I've modified the > summary process so that it ignores these for the moment. > > Hopefully this means that the failure messages that haven't been making > the commits list will start getting through. > > Regards, > Mark. > > > > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Paulex Yang China Software Development Lab IBM - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Added patch for SUID wanings to the same issue.H-1926 2006/10/20, Tim Ellison <[EMAIL PROTECTED]>: Denis Kishenko wrote: > Almost all unused variables are local. Only two of them are class > fields and they are not using in native code. Build and all tests are > successful. Thanks Denis! Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Denis M. Kishenko Intel Middleware Products Division - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Denis Kishenko wrote: > Almost all unused variables are local. Only two of them are class > fields and they are not using in native code. Build and all tests are > successful. Thanks Denis! Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Almost all unused variables are local. Only two of them are class fields and they are not using in native code. Build and all tests are successful. 2006/10/20, Tim Ellison <[EMAIL PROTECTED]>: Denis Kishenko wrote: > I have made fix[1] for "unused" category (only java code). If nobody > has objections I could fix other caterogies. > > [1] http://issues.apache.org/jira/browse/HARMONY-1926 See Nathan's comments about unused vars -- they cannot be fixed safely with the Java wizard. Unused imports should be ok to fix that way. Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Denis M. Kishenko Intel Middleware Products Division - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
On 10/20/06, Tim Ellison <[EMAIL PROTECTED]> wrote: Denis Kishenko wrote: > I have made fix[1] for "unused" category (only java code). If nobody > has objections I could fix other caterogies. > > [1] http://issues.apache.org/jira/browse/HARMONY-1926 See Nathan's comments about unused vars -- they cannot be fixed safely with the Java wizard. Unused imports should be ok to fix that way. I see one step to be added to the unused vars removing: to check if 'unused var' is unused in the native code:) However I think it is related only for unused fields, local variables from subroutines and methods can be deleted without any checks. Regards, Ilya. Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- -- Ilya Okomin Intel Middleware Products Division
Re: [classlib] Build failure messages
Denis Kishenko wrote: > I have made fix[1] for "unused" category (only java code). If nobody > has objections I could fix other caterogies. > > [1] http://issues.apache.org/jira/browse/HARMONY-1926 See Nathan's comments about unused vars -- they cannot be fixed safely with the Java wizard. Unused imports should be ok to fix that way. Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Nathan Beyer wrote: > * unused - unused private fields, methods; these are tough as some of > them are seemingly used by native code yes, as you say some of these slots are used by native code to stash state; and some are there for serialization compatibility even though our impl doesn't use them. Each case will have to be determined (and documented in the code!) then neutralized with the suppress warnings annotation. Regards, Tim -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
I have made fix[1] for "unused" category (only java code). If nobody has objections I could fix other caterogies. [1] http://issues.apache.org/jira/browse/HARMONY-1926 2006/10/20, Nathan Beyer <[EMAIL PROTECTED]>: I've been trying to work through many of the warnings, module by module. Most of the warnings fall into the following categories. * serial - Serializable classes that don't have explicit serialVersionUID fields * unchecked - missing type variables, mostly collection references * unused - unused private fields, methods; these are tough as some of them are seemingly used by native code * missing @Override - methods that override other methods are annotated for safety -Nathan On 10/19/06, Tim Ellison <[EMAIL PROTECTED]> wrote: > Nathan has turned off all warnings. I suggest that we start to turn > them back on incrementally and fix the warnings we see. > > Regards, > Tim > > Denis Kishenko wrote: > > I also worried about a lot of warnings and I think warnings like these > > can be fixed > > > > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java > > > >[javac] (at line 26) > >[javac] import java.io.ObjectInputStream; > >[javac]^ > >[javac] The import java.io.ObjectInputStream is never used > > > >[javac] 34. WARNING in > > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java > > > >[javac] (at line 130) > >[javac] private static final class Sync extends > > AbstractQueuedSynchronizer { > >[javac] > >[javac] The serializable class Sync does not declare a static > > final serialVersionUID field of type long > > > > 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: > >> I added a compiler arg to the main 'build-java.xml' compile to disable > >> all of the warnings. > >> > >> This should take care of the warnings. > >> > >> -Nathan > >> > >> On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: > >> > Can we just shut them off for now? it's a good project for someone to > >> > go and clean those up, but there's no need for all of us to see them. > >> > > >> > geir > >> > > >> > Mark Hindess wrote: > >> > > These messages aren't getting through on the commits list on due > >> to the > >> > > large number of warnings since switching to ecj. I've modified the > >> > > summary process so that it ignores these for the moment. > >> > > > >> > > Hopefully this means that the failure messages that haven't been > >> making > >> > > the commits list will start getting through. > >> > > > >> > > Regards, > >> > > Mark. > >> > > > >> > > > >> > > > >> > > > >> > > - > >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html > >> > > To unsubscribe, e-mail: [EMAIL PROTECTED] > >> > > For additional commands, e-mail: > >> [EMAIL PROTECTED] > >> > > > >> > > >> > - > >> > Terms of use : http://incubator.apache.org/harmony/mailing.html > >> > To unsubscribe, e-mail: [EMAIL PROTECTED] > >> > For additional commands, e-mail: [EMAIL PROTECTED] > >> > > >> > > >> > >> - > >> Terms of use : http://incubator.apache.org/harmony/mailing.html > >> To unsubscribe, e-mail: [EMAIL PROTECTED] > >> For additional commands, e-mail: [EMAIL PROTECTED] > >> > >> > > > > > > -- > > Tim Ellison ([EMAIL PROTECTED]) > IBM Java technology centre, UK. > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Denis M. Kishenko Enterprise Solutions Software Division - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
You have patience I can only dream about... geir Nathan Beyer wrote: I've been trying to work through many of the warnings, module by module. Most of the warnings fall into the following categories. * serial - Serializable classes that don't have explicit serialVersionUID fields * unchecked - missing type variables, mostly collection references * unused - unused private fields, methods; these are tough as some of them are seemingly used by native code * missing @Override - methods that override other methods are annotated for safety -Nathan On 10/19/06, Tim Ellison <[EMAIL PROTECTED]> wrote: Nathan has turned off all warnings. I suggest that we start to turn them back on incrementally and fix the warnings we see. Regards, Tim Denis Kishenko wrote: > I also worried about a lot of warnings and I think warnings like these > can be fixed > > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java > >[javac] (at line 26) >[javac] import java.io.ObjectInputStream; >[javac]^ >[javac] The import java.io.ObjectInputStream is never used > >[javac] 34. WARNING in > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java > >[javac] (at line 130) >[javac] private static final class Sync extends > AbstractQueuedSynchronizer { >[javac] >[javac] The serializable class Sync does not declare a static > final serialVersionUID field of type long > > 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: >> I added a compiler arg to the main 'build-java.xml' compile to disable >> all of the warnings. >> >> This should take care of the warnings. >> >> -Nathan >> >> On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: >> > Can we just shut them off for now? it's a good project for someone to >> > go and clean those up, but there's no need for all of us to see them. >> > >> > geir >> > >> > Mark Hindess wrote: >> > > These messages aren't getting through on the commits list on due >> to the >> > > large number of warnings since switching to ecj. I've modified the >> > > summary process so that it ignores these for the moment. >> > > >> > > Hopefully this means that the failure messages that haven't been >> making >> > > the commits list will start getting through. >> > > >> > > Regards, >> > > Mark. >> > > >> > > >> > > >> > > >> > > - >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > For additional commands, e-mail: >> [EMAIL PROTECTED] >> > > >> > >> > - >> > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > For additional commands, e-mail: [EMAIL PROTECTED] >> > >> > >> >> - >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> > > -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
coolio - thx Nathan Beyer wrote: I added a compiler arg to the main 'build-java.xml' compile to disable all of the warnings. This should take care of the warnings. -Nathan On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: Can we just shut them off for now? it's a good project for someone to go and clean those up, but there's no need for all of us to see them. geir Mark Hindess wrote: > These messages aren't getting through on the commits list on due to the > large number of warnings since switching to ecj. I've modified the > summary process so that it ignores these for the moment. > > Hopefully this means that the failure messages that haven't been making > the commits list will start getting through. > > Regards, > Mark. > > > > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
I've been trying to work through many of the warnings, module by module. Most of the warnings fall into the following categories. * serial - Serializable classes that don't have explicit serialVersionUID fields * unchecked - missing type variables, mostly collection references * unused - unused private fields, methods; these are tough as some of them are seemingly used by native code * missing @Override - methods that override other methods are annotated for safety -Nathan On 10/19/06, Tim Ellison <[EMAIL PROTECTED]> wrote: Nathan has turned off all warnings. I suggest that we start to turn them back on incrementally and fix the warnings we see. Regards, Tim Denis Kishenko wrote: > I also worried about a lot of warnings and I think warnings like these > can be fixed > > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java > >[javac] (at line 26) >[javac] import java.io.ObjectInputStream; >[javac]^ >[javac] The import java.io.ObjectInputStream is never used > >[javac] 34. WARNING in > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java > >[javac] (at line 130) >[javac] private static final class Sync extends > AbstractQueuedSynchronizer { >[javac] >[javac] The serializable class Sync does not declare a static > final serialVersionUID field of type long > > 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: >> I added a compiler arg to the main 'build-java.xml' compile to disable >> all of the warnings. >> >> This should take care of the warnings. >> >> -Nathan >> >> On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: >> > Can we just shut them off for now? it's a good project for someone to >> > go and clean those up, but there's no need for all of us to see them. >> > >> > geir >> > >> > Mark Hindess wrote: >> > > These messages aren't getting through on the commits list on due >> to the >> > > large number of warnings since switching to ecj. I've modified the >> > > summary process so that it ignores these for the moment. >> > > >> > > Hopefully this means that the failure messages that haven't been >> making >> > > the commits list will start getting through. >> > > >> > > Regards, >> > > Mark. >> > > >> > > >> > > >> > > >> > > - >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > For additional commands, e-mail: >> [EMAIL PROTECTED] >> > > >> > >> > - >> > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > For additional commands, e-mail: [EMAIL PROTECTED] >> > >> > >> >> - >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> > > -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Nathan has turned off all warnings. I suggest that we start to turn them back on incrementally and fix the warnings we see. Regards, Tim Denis Kishenko wrote: > I also worried about a lot of warnings and I think warnings like these > can be fixed > > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java > >[javac] (at line 26) >[javac] import java.io.ObjectInputStream; >[javac]^ >[javac] The import java.io.ObjectInputStream is never used > >[javac] 34. WARNING in > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java > >[javac] (at line 130) >[javac] private static final class Sync extends > AbstractQueuedSynchronizer { >[javac] >[javac] The serializable class Sync does not declare a static > final serialVersionUID field of type long > > 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: >> I added a compiler arg to the main 'build-java.xml' compile to disable >> all of the warnings. >> >> This should take care of the warnings. >> >> -Nathan >> >> On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: >> > Can we just shut them off for now? it's a good project for someone to >> > go and clean those up, but there's no need for all of us to see them. >> > >> > geir >> > >> > Mark Hindess wrote: >> > > These messages aren't getting through on the commits list on due >> to the >> > > large number of warnings since switching to ecj. I've modified the >> > > summary process so that it ignores these for the moment. >> > > >> > > Hopefully this means that the failure messages that haven't been >> making >> > > the commits list will start getting through. >> > > >> > > Regards, >> > > Mark. >> > > >> > > >> > > >> > > >> > > - >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > For additional commands, e-mail: >> [EMAIL PROTECTED] >> > > >> > >> > - >> > Terms of use : http://incubator.apache.org/harmony/mailing.html >> > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > For additional commands, e-mail: [EMAIL PROTECTED] >> > >> > >> >> - >> Terms of use : http://incubator.apache.org/harmony/mailing.html >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> > > -- Tim Ellison ([EMAIL PROTECTED]) IBM Java technology centre, UK. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
+1 I never know if it's something important or not. On 10/19/06, Denis Kishenko <[EMAIL PROTECTED]> wrote: I also worried about a lot of warnings and I think warnings like these can be fixed last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java [javac] (at line 26) [javac] import java.io.ObjectInputStream; [javac]^ [javac] The import java.io.ObjectInputStream is never used [javac] 34. WARNING in C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java [javac] (at line 130) [javac] private static final class Sync extends AbstractQueuedSynchronizer { [javac] [javac] The serializable class Sync does not declare a static final serialVersionUID field of type long 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: > I added a compiler arg to the main 'build-java.xml' compile to disable > all of the warnings. > > This should take care of the warnings. > > -Nathan > > On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: > > Can we just shut them off for now? it's a good project for someone to > > go and clean those up, but there's no need for all of us to see them. > > > > geir > > > > Mark Hindess wrote: > > > These messages aren't getting through on the commits list on due to the > > > large number of warnings since switching to ecj. I've modified the > > > summary process so that it ignores these for the moment. > > > > > > Hopefully this means that the failure messages that haven't been making > > > the commits list will start getting through. > > > > > > Regards, > > > Mark. > > > > > > > > > > > > > > > - > > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > - > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Denis M. Kishenko Enterprise Solutions Software Division - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Mikhail Fursov
Re: [classlib] Build failure messages
I also worried about a lot of warnings and I think warnings like these can be fixed last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java [javac] (at line 26) [javac] import java.io.ObjectInputStream; [javac] ^ [javac] The import java.io.ObjectInputStream is never used [javac] 34. WARNING in C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java [javac] (at line 130) [javac] private static final class Sync extends AbstractQueuedSynchronizer { [javac] [javac] The serializable class Sync does not declare a static final serialVersionUID field of type long 2006/10/19, Nathan Beyer <[EMAIL PROTECTED]>: I added a compiler arg to the main 'build-java.xml' compile to disable all of the warnings. This should take care of the warnings. -Nathan On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: > Can we just shut them off for now? it's a good project for someone to > go and clean those up, but there's no need for all of us to see them. > > geir > > Mark Hindess wrote: > > These messages aren't getting through on the commits list on due to the > > large number of warnings since switching to ecj. I've modified the > > summary process so that it ignores these for the moment. > > > > Hopefully this means that the failure messages that haven't been making > > the commits list will start getting through. > > > > Regards, > > Mark. > > > > > > > > > > - > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Denis M. Kishenko Enterprise Solutions Software Division - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
I added a compiler arg to the main 'build-java.xml' compile to disable all of the warnings. This should take care of the warnings. -Nathan On 10/18/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote: Can we just shut them off for now? it's a good project for someone to go and clean those up, but there's no need for all of us to see them. geir Mark Hindess wrote: > These messages aren't getting through on the commits list on due to the > large number of warnings since switching to ecj. I've modified the > summary process so that it ignores these for the moment. > > Hopefully this means that the failure messages that haven't been making > the commits list will start getting through. > > Regards, > Mark. > > > > > - > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [classlib] Build failure messages
Can we just shut them off for now? it's a good project for someone to go and clean those up, but there's no need for all of us to see them. geir Mark Hindess wrote: These messages aren't getting through on the commits list on due to the large number of warnings since switching to ecj. I've modified the summary process so that it ignores these for the moment. Hopefully this means that the failure messages that haven't been making the commits list will start getting through. Regards, Mark. - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]