comparison tests/test-subrepo-svn.t @ 21564:2e91d4964ecd stable

subrepo: make "_sanitize()" work

"_sanitize()" was introduced by 224e96078708 on "stable" branch, but
it has done nothing for sanitizing since 224e96078708.

"_sanitize()" assumes "Visitor" design pattern:

    "os.walk()" should invoke specified function ("v" in this case)
    for each directory elements under specified path

but "os.walk()" assumes "Iterator" design pattern:

    callers of it should drive loop to scan each directory elements
    under specified path by themselves with the returned generator
    object

Because of this mismatching, "_sanitize()" just discards the generator
object returned by "os.walk()" and does nothing for sanitizing.

This patch makes "_sanitize()" work.

This patch also changes the format of warning message to show each
unlinked files, for multiple appearances of "potentially hostile
.hg/hgrc".

21556:5e13507a3b4e

  $ hg up
  A    *subdir/a (glob)
  Checked out revision 15.
  2 files updated, 0 files merged, 0 files removed, 0 files unresolved
  $ cd ..

21564:2e91d4964ecd

  $ hg up
  A    *subdir/a (glob)
  Checked out revision 15.
  2 files updated, 0 files merged, 0 files removed, 0 files unresolved
  $ cd ..

Test sanitizing ".hg/hgrc" in subrepo

  $ cd sub/t
  $ hg update -q -C tip
  $ cd s
  $ mkdir .hg
  $ echo '.hg/hgrc in svn repo' > .hg/hgrc
  $ mkdir -p sub/.hg
  $ echo 'sub/.hg/hgrc in svn repo' > sub/.hg/hgrc
  $ svn add .hg sub
  A         .hg
  A         .hg/hgrc (glob)
  A         sub
  A         sub/.hg (glob)
  A         sub/.hg/hgrc (glob)
  $ svn ci -m 'add .hg/hgrc to be sanitized at hg update'
  Adding         .hg
  Adding         .hg/hgrc (glob)
  Adding         sub
  Adding         sub/.hg (glob)
  Adding         sub/.hg/hgrc (glob)
  Transmitting file data ..
  Committed revision 16.
  $ svn up -q
  $ cd ..
  $ hg commit -S -m 'commit with svn revision including .hg/hgrc'
  $ grep ' s$' .hgsubstate
  16 s
  $ cd ..

  $ cd tc
  $ hg pull -u -q 2>&1 | sort
  warning: removing potentially hostile 'hgrc' in 's/.hg' (glob)
  warning: removing potentially hostile 'hgrc' in 's/sub/.hg' (glob)
  $ grep ' s$' .hgsubstate
  16 s
  $ cat s/.hg/hgrc
  cat: s/.hg/hgrc: No such file or directory
  [1]
  $ cat s/sub/.hg/hgrc
  cat: s/sub/.hg/hgrc: No such file or directory
  [1]

  $ cd ../..