[fix] You'll need to also remove this line: from bzrlib.export.tar_exporter import export_tarball
from history.py since it's not used and doesn't exist. I fixed the unbound local warning here: https://code.launchpad.net/~mbp/bzr/trivial/+merge/67474 When I run the tests for this (with those two fixes) I get a failure in your new test_download_tarball, because the response is 303 See Other The resource has been moved to /; you should be redirected automatically. which makes it look like it's not actually matching the URL. Do you see this too? -- https://code.launchpad.net/~xaav/loggerhead/export-tarball/+merge/66408 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~xaav/loggerhead/export-tarball into lp:loggerhead. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
