<p dir="ltr"><br>
Op 28-jul.-2013 15:15 schreef "Thomas De Schampheleire" <<a href="mailto:patrickdepinguin%2Bbuildroot@gmail.com">patrickdepinguin+buildroot@gmail.com</a>> het volgende:<br>
><br>
> On Sun, Jul 28, 2013 at 3:03 PM, Thomas Petazzoni<br>
> <<a href="mailto:thomas.petazzoni@free-electrons.com">thomas.petazzoni@free-electrons.com</a>> wrote:<br>
> > Dear Tzu-Jung Lee,<br>
> ><br>
> > On Sun, 28 Jul 2013 18:39:23 +0800, Tzu-Jung Lee wrote:<br>
> >> This patch allows users to override the options or entire command<br>
> >> line of rsync.  The default options filters out object or libraries<br>
> >> during rsync.  For local packages which come with pre-built binaries,<br>
> >> the copy in build/ directory will not have complete files as src/<br>
> ><br>
> > Hum, I am not sure this is the right way to fix this. When it was<br>
> > introduced, I thought the --cvs-exclude option was only ignoring things<br>
> > like CVS/ directories, .svn/ directories or .git/ directories. But now<br>
> > that I read the rsync man page, I see that it excludes a huge number of<br>
> > other file name patterns, which I think isn't desirable.<br>
> ><br>
> > So rather than providing a way for the user to override the rsync<br>
> > command, I'd prefer the rsync command to behave appropriately. I<br>
> > believe it should simply copy /everything/, including version control<br>
> > directories. Yes that's longer, but there's no reasonable generic way<br>
> > to determine which files should be copied and which files should not.<br>
><br>
> In my version of 'man rsync', here is the list of excluded patterns<br>
> when using cvs-exclude:<br>
><br>
> RCS SCCS CVS CVS.adm RCSLOG cvslog.* tags TAGS .make.state<br>
> .nse_depinfo *~ #* .#* ,* _$* *$ *.old *.bak *.BAK *.orig *.rej .del-*<br>
> *.a *.olb *.o *.obj *.so *.exe *.Z *.elc *.ln core .svn/ .git/ .hg/<br>
> .bzr/<br>
><br>
><br>
> I can see how excluding *.a *.o *.obj *.so *.exe can be debatable.<br>
> I would certainly like to keep excluding .svn, .git, .hg, and .bzr.<br>
> These directories can become very big, and in the general use case,<br>
> aren't needed.<br>
> Frankly, I don't care a lot about the other patterns.<br>
><br>
> Here are three suggestions:<br>
> - follow the proposed patch, keeping the default as it is and<br>
> providing a way to overrule it per package.<br>
> - simplifying the default exclusion list to .svn, .git, .hg and .bzr<br>
> (but this wouldn't cover the use case of Tzu-Jung Lee)<br>
> - make the exclusion list a global config option, defaulting to e.g.<br>
> .svn, .git, .hg, .bzr and maybe others, but changeable by the user.</p>
<p dir="ltr">Other suggestion:<br>
- add a config option 'don't exclude version control files when rsyncing' and don't provide further flexibility.</p>