Ticket #1241: exclude-from-tilde-expansion.darcs.patch

1 patch for repository davidsarah@dev.allmydata.org:/home/darcs/tahoe/trunk:

Sun Oct 31 03:37:54 GMT Standard Time 2010  david-sarah@jacaranda.org
  * tahoe backup: perform tilde expansion in exclude-from filename. fixes #1241

New patches:

[tahoe backup: perform tilde expansion in exclude-from filename. fixes #1241
david-sarah@jacaranda.org**20101031033754
 Ignore-this: dd5d3e37413944a95dcaab49b953c9cc
] {
hunk ./src/allmydata/scripts/cli.py 4
 import os.path, re, sys, fnmatch
 from twisted.python import usage
 from allmydata.scripts.common import BaseOptions, get_aliases, get_default_nodedir, DEFAULT_ALIAS
-from allmydata.util.encodingutil import argv_to_unicode, argv_to_abspath, quote_output
+from allmydata.util.encodingutil import argv_to_unicode, argv_to_abspath, \
+    quote_output, abspath_expanduser_unicode
 
 NODEURL_RE=re.compile("http(s?)://([^:]*)(:([1-9][0-9]*))?")
 
hunk ./src/allmydata/scripts/cli.py 312
         """Ignore file matching glob patterns listed in file, one per
         line. The file is assumed to be in the argv encoding."""
         try:
-            exclude_file = file(filepath)
+            exclude_file = file(abspath_expanduser_unicode(argv_to_unicode(filepath)))
         except:
             raise BackupConfigurationError('Error opening exclude file %r.' % filepath)
         try:
hunk ./src/allmydata/test/test_cli.py 8
 import urllib, re
 import simplejson
 
+from mock import patch
+
 from allmydata.util import fileutil, hashutil, base32
 from allmydata import uri
 from allmydata.immutable import upload
hunk ./src/allmydata/test/test_cli.py 1872
         self._check_filtering(filtered, root_listdir, (u'lib.a', u'_darcs', u'subdir'),
                               (nice_doc,))
 
+    @patch('__builtin__.file')
+    def test_exclude_from_tilde_expansion(self, mock):
+        basedir = "cli/Backup/exclude_from_tilde_expansion"
+        fileutil.make_dirs(basedir)
+        nodeurl_path = os.path.join(basedir, 'node.url')
+        fileutil.write(nodeurl_path, 'http://example.net:2357/')
+
+        # ensure that tilde expansion is performed on exclude-from argument
+        exclude_file = u'~/.tahoe/excludes.dummy'
+        backup_options = cli.BackupOptions()
+
+        mock.return_value = StringIO()
+        backup_options.parseOptions(['--exclude-from', unicode_to_argv(exclude_file),
+                                     '--node-directory', basedir, 'from', 'to'])
+        self.failUnlessIn(((abspath_expanduser_unicode(exclude_file),), {}), mock.call_args_list)
+
     def test_ignore_symlinks(self):
         if not hasattr(os, 'symlink'):
             raise unittest.SkipTest("Symlinks are not supported by Python on this platform.")
}

Context:

[NEWS: add news entry for #1223
Francois Deppierraz <francois@ctrlaltdel.ch>**20101030111130
 Ignore-this: 6b6afd4b0f0527a3c9784c1db95d083
] 
[NEWS: add a NEWS entry about bug #1045
Francois Deppierraz <francois@ctrlaltdel.ch>**20101030101351
 Ignore-this: 7e758afbbd0f1d22a5d0b4fc38661c1d
] 
[setup: run require_auto_deps() before attempting to import any deps in __init__.py
zooko@zooko.com**20101030081035
 Ignore-this: ffcaf2450628543e020e9919e455f691
 For one thing, this makes missing-dependency failures into DistributionNotFound errors instead of ImportErrors, which might be more useful to the user. For another thing, if someone is using distributions that were installed with --multi-version, then they might be not importable until after require_auto_deps() has been run. (The docs claim that this would be the case, but we don't have an example of this happening at this time.)
] 
[setup: show-tool-versions: emit module and __version__ information even when module name != distribution (package) name, and add TwistedCore, TwistedWeb, and TwistedConch
zooko@zooko.com**20101030070233
 Ignore-this: 3df19910090d44502ddeeef5d9c29a7
] 
[misc/build_helpers/test-with-fake-pkg.py: look for eggs in the parent of the src directory. refs #1190
david-sarah@jacaranda.org**20101030034303
 Ignore-this: 4a3cf286272cdb5d06aac15fb5998b33
] 
[scripts/runner.py: fix unused import of allmydata. refs #1190
david-sarah@jacaranda.org**20101030003149
 Ignore-this: b2fc67f6192ea7ccf8a5ad010ce74a64
] 
[scripts/runner.py: remove pkg_resources.require() calls. These are at best redundant because we have already called _auto_deps.require_auto_deps() (from allmydata.__init__) at that point, and they are causing failure of the test-from-prefixdir step on some buildslaves. refs #1190
david-sarah@jacaranda.org**20101029235328
 Ignore-this: e00dee63acc7b76a5755025d75abf524
] 
[misc/build_helpers/run_trial.py: look for zetuptoolz egg in the parent directory, not the cwd of run_trial. refs #1190
david-sarah@jacaranda.org**20101029230329
 Ignore-this: 1596fb8c290d1c706f079701b1857db8
] 
[scripts/tahoe_backup.py: oops, fix missing import, thanks pyflakes
Brian Warner <warner@lothar.com>**20101029094223
 Ignore-this: 285c35af824935641a5be35c008b080c
 
 test_cli.py: hush minor pyflakes complaint
] 
[mutable/servermap.py: update comment. Closes #1231.
Brian Warner <warner@lothar.com>**20101029091424
 Ignore-this: 80bf854123fc254e097a81b82bdf4990
] 
[tahoe_cp.py: Don't call urllib.quote with an Unicode argument, fix #1224
Brian Warner <warner@lothar.com>**20101029084520
 Ignore-this: 5524722d5e5babbb73ca0969d54967f6
 tahoe_backup.py: Fix another (potential) occurrence of calling urllib.quote()
 with an Unicode parameter
] 
[fix #1223, crash+inefficiency during repair due to read overrun
Brian Warner <warner@lothar.com>**20101029082036
 Ignore-this: e6aa0295ad254544da3b5cc41b33d862
 
 * repairer (really the uploader) reads beyond end of input file (Uploadable)
 * new-downloader does not tolerate overreads
 * uploader does lots of tiny reads (inefficient)
 
 This fixes the last two. The uploader still does a single overread at the end
 of the input file, but now that's ok so we can leave it in place. The
 uploader now expects the Uploadable to behave like a normal disk
 file (reading beyond EOF will return less data than was asked for), and now
 the new-downloadable behaves that way.
] 
[add misc/build_helpers/test-with-fake-pkg.py. refs #1190
david-sarah@jacaranda.org**20101029025150
 Ignore-this: 995f220962708f1bad83092161130f67
] 
[bundled zetuptoolz: if __main__.__requires__ exists then do not add packages to the working set if they provide an incompatible version of a package. Also put a complete __requires__ listing the transitive closure of dependencies at the beginning of generated scripts, rather than a shallow __requires__ specifying only the application version. refs #1190
david-sarah@jacaranda.org**20101029223111
 Ignore-this: a95f1967884340e53bf3adf90db40cfc
] 
[setup.py, misc/build_helpers/run_trial.py: use undocumented __requires__ variable to cause setuptools/zetuptoolz to put the correct versions of dependencies on sys.path. Also ensure that run_trial adds the bundled zetuptoolz egg at the start of sys.path if present. Make the source directory comparison work correctly for the test-with-fake-pkg build step. refs #1190
david-sarah@jacaranda.org**20101029222825
 Ignore-this: 8b09366eb6ce3d55c7db5239077a0fac
] 
[test_runner.py: fix error in BinTahoe.test_version_no_noise introduced by last patch. refs #1235
david-sarah@jacaranda.org**20101029221123
 Ignore-this: 4bf21ea34768e8e6adf104e56f939fd0
] 
[test_runner.py: also allow 'from pkg_resources import load_entry_point' as noise. refs #1235.
david-sarah@jacaranda.org**20101029204246
 Ignore-this: a47440aa2cdd29ce55ac7c6c7f4bcaf2
] 
[test_runner.py: if the only noise is 'UserWarning: Unbuilt egg for setuptools', skip instead of failing the no_noise tests. This version avoids 'any' to be compatible with Python < 2.5. refs #1235.
david-sarah@jacaranda.org**20101029191804
 Ignore-this: 83ca1543fc9673e664a8eeefe1eba429
] 
[NEWS: clarify (strengthen) description of what backdoors.rst declares, and add bugfix entries for 'tahoe cp' and Windows console bugs. refs #1216, #1224, #1232
david-sarah@jacaranda.org**20101028180046
 Ignore-this: 1c3eef3cd353b06b6ee00ce87c5ef59a
] 
[startstop_node.py: pyflakes import fix. refs #1190
david-sarah@jacaranda.org**20101028014805
 Ignore-this: 369ef5022c8ee5a0d8341af01553bcef
] 
['tahoe start': use import+call rather than find+spawn
"Brian Warner <warner@lothar.com>"**20101027061342
 
 This removes the need to use a locally-built (dependency) bin/twistd, and
 removes a big chunk of behavior differences between unix and windows. It
 also happens to resolve the "client node probably started" uncertainty.
 Might help with #1190, #602, and #71.
] 
[make ResponseCache smarter to avoid memory leaks: don't record timestamps, use DataSpans to merge entries, and clear the cache when we see a new seqnum. refs #1045, #1229
david-sarah@jacaranda.org**20101027043302
 Ignore-this: 88fd6fba7f35a2f8af1693b92718f5f3
] 
[windows/fixups.py: limit length of string passed in a single call to WriteConsoleW. fixes #1232.
david-sarah@jacaranda.org**20101027021636
 Ignore-this: fbd99e0d22493974696d37925d97c7d6
] 
[docs/known_issues.rst: Add section on traffic analysis. Fix URL for current version of file.
david-sarah@jacaranda.org**20101024234259
 Ignore-this: f3416e79d3bb833f5118da23e85723ad
] 
[test_mutable.py: add test for ResponseCache memory leak. refs #1045, #1129
david-sarah@jacaranda.org**20101024193409
 Ignore-this: 3aee7f0677956cc6deaccb4d5b8e415f
] 
[docs/running.html: fix missing end-quote, and change frontends/ doc references to .rst.
david-sarah@jacaranda.org**20101024171500
 Ignore-this: 47c645a6595e1790b1d1adfa71af0e1d
] 
[docs/running.html: 'tahoe create-client' now creates a node with storage disabled. Also change configuration.txt references to configuration.rst.
david-sarah@jacaranda.org**20101024170431
 Ignore-this: e5b048055494ba3505bb8a506610681c
] 
[test_encodingutil.py: test_argv_to_unicode modified the wrong encoding variable. fixes #1214
david-sarah@jacaranda.org**20101023035810
 Ignore-this: e5f1f849931b96939facc53d93ff61c5
] 
[doc: add explanation of the motivation for the surprising and awkward API to erasure coding
zooko@zooko.com**20101015060202
 Ignore-this: 428913ff6e1bf5b393deffb1f20b949b
] 
[setup: catch and log ValueError from locale.getdefaultlocale() in show-tool-versions.py
zooko@zooko.com**20101015054440
 Ignore-this: 827d91490562c32ed7cf6526dfded773
 I got a bug report from Mathias Baert showing that locale.getdefaultlocale() raises an exception on his Mac OS X system. Heh.
] 
[docs: update how-to-make-a-release doc with a few tweaks from the 1.8.0 process
zooko@zooko.com**20101015054413
 Ignore-this: ca5e9478531a3393792ae283239549dd
] 
[docs: update NEWS ref: #1216
zooko@zooko.com**20101015053719
 Ignore-this: 2e0b92e4145d667cdf075e64b7965530
] 
[docs: fix tab-vs-spaces, make some CLI examples <tt>/"literal", wrap some to
Brian Warner <warner@lothar.com>**20101015060606
 Ignore-this: eae08bdf0afb19a2fbf41c31e70a8122
 80-cols, remove spurious whitespace. Add rst2html.py rule to Makefile.
] 
[docs: add Peter Secor, Shawn Willden, and Terrell Russell as signatories to docs/backdoors.rst
zooko@zooko.com**20101015053242
 Ignore-this: c77adf819d664f673e17c4aaeb353f33
] 
[docs: convert all .txt docs to .rst thanks to Ravi Pinjala
zooko@zooko.com**20101015052913
 Ignore-this: 178a5122423189ecfc45b142314a78ec
 fixes #1225
] 
[docs: add statement on our refusal to insert backdoors
zooko@zooko.com**20101006051147
 Ignore-this: 644d308319a7b80c4434bdff9760404a
] 
[setup: add --multi-version to the "setup.py develop" command-line
zooko@zooko.com**20101005182350
 Ignore-this: 709155cc21caff29826b8d41a8c8d63d
 fixes #530. I earlier tried this twice (see #530 for history) and then twice rolled it back due to some problems that arose. However, I didn't write down what the problems were in enough detail on the ticket that I can tell today whether those problems are still issues, so here goes the third attempt. (I did write down on the ticket that it would not create site.py or .pth files in the target directory with --multi-version mode, but I didn't explain why *that* was a problem.)
] 
[setup: use execfile to access _auto_deps.py in its proper location of src/allmydata/ instead of copying it into place when setup.py is executed
zooko@zooko.com**20100906055714
 Ignore-this: c179b42672d775580afad40121f86812
] 
[trivial: M-x whitespace-cleanup
zooko@zooko.com**20100903144712
 Ignore-this: 1bb764d11ac69b4a35ea091cfb13158a
] 
[minor: remove unused interface declaration, change allmydata.org to tahoe-lafs.org in email address, fix wording in relnotes.txt
zooko@zooko.com**20100930153708
 Ignore-this: a452969228afed2774de375e29fa3048
] 
[immutable/repairer.py: don't use the default happiness setting when repairing
Kevan Carstensen <kevan@isnotajoke.com>**20100927200102
 Ignore-this: bd704d9744b970849da8d46a16b8089a
] 
[NEWS: note dependency updates to pycryptopp and pycrypto.
david-sarah@jacaranda.org**20100924191207
 Ignore-this: eeaf5c9c9104f24c450c2ec4482ac1ee
] 
[TAG allmydata-tahoe-1.8.0
zooko@zooko.com**20100924021631
 Ignore-this: 494ca0a885c5e20c883845fc53e7ab5d
] 
Patch bundle hash:
f764f7dbc260629687de69d83e30387cd4911eea