Where is this error?
Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com
On Mon, Oct 5, 2009 at 5:28 PM, smat <g...@cictec.com> wrote:
> Thanks for information about the --full-index option.
> The good news is that the previous error no longer occurs. The bad
> news is that there is now an "int argument required" error in its
> I would use post-review, unfortunately the machine I am working on is
> behind a firewall and only has outside access via a web proxy, and the
> reviews site will not accept "http://github.com/reviewboard/
> reviewboard.git" as the repository URL. Altering the repository URL to
> git://github.com/reviewboard/reviewboard.git" did not work as post-
> review tries to communicate with the repository.
> Setting up post-review on a machine with real internet access would
> work, but surely there's a better way to do this?
> Solomon Matthews
> Systems Engineer
> Australian Centre for Field Robotics
> The University of Sydney
> On Oct 2, 2:47 pm, Christian Hammond <chip...@chipx86.com> wrote:
> > Hi,
> > Looks like we're not importing the "logging" module in scmtools/git.py.
> > fix this and update the site after tonight's midnight (PST) build.
> > The reason you're getting the error, though, is that our git support
> > requires a full SHA1 instead of the default shortened SHA1 in order to
> > up files from GitHub. I recommend either using post-review (which makes
> > far easier to post/update review requests), or passing the --full-index
> > option to `git diff`.
> > Christian
> > --
> > Christian Hammond - chip...@chipx86.com
> > Review Board -http://www.review-board.org
> > VMware, Inc. -http://www.vmware.com
> > On Thu, Oct 1, 2009 at 7:24 PM, smat <g...@cictec.com> wrote:
> > > Dear all,
> > > I have recently set up Review Board for my organization, and in the
> > > process found what looks like a bug in group enumeration in the AD
> > > authentication backend. I would like to submit a fix for the project's
> > > configuration. When I try to create a new review request, an error
> > > message "global name 'logging' is not defined" appears next to "diff"
> > > on the new review request page when I click "create review request".
> > > This is the patch that I attached:
> > > diff --git a/reviewboard/accounts/backends.py b/reviewboard/accounts/
> > > backends.py
> > > index 59da9cc..c3a1ea4 100644
> > > --- a/reviewboard/accounts/backends.py
> > > +++ b/reviewboard/accounts/backends.py
> > > @@ -231,7 +231,8 @@ class ActiveDirectoryBackend(object):
> > > for group in new_groups:
> > > if group in old_seen:
> > > continue
> > > - group_data = self.search_ad(con, '(&
> > > (objectClass=group)(saMAccountName=%s))' % group)
> > > + # Search for groups with the specified DN. Use
> > > the DN rather than sAMAccountName so that behavior is correct when the
> > > values differ (e.g. if a "pre-Windows 2000" group name is set in AD).
> > > + group_data = self.search_ad(con, '(&
> > > (objectClass=group)(dn=%s))' % group)
> > > seen.update(self.get_member_of(con, group_data,
> > > seen=seen, depth=depth))
> > > else:
> > > logging.warning('ActiveDirectory recursive group
> > > check reached maximum recursion depth.')
> > > Although the text "logging" is present in the diff, I don't believe
> > > that this is relevant - removing "logging" from the diff does not
> > > alter the error message.
> > > I created the diff by running "git diff" in my local repository - am I
> > > doing something wrong?
You received this message because you are subscribed to the Google Groups
To post to this group, send email to email@example.com
To unsubscribe from this group, send email to
For more options, visit this group at