Merge lp:~parthm/bzr/564605-ignore-java-classfile into lp:bzr

[Parth Malwankar]

=== Fixed Bug #564605 ===
This is a trivial patch that adds *.class to the default ignore list (i.e. bzrlib.ignores.USER_DEFAULTS)

[Vincent Ladeuil (vila)]

We have been pretty conservative about what we put in the default ignore list for performance reasons (anybody as an update on that ?).

If we start adding more regexps there it may be worth getting feedback from the mailing list first.

I'll vote 'Needs Information' and will put the mp status wip until we get it.

review: Needs Information

[Parth Malwankar (parthm)]

> We have been pretty conservative about what we put in the default ignore list
> for performance reasons (anybody as an update on that ?).
>

That sounds reasonable. I will reject this patch and mark the bug as invalid.

Unmerged revisions

5170. By Parth Malwankar on 2010-04-20

updated NEWS

5169. By Parth Malwankar on 2010-04-20

added *.class to default ignore list.

=== modified file 'NEWS'
--- NEWS	2010-04-20 10:30:30 +0000
+++ NEWS	2010-04-20 12:52:21 +0000
@@ -13,6 +13,9 @@
 Compatibility Breaks
 ********************
 
+* Java class files (\*.class) are now ignored by default.
+  (Parth Malwankar, #564605)
+
 New Features
 ************
 

=== modified file 'bzrlib/ignores.py'
--- bzrlib/ignores.py	2010-03-29 00:54:27 +0000
+++ bzrlib/ignores.py	2010-04-20 12:52:21 +0000
@@ -92,6 +92,7 @@
 # please keep these sorted (in C locale order) to aid merging
 USER_DEFAULTS = [
     '*.a',
+    '*.class',
     '*.o',
     '*.py[co]',
     '*.so',