Many diff / status / update commands miss files with the deleted flag (patch check)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Many diff / status / update commands miss files with the deleted flag (patch check)

Andrew Williams-5
I am sending this to the list before I try to commit (not certain I  
have privs tbh...) to make sure that folk are in agreement with this  
change:

Basically many commands (svn diff, status and up for a start) do not  
report files that have been deleted (reported so using the deleted  
flag) as the file no longer exists.
Surely fixing this has to be the right thing to do, as I want to know  
when my files have been deleted ;)

The only thing in this patch that I am not 100% sure of is in maven-
scm-providers/maven-scm-provider-bazaar/src/main/java/org/apache/maven/
scm/provider/bazaar/command/diff/BazaarDiffConsumer.java
it will allow a file that is marked deleted to appear in the log even  
though (looking at the code) it might have a preceding '/' - does the  
Bazaar parser really not know whether or not to expect a preceding  
slash?!?

if I don't hear any grumbles I will try to commit this - it is a major  
annoyance for me...

Andy




maven-scm-show-deleted.patch (3K) Download Attachment