[darcs-devel] [darcs #223] pull --dry-run -a takes forever (approximately)

zooko at zooko.com via RT bugs at darcs.net
Mon Feb 21 11:28:52 PST 2005


Mon Feb 21 14:28:49 2005: New ticket: 223.
Transaction: Ticket created by zooko at zooko.com
       Queue: Darcs
     Subject: pull --dry-run -a takes forever (approximately)
       Owner: Nobody
  Requestors: zooko at zooko.com
      Status: new
 Ticket <URL: http://bugs.darcs.net/.//Ticket/Display.html?id=223 >

Note that this delay is *before* it attempts to merge, so this isn't 
the problem with O(2^N) merges, I guess.  That is, if I do pull -v 
without the "-a" or the "--dry-run", then it takes approximately 
forever before it ever prints anything to stdout about what patches 
were found.  The printout that it *does* printout, included in this bug 
report, appears only after you give up and hit C-c.

This is a critical bug for me in the sense that it prevents me from 
pulling a needed patch from another branch.  I guess I'm going to 
work-around it by running "darcs diff" on that branch and applying the 
resulting diff with "patch" on my branch.

As usual, access to the repository in question is restricted to people 
who have offered to preserve our confidentiality on the enclosed source 
code, which so far is David Roundy and Mark Stosberg.

http://yumyum.zooko.com:19144/priv/$PRIVATEURL/

Platform is Ubuntu Linux 64-bit on Athlon64 @ 2 GHz, 1 GB RAM.

Regards,

Zooko

------- begin included transcript
time darcs pull -v -v -v --dry-run -a ../t/unstable &> 
pull-dryrun-log.txt

real    12m32.949s
user    12m2.543s
sys     0m0.272s
HACL yumyum:~/playground/zfone-repos/unstable$ cd ..
HACL yumyum:~/playground/zfone-repos$ time tar cf 
indefinite-pull-dry.tar t unstable
time
real    0m1.571s
user    0m0.035s
sys     0m0.352s
HACL yumyum:~/playground/zfone-repos$ time bzip2 -9 
indefinite-pull-dry.tar

real    0m32.718s
user    0m32.235s
sys     0m0.214s
HACL yumyum:~/playground/zfone-repos$ sudo mv 
indefinite-pull-dry.tar.bz2 /var/www/priv/$PRIVATEURL/
HACL yumyum:~/playground/zfone-repos$
------- end included transcript

------- begin included exact-version
darcs compiled on Feb 18 2005, at 12:48:28
# configured Fri Feb 18 12:30:54 AST 2005
sh ./configure --prefix=/usr/local/stow/darcs

Context:

[merge_three_patches-test-use-DARCS-not-darcs-for-get-tempOld-tempB
petersen at haskell.org**20050217052705]

[fix bug in what_sl test.
David Roundy <droundy at abridgegame.org>**20050217133957]

[fix non-matching regexp in unit test
Will <will at glozer.net>**20050217074209]

[add self-test routines to cgi
Will <will at glozer.net>**20050217071519]

[Revert to old behaviour for "whatsnew -sl" wrt. showing modified files 
(bug #210 in RT)
peter at zarquon.se**20050216231858]

[#217: darcs test suite ignores ~/.darcs
Yuval Kogman <nothingmuch at woobling.org>**20050216000637
  Every test run has env HOME=`pwd` prepended to it, which causes darcs 
to look
  for .darcs in the src/darcs/test directory, instead of the actual home 
of the
  user running the test suite.
]

[quote filename passed to $EDITOR
Benedikt Schmidt <beschmi at cloaked.de>**20050210193135
  fix bug when working directory contains spaces reported by
  Steven E. Harris.
]

[RT#201 - add test for takeLock failure on pull
Mark Stosberg <mark at summersault.com>**20050206040412]

[Build-depend on libcurl2 or libcurl3
Jan-Benedict Glaw <jbglaw at lug-owl.de>**20050131133404

  Darcs builds perfectly okay with libcurl3, so there's no need to force
  using an older version of this lib.

]

[Show patch number while (verbosely) downloading patches
Jan-Benedict Glaw <jbglaw at lug-owl.de>**20050131121046]

[Close both ends of pipes.
Jan-Benedict Glaw <jbglaw at lug-owl.de>**20050130201542]

[try to avoid email loops.
David Roundy <droundy at abridgegame.org>**20050129150828]

[Fix off-by-one error in WIN32 version of mkstemp()
Jan-Benedict Glaw <jbglaw at lug-owl.de>**20050127234048

  This patch solves different issues with the mkstemp() funktion provided
  for WIN32 versions of darcs:

  - THIS PATCH IS NOT TESTED: I'VE GOT NO WINDOWS AROUND!
  - Move file-local variable into the mkstemp() since it's only used
    by mkstemp().
  - Remove unneeded variable p2.
  - Remove unneeded variable fd by directly returning the result of
    open().
  - Correctly return EINVAL if the supplied template isn't at least
    6 bytes long and doesn't end in 6 consecutive 'XXXXXX'.
  - Write 6 bytes of hex garbage instead of 5 bytes of decimal garbage.
    This was caused because the final \0 wasn't put into account for
    the snprintf() call.
]

[automatically add parent directories for darcs add
Benedikt Schmidt <beschmi at cloaked.de>**20050205014603

  Instead of complaining that the parent directory of the file isn't
  in the repo, add parent directories recursively. (See Bug #20)
]

[fix makefile problem with darcs-createrepo.
David Roundy <droundy at abridgegame.org>**20050201123415]

[fix makefile bug where darcs got relinked.
David Roundy <droundy at abridgegame.org>**20050201115547
  The trouble was that it's hard to tell make that compiling a haskell 
file
  produces both a .o file and a .hi file, so there was a dummy rule for
  creating a .hi from a .o which was causing relinking.
]

[Compare the right files when relinking.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050201021020
  When deciding whether to relink, we were comparing the wrong files.
  This could cause us to relink files that are not identical, which
  could cause repository corruption.
]

[give better error message when pushing over unsupported protocols.
David Roundy <droundy at abridgegame.org>**20050130215919]

[Fix confusion about the pristine tree.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050130211338]

[Complete relinking documentation.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050130205404]

[Fix TeX typo.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050130205256]

[re-write DARCS_WGET docs.
Mark Stosberg <mark at summersault.com>**20050129143939

  Thanks to David Roundy for clarifying these and Dave Roberts for 
noticing
  that they needed to be improved.

  I checked the source code to confirm how it works, but a fact check
  of this update is certainly welcome.

      Mark
]

[remove rss news from web page.
David Roundy <droundy at abridgegame.org>**20050129125119
  The javascript link thing seems to be broken.
]

[TAG 1.0.2rc3
David Roundy <droundy at abridgegame.org>**20050129122928]

[bump to version 1.0.2rc3.
David Roundy <droundy at abridgegame.org>**20050129122912]

[fix markup in best_practices
Tommy Pettersson <ptp at lysator.liu.se>**20050128235516]

[add explanation to section Basics in best_practices
Tommy Pettersson <ptp at lysator.liu.se>**20050128235324]

[change from `current' to `pristine' in best_practices
Tommy Pettersson <ptp at lysator.liu.se>**20050128235128]

[Document --relink and --no-pristine-tree.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050128204513]

[Remove obsolete code (commented out).
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050125071412]

[fix bug in (unused function) mapPS.
David Roundy <droundy at abridgegame.org>**20050128123024]

[attributions conflict resolution.
David Roundy <droundy at abridgegame.org>**20050128113918]

[sprinkle some attributes for this release into the ChangeLog
Mark Stosberg <mark at summersault.com>**20050128011125

  I added a few attributions, but I don't have a complete list. Feel 
free to
  update others if you care (and especially if I got any wrong!).
]

[Slurp again in Get, but only if slurping is cheap.
Juliusz Chroboczek <jch at pps.jussieu.fr>*-20050124005142]

[resolve conflict with my stupid self.
David Roundy <droundy at abridgegame.org>**20050128110739
  I tagged 1.0.2 twice, having unpulled once, but now both versions got 
sent
  to the central repository--one to darcs-unstable and one to darcs.  :(
  Foolish David.  I had thought that I had intercepted them before they
  arrived and passed tests.  So now I'm picking up the pieces.
]

[TAG 1.0.2rc2
David Roundy <droundy at abridgegame.org>**20050127114756]

[bump version (so soon!) to 1.0.2rc2
David Roundy <droundy at abridgegame.org>**20050127114738]

[TAG 1.0.2rc2
David Roundy <droundy at abridgegame.org>**20050127113814]

[bump version (so soon!) to 1.0.2rc2.
David Roundy <droundy at abridgegame.org>**20050127113757]

[quote filenames in win32 exec
Will <will at glozer.net>**20050127001406]

[really fix repair this time.
David Roundy <droundy at abridgegame.org>**20050129213231
  Foolish David ought to test his patches before sending them off to the
  central repository...
]

[don't be so paranoid in buggy pending error message.
David Roundy <droundy at abridgegame.org>**20050129211235]

[fix recently-introduced bug in darcs repair.
David Roundy <droundy at abridgegame.org>**20050129201344]

[make optimize --no-compress behave as documented.
David Roundy <droundy at abridgegame.org>**20050129201302]

[repos-[new|old] => [new|old]-repos
thies at thieso.net**20050128202058

  see: 
http://article.gmane.org/gmane.comp.version-control.darcs.user/5329

  PS: i don't know if this is the RightWay(tm) to send stuff like this, 
just trying;-) re, tc
]

[don't create a filename too long to fit in buffer.
David Roundy <droundy at abridgegame.org>**20050129120603
  The "\n" in the filename was making it one character too long... 
besides
  which, it's usually kinder to never create filenames with trailing
  newlines.  :)
]

[Fix memory and file descriptor leaks at error handling time
Jan-Benedict Glaw <jbglaw at lug-owl.de>**20050128204255

  This patch fixes the most "obvious" memory and file descriptor leaks
  in allthe error handling paths. I'm not yet convinced that I got 'em
  all, but I guess this is at least better than nothing...

]

[Tweaks to maybe_relink.
Juliusz Chroboczek <jch at pps.jussieu.fr>**20050126181118
  Thanks to Jan-Benedict Glaw
]

[TAG 1.0.2rc1
David Roundy <droundy at abridgegame.org>**20050126140813]

------- end included exact-version






More information about the darcs-devel mailing list