Re: [PATCH 1/2] --cvs-exclude for mkfs.jffs2

[Michael Richardson <mcr@sandelman.ottawa.on.ca>]

[-- Attachment #1: Type: text/plain, Size: 1399 bytes --]


>>>>> "Wolfgang" == Wolfgang Denk <wd@denx.de> writes:
    Wolfgang> In message <v0mzjofaup.fsf@marajade.sandelman.ca> you wrote:
    >> This adds an option to ignore CVS directories.

    Wolfgang> This doesn't make sense to me. You can and should use "cvs
    Wolfgang> export" to create the directory trees you want to package.

  yes, one could do that. I chose not to.
  It isn't like I forced this option to be on.

  cvs export requires that I actually check my changes in before I can test
them, which causes a lot of needless churn.  
  It may also mean that developers in my team will check in debug statements
and forget to remove them.  
  
    Wolfgang> You use CVS, others use GIT, or Subversion, or ClearCase, or
    Wolfgang> Monotone, or darcs, or ... - it would not be really practical
    Wolfgang> to add special rules for everybody's favorite tool.

  It would very much be practical to do this.
  I'd rather have a more general --exclude.

  Of course, you argue for removing equivalent options from rsync and other
things.  Accept the patch or not. 

-- 
]       ON HUMILITY: to err is human. To moo, bovine.           |  firewalls  [
]   Michael Richardson,    Xelerance Corporation, Ottawa, ON    |net architect[
] mcr@xelerance.com      http://www.sandelman.ottawa.on.ca/mcr/ |device driver[
] panic("Just another Debian GNU/Linux using, kernel hacking, security guy"); [

[-- Attachment #2: Type: application/pgp-signature, Size: 480 bytes --]