Darryl, The --p4-* options have a bug where settings in a .p4config file still had higher priority than the command-line options. I'll make sure that the fix for that bug is in the next rbtools point release.
Thanks, -David On Thu, Feb 20, 2014 at 6:16 AM, Darryl McCutcheon <[email protected] > wrote: > I'm seeing a couple issues with the perforce integration as well. The one > above with the case mismatch, as well as the --p4-client parameter is not > respected in RBTools 0.5.7. > > An integration with eclipse and reviewboard ( > https://code.google.com/p/p4reviewboard/) breaks because this parameter > is not respected. > > > On Saturday, 15 February 2014 20:03:45 UTC-5, David Trowbridge wrote: > >> Greg, >> >> You're probably getting hit by case mismatches ( >> https://code.google.com/p/reviewboard/issues/detail?id=3221). We'll be >> fixing this for 0.5.8. >> >> -David >> >> >> On Wed, Feb 12, 2014 at 5:14 PM, Greg Dean <[email protected]> wrote: >> >>> Hi there, I've updated to 0.5.7 and I'm running from cygwin and I get >>> the same result. Can rbt be run from cygwin? Here is what I get: >>> >>> >>> $ rbt post -d -o 299074 >>> DEBUG:root:Checking for a Subversion repository... >>> DEBUG:root:Checking for a Git repository... >>> DEBUG:root:Checking for a Mercurial repository... >>> DEBUG:root:Checking for a CVS repository... >>> DEBUG:root:Checking for a Perforce repository... >>> DEBUG:root:Running: p4 info >>> DEBUG:root:Checking for a Plastic repository... >>> DEBUG:root:Checking for a ClearCase repository... >>> DEBUG:root:Checking for a Bazaar repository... >>> ERROR:root:The current directory does not contain a checkout from a >>> supported source code repository. >>> >>> >>> On Tuesday, February 4, 2014 1:14:16 PM UTC-8, Karl Nordstrom wrote: >>>> >>>> Thanks for the quick fix! 0.5.7 did it. >>>> >>>> Karl >>>> >>>> On Monday, 3 February 2014 13:50:03 UTC-8, Karl Nordstrom wrote: >>>>> >>>>> Okay, looking forward to getting the fix. >>>>> >>>>> Thanks, >>>>> Karl >>>>> >>>>> On Monday, 3 February 2014 13:28:17 UTC-8, David Trowbridge wrote: >>>>>> >>>>>> Karl, >>>>>> >>>>>> It looks like we introduced a regression in 0.5.6 where it only will >>>>>> post from the client root. We'll get this sorted out asap. >>>>>> >>>>>> -David >>>>>> >>>>>> >>>>>> On Mon, Feb 3, 2014 at 1:17 PM, Karl Nordstrom <[email protected]>wrote: >>>>>> >>>>>>> I am attempting to generate a diff using rbt diff from a perforce >>>>>>> workspace. I get the same error when I do rbt post. >>>>>>> >>>>>>> (Thank you for helping with the previous problem I was having, it >>>>>>> appears to be resolved with RBTools 0.5.6.) >>>>>>> >>>>>>> My p4config is as follows: >>>>>>> >>>>>>> P4PORT=stork:1666 >>>>>>> P4USER=knordstrom >>>>>>> P4CLIENT=knordstrom_passenger >>>>>>> >>>>>>> The p4config appears to be setup correctly. I can do p4 diff from >>>>>>> the commandline and generate a diff without errors. Unfortunately, this >>>>>>> diff is not in a format that ReviewBoard understands. >>>>>>> >>>>>>> My .reviewboard rc file is as follows (just one line): >>>>>>> >>>>>>> REVIEWBOARD_URL = "http://reviewboard/"; >>>>>>> >>>>>>> I have tried the following on the commandline: >>>>>>> >>>>>>> rbt diff -d >>>>>>> rbt diff -d 25012 (the latest changelist number) >>>>>>> rbt post -d >>>>>>> rbt post -d 25012 >>>>>>> >>>>>>> In all cases I get the same result: >>>>>>> >>>>>>> DEBUG:root:Checking for a Subversion repository... >>>>>>> DEBUG:root:Checking for a Git repository... >>>>>>> DEBUG:root:Checking for a Mercurial repository... >>>>>>> DEBUG:root:Checking for a CVS repository... >>>>>>> DEBUG:root:Checking for a Perforce repository... >>>>>>> DEBUG:root:Running: p4 info >>>>>>> DEBUG:root:Checking for a Plastic repository... >>>>>>> DEBUG:root:Checking for a ClearCase repository... >>>>>>> DEBUG:root:Checking for a Bazaar repository... >>>>>>> ERROR:root:The current directory does not contain a checkout from a >>>>>>> supported source code repository. >>>>>>> >>>>>>> When I do p4 info it provides relevant information. I'm hesitant to >>>>>>> post it here as it shows company specific information. >>>>>>> >>>>>>> Is there something that I need to do in the configuration? >>>>>>> >>>>>>> Is this something to do with the way that Perforce works? As far as >>>>>>> I can tell, Perforce works on workspaces (a collection of files that are >>>>>>> being edited) rather than directories. That said, I would have thought >>>>>>> that >>>>>>> RBTools could find it via p4 diff, which references p4config. >>>>>>> >>>>>>> Ideally, I want to post to reviewboard based on changed files rather >>>>>>> than an already committed changelist, but right now any solution would >>>>>>> be >>>>>>> welcome. >>>>>>> >>>>>>> Does anyone have an example of how to configure RBTools to work with >>>>>>> Perforce? I have searched all over and have not found any concrete >>>>>>> examples. >>>>>>> >>>>>>> Thanks, >>>>>>> Karl >>>>>>> >>>>>>> -- >>>>>>> Get the Review Board Power Pack at http://www.reviewboard.org/pow >>>>>>> erpack/ >>>>>>> --- >>>>>>> Sign up for Review Board hosting at RBCommons: >>>>>>> https://rbcommons.com/ >>>>>>> --- >>>>>>> Happy user? Let us know at http://www.reviewboard.org/users/ >>>>>>> --- >>>>>>> You received this message because you are subscribed to the Google >>>>>>> Groups "reviewboard" group. >>>>>>> To unsubscribe from this group and stop receiving emails from it, >>>>>>> send an email to [email protected]. >>>>>>> For more options, visit https://groups.google.com/groups/opt_out. >>>>>>> >>>>>> >>>>>> -- >>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ >>> --- >>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ >>> --- >>> Happy user? Let us know at http://www.reviewboard.org/users/ >>> --- >>> You received this message because you are subscribed to the Google >>> Groups "reviewboard" group. >>> To unsubscribe from this group and stop receiving emails from it, send >>> an email to [email protected]. >>> For more options, visit https://groups.google.com/groups/opt_out. >>> >> >> -- > Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ > --- > Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ > --- > Happy user? Let us know at http://www.reviewboard.org/users/ > --- > You received this message because you are subscribed to the Google Groups > "reviewboard" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/groups/opt_out. > -- Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ --- Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ --- Happy user? Let us know at http://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
