Hi Christian,
Thanks for the tip - the 500 is because of an exception to do with a
licensing issue with p4:
Traceback:
File "/usr/lib/python2.5/site-packages/django/core/handlers/base.py"
in get_response
86. response = callback(request, *callback_args,
**callback_kwargs)
File "/usr/lib/python2.5/site-packages/django/views/decorators/
cache.py" in _wrapped_view_func
44. response = view_func(request, *args, **kwargs)
...
...
35. changeset =
repository.get_scmtool().get_changeset(changenum)
File "/home/guest/dist/reviewboard/scmtools/perforce.py" in
get_changeset
57. changeset = self.p4.run_describe('-s',
str(changesetid))
File "/usr/lib/python2.5/site-packages/P4.py" in <lambda>
215. f = lambda *args: self.run(cmd, *args)
File "/usr/lib/python2.5/site-packages/P4.py" in run
260. return P4API.P4Adapter.run(self, *self.__flatten(args))
Exception Type: P4Exception at /api/json/reviewrequests/new/
Exception Value: [P4#run] Errors during command execution( "p4
describe -s 23710" )
[Error]: Can't create a new user - over license quota.
Try deleting old users with 'user -d'.
License count: 55 users used of 55 licensed.
The reason for this was that I needed to "source ~/.p4c" from the
shell, so that it could set the valid $P4USER, before then running ./
manage.py runserver
On Oct 28, 7:46 pm, "Christian Hammond" <[EMAIL PROTECTED]> wrote:
> If you run with --debug, you should be able to see more error info.
>
> Christian
>
> --
> Christian Hammond - [EMAIL PROTECTED]
> VMware, Inc.
>
> On Tue, Oct 28, 2008 at 5:00 AM, tom <[EMAIL PROTECTED]> wrote:
>
> > Hi,
>
> > Has this been resolved?
>
> > If not, this is to do with the fact that the newline character(s) are
> > in the name of the local file extracted:
>
> > here's a diff of what I did to fix the issue as a quick solution:
>
> > #
> > # Repository URL.
> > @@ -963,6 +963,7 @@
> > else:
> > die("Unknown change type '%s' for %s" % (changetype,
> > depot_path))
>
> > + new_file = new_file.rstrip('\r\n');
> > diff_cmd = ["diff", "-urNp", old_file, new_file]
> > # Diff returns "1" if differences were found.
>
> > However, I got an internal server error when I tried a test post:
>
> > Unable to accesshttp://localhost:8000/api/json/reviewrequests/new/.
> > The host path may be invalid
> > HTTP Error 500: INTERNAL SERVER ERROR
>
> > logs:
>
> > [28/Oct/2008 05:59:48] "POST /api/json/reviewrequests/new/ HTTP/1.1"
> > 500 96260
>
> > On Oct 16, 8:54 am, "Florian Föbel" <[EMAIL PROTECTED]>
> > wrote:
> > > Hi,
>
> > > I just got the same error after upgrading post-review. What I found out
> > is
> > > that the subprocess.stdout that is read for the diff command contains a
> > > newline (\n) right after the second filename in the diff header. Kind of
> > > weird...
>
> > > No fix or explanation found yet.
>
> > > Regards,
> > > Florian
>
> > > 2008/10/16 Joshua Slominski <[EMAIL PROTECTED]>
>
> > > > Can you post the debug output?
>
> > > > Sent from my iPhone
>
> > > > On Oct 15, 2008, at 6:00 PM, AndyP <[EMAIL PROTECTED]> wrote:
>
> > > > > Well I have a .reviewboardrc with valid stuff in it. I modified
> > (just
> > > > > now) the script too with the same data but it didn't work.
>
> > > > > I really suspect that that the 'execute' command isn't working right.
>
> > > > > Note that it gets past that if I change those Popen calls from 'True'
> > > > > back to 'False'.
>
> > > > > If they are 'False' it can't find the repository, and if it is 'True'
> > > > > it can't execute the diff (per the original posts in this thread).
>
> > > > > Thanks,
>
> > > > > On Oct 15, 2:01 pm, "Joshua Slominski" <[EMAIL PROTECTED]> wrote:
> > > > >> I seen that error when i forgot to put in my review board URL in
> > > > >> post-review. Also be sure that post-review is using the correct
> > > > >> SCM tool.
> > > > >> It defaults to SVN if it can't find anything else.
>
> > > > >> On Wed, Oct 15, 2008 at 4:48 PM, AndyP <[EMAIL PROTECTED]>
> > wrote:
>
> > > > >>> That didn't work for me out of the box. The two Popen() statements
> > > > >>> are different than the version that I had (v0.7 that worked), but
> > > > >>> when
> > > > >>> I changed it to True I got a different error:
>
> > > > >>> "The current directory does not contain a checkout from a supported
> > > > >>> source code repository."
>
> > > > >>> Looking at this further, it appears that the only real difference
> > is
> > > > >>> the way the execute command is created:
>
> > > > >>> v0.7
> > > > >>> data = execute('p4 info', ignore_errors=True)
>
> > > > >>> v0.8
> > > > >>> data = execute(["p4", "info"], ignore_errors=True)
>
> > > > >>> On Oct 15, 12:45 pm, "Joshua Slominski" <[EMAIL PROTECTED]>
> > > > >>> wrote:- Hide quoted text -
>
> > > > >> - Show quoted text -
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"reviewboard" group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
-~----------~----~----~----~------~----~------~--~---
