/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzr-upload-pack

  • Committer: John Arbash Meinel
  • Date: 2009-10-14 15:53:34 UTC
  • mto: This revision was merged to the branch mainline in revision 4746.
  • Revision ID: john@arbash-meinel.com-20091014155334-dxdm0ns610scz5c9
Add a test case for the bug w/ NotImplemented.

Basically, bool(NotImplemented) == True, which means that you have to
check for 'result == Py_NotImplemented' before you check PyObject_IsTrue().
Going further, though, I simplified the _is_equal a lot by switching to
using object definitions (rather than PyObject *).
This now handles exceptions and refcounts properly, and the code is cleaner
to boot. (Exceptions will now include these lines in the traceback, etc.)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzr-upload-pack
1
 
#!/usr/bin/env python
2
 
import bzrlib
3
 
from bzrlib.plugins.git.server import BzrBackend
4
 
from dulwich.server import UploadPackHandler
5
 
import sys, os, optparse
6
 
 
7
 
parser = optparse.OptionParser(usage="usage: git-upload-pack [--strict] [--timeout=nn] <dir>")
8
 
parser.add_option("--strict", action="store_true", dest="strict", default=False)
9
 
parser.add_option("--timeout", type="int", dest="timeout", default=-1)
10
 
options, args = parser.parse_args()
11
 
 
12
 
if len(args) != 1 or not os.path.isdir(args[0]):
13
 
    print "usage: " + parser.usage
14
 
    sys.exit(1)
15
 
 
16
 
backend = BzrBackend(sys.argv[1])
17
 
 
18
 
def write_fn(data):
19
 
    sys.stdout.write(data)
20
 
    sys.stdout.flush()
21
 
 
22
 
server = UploadPackHandler(backend, sys.stdin.read, write_fn)
23
 
server.handle()