hgweb: mq repos should be in non-recursive collections, too

mercurial/util.py

@@ -1902,13 +1902,12 b' def walkrepos(path, followsym=False, see'
     for root, dirs, files in os.walk(path, topdown=True, onerror=errhandler):
         if '.hg' in dirs:
             yield root # found a repository
+            qroot = os.path.join(root, '.hg', 'patches')
+            if os.path.isdir(os.path.join(qroot, '.hg')):
+                yield qroot # we have a patch queue repo here
             if recurse:
                 # avoid recursing inside the .hg directory
-                # the mq repository is added in any case
                 dirs.remove('.hg')
-                qroot = os.path.join(root, '.hg', 'patches')
-                if os.path.isdir(os.path.join(qroot, '.hg')):
-                    yield qroot # we have a patch queue repo here
             else:
                 dirs[:] = [] # don't descend further
         elif followsym:

tests/test-hgwebdir.out

@@ -31,6 +31,7 b' 200 Script output follows'
 
 /b/
 /coll/a/
+/coll/a/.hg/patches/
 /coll/b/
 /coll/c/
 /rcoll/a/
@@ -115,6 +116,7 b' 200 Script output follows'
 
 
 /coll/a/
+/coll/a/.hg/patches/
 /coll/b/
 /coll/c/
 
@@ -138,6 +140,7 b' 200 Script output follows'
 
 
 /a/
+/a/.hg/patches/
 /b/
 /c/
 

tests/test-mq-qclone-http

@@ -22,6 +22,8 b' root=`pwd`'
 
 cd ..
 
+echo % test with recursive collection
+
 cat > collections.conf <<EOF
 [paths]
 /=$root/**
@@ -38,6 +40,8 b' hg --cwd b log --template "{desc}\\n"'
 hg --cwd b qpush -a
 hg --cwd b log --template "{desc}\n"
 
+echo % test with normal collection
+
 cat > collections1.conf <<EOF
 [paths]
 /=$root/*
@@ -53,3 +57,21 b' hg qclone http://localhost:$HGPORT1/a c'
 hg --cwd c log --template "{desc}\n"
 hg --cwd c qpush -a
 hg --cwd c log --template "{desc}\n"
+
+echo % test with old-style collection
+
+cat > collections2.conf <<EOF
+[collections]
+$root=$root
+EOF
+
+hg serve -p $HGPORT2 -d --pid-file=hg.pid --webdir-conf collections2.conf \
+    -A access-paths.log -E error-paths-1.log
+cat hg.pid >> $DAEMON_PIDS
+
+"$TESTDIR/get-with-headers.py" localhost:$HGPORT2 '/?style=raw'
+
+hg qclone http://localhost:$HGPORT2/a d
+hg --cwd d log --template "{desc}\n"
+hg --cwd d qpush -a
+hg --cwd d log --template "{desc}\n"

tests/test-mq-qclone-http.out

@@ -3,6 +3,7 b' adding b'
 [mq]: b.patch
 a
 b.patch
+% test with recursive collection
 200 Script output follows
 
 
@@ -27,10 +28,12 b' applying b.patch'
 Now at: b.patch
 imported patch b.patch
 a
+% test with normal collection
 200 Script output follows
 
 
 /a/
+/a/.hg/patches/
 
 requesting all changes
 adding changesets
@@ -41,10 +44,37 b' requesting all changes'
 adding changesets
 adding manifests
 adding file changes
-added 2 changesets with 2 changes to 2 files
+added 1 changesets with 3 changes to 3 files
 updating working directory
-2 files updated, 0 files merged, 0 files removed, 0 files unresolved
+3 files updated, 0 files merged, 0 files removed, 0 files unresolved
 1 files updated, 0 files merged, 0 files removed, 0 files unresolved
 a
-no patches in series
+applying b.patch
+Now at: b.patch
+imported patch b.patch
 a
+% test with old-style collection
+200 Script output follows
+
+
+/a/
+/a/.hg/patches/
+
+requesting all changes
+adding changesets
+adding manifests
+adding file changes
+added 2 changesets with 2 changes to 2 files
+requesting all changes
+adding changesets
+adding manifests
+adding file changes
+added 1 changesets with 3 changes to 3 files
+updating working directory
+3 files updated, 0 files merged, 0 files removed, 0 files unresolved
+1 files updated, 0 files merged, 0 files removed, 0 files unresolved
+a
+applying b.patch
+Now at: b.patch
+imported patch b.patch
+a