Merge lp:~camptocamp/ocb-server/6.1-leaner-bzrignore-lep into lp:ocb-server/6.1

[Leonardo Pistone]

[Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter)]

I disagree with taking out .swp files.
Having .swp in the ignore file means that we would never want that kind of file tracked.
I suggest using a different method to track swp files such as:
bzr ignored | grep .swp

review: Disapprove

[Leonardo Pistone (lepistone)]

Sandy,

1. Do you think there is a use case where you want to have .swp files around but you want "bzr status" to tell you all is good and clean?

2. What do you think about the other two removals? I'm happy to remove only what we find an agreement on.

Thanks!

[Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter)]

1. Sorry to put it bluntly, but if someone loses track of her .swp file, it's her own problem
If someone accidentally commits a .swp file he had lying around, it becomes everyone's problem.
Look at any project, .swp is standard for .*ignore files, as are *~.
Having editor specific files pollute our bzr statuses, in my opinion affects more people than those who find it useful to see it.

2. I am ok with taking out .bzrignore, but that's it. I can see the point (though I don't see it particularity well thought out) in ignoring new files from openerp/addons/ as there shouldn't be any new ones and changes to files in there will still be tracked.

This looks like a pointless divergence from mainline OE, hence the disapproval.

[Holger Brunn (Therp) (hbrunn)]

agreed with Sandy's points

review: Disapprove

Unmerged revisions

4328. By Leonardo Pistone on 2014-03-17

[fix] do not ignore:

- openerp/addons/* (lots of real code there)
- .bzrignore (users should use local ignore, not silently add here),
- *.swp (if an editor crashed, or I forgot to save, I expect bzr to tell me that the tree is not clean).

=== modified file '.bzrignore'
--- .bzrignore	2012-02-01 13:09:57 +0000
+++ .bzrignore	2014-03-17 11:33:19 +0000
@@ -1,7 +1,4 @@
 *.pyc
-.*.swp
-.bzrignore
-openerp/addons/*
 openerp/filestore*
 .Python
 include