Re: how does post-review know if files changed?

2012-06-09 Thread Ryan Raasch
Yea. I do not know about the web API, I'll have to look into that.
On Jun 9, 2012 2:02 AM, Dado Feigenblatt d...@dado.org wrote:

 Hi Ryan, thanks for the reply.

 Unfortunately my server doesn't understand that request.
 I'm running ReviewBoard 1.6.3.
 I see you don't have '/api/' in your URL.
 Are you intentionally not using the api, or is this the older method of
 interfacing with the server?

 I tried the api method
  /api/review-requests/{review_**request_id}/diffs/{diff_**revision}/
 but it doesn't contain the actual text of diff.

 Eventually I got your example to work after I removed '/reviews/' from the
 url
  
 http://example.review.com/r/9/**diff/raw/http://example.review.com/r/9/diff/raw/
 But that returns a plain unicode string of the diff instead of json or xml.
 Definitely better than nothing and I might end up using that, but I just
 want to make sure there's no blessed api method for getting the actual
 text of the diffs.

 Thanks,
 Dado


 On Fri, 8 Jun 2012 00:47:43 -0700 (PDT), Ryan Raasch 
 ryan.raa...@gmail.com wrote:

 On Friday, June 8, 2012 1:41:32 AM UTC+2, dado wrote: When updating a
 review, post-review will tell you if the file changed
  since it was last submitted for review.
  How does it know that? Does RB save diffs for each request update?

 As far as I can tell, it doesn't. Post-review generates a diff based
 on the local
 changes. It then posts this diff to the server. Which maintains
 versions of the diff.
 There are not diffs of diffs, only versions.
  If so, how can I retrieve them via the web api?

 Just use curl or wget with
 http://example.review.com/**reviews/r/9/diff/raw/http://example.review.com/reviews/r/9/diff/raw/

  Thanks,
  Dado

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/**donate/ http://www.reviewboard.org/donate/[1]
  Happy user? Let us know at 
 http://www.reviewboard.org/**users/http://www.reviewboard.org/users/[2]
  -~--~~~~--**~~--~--~---
  To unsubscribe from this group, send email to
 reviewboard+unsubscribe@**googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
  For more options, visit this group at
 http://groups.google.com/**group/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en[3]

 Links:
 --
 [1] http://www.reviewboard.org/**donate/http://www.reviewboard.org/donate/
 [2] http://www.reviewboard.org/**users/http://www.reviewboard.org/users/
 [3] 
 http://groups.google.com/**group/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en


 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/**donate/ http://www.reviewboard.org/donate/
 Happy user? Let us know at 
 http://www.reviewboard.org/**users/http://www.reviewboard.org/users/
 -~--~~~~--**~~--~--~---
 To unsubscribe from this group, send email to reviewboard+unsubscribe@**
 googlegroups.com reviewboard%2bunsubscr...@googlegroups.com
 For more options, visit this group at http://groups.google.com/**
 group/reviewboard?hl=en http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: how does post-review know if files changed?

2012-06-08 Thread Ryan Raasch


On Friday, June 8, 2012 1:41:32 AM UTC+2, dado wrote:

 When updating a review, post-review will tell you if the file changed 
  since it was last submitted for review. 
  How does it know that? Does RB save diffs for each request update? 

As far as I can tell, it doesn't. Post-review generates a diff based on the 
local
changes. It then posts this diff to the server. Which maintains versions of 
the diff.
There are not diffs of diffs, only versions.
 

  If so, how can I retrieve them via the web api? 


Just use curl or wget with http://example.review.com/reviews/r/9/diff/raw/
 


  Thanks, 
  Dado 


 

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: how does post-review know if files changed?

2012-06-08 Thread Ryan Raasch


On Friday, June 8, 2012 1:41:32 AM UTC+2, dado wrote:

 When updating a review, post-review will tell you if the file changed 
  since it was last submitted for review. 


Actually it can tell you the difference between two diffs.
 

  How does it know that? Does RB save diffs for each request update? 
  If so, how can I retrieve them via the web api? 

  Thanks, 
  Dado 



-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: how does post-review know if files changed?

2012-06-08 Thread Dado Feigenblatt

Hi Ryan, thanks for the reply.

Unfortunately my server doesn't understand that request.
I'm running ReviewBoard 1.6.3.
I see you don't have '/api/' in your URL.
Are you intentionally not using the api, or is this the older method of 
interfacing with the server?


I tried the api method
  /api/review-requests/{review_request_id}/diffs/{diff_revision}/
but it doesn't contain the actual text of diff.

Eventually I got your example to work after I removed '/reviews/' from 
the url

  http://example.review.com/r/9/diff/raw/
But that returns a plain unicode string of the diff instead of json or 
xml.
Definitely better than nothing and I might end up using that, but I 
just want to make sure there's no blessed api method for getting the 
actual text of the diffs.


Thanks,
Dado


On Fri, 8 Jun 2012 00:47:43 -0700 (PDT), Ryan Raasch 
ryan.raa...@gmail.com wrote:

On Friday, June 8, 2012 1:41:32 AM UTC+2, dado wrote: When updating a
review, post-review will tell you if the file changed
 since it was last submitted for review.
 How does it know that? Does RB save diffs for each request update?

As far as I can tell, it doesn't. Post-review generates a diff based
on the local
changes. It then posts this diff to the server. Which maintains
versions of the diff.
There are not diffs of diffs, only versions.
  If so, how can I retrieve them via the web api?

Just use curl or wget with
http://example.review.com/reviews/r/9/diff/raw/

 Thanks,
 Dado

 --
 Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/ [1]
 Happy user? Let us know at http://www.reviewboard.org/users/ [2]
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en [3]

Links:
--
[1] http://www.reviewboard.org/donate/
[2] http://www.reviewboard.org/users/
[3] http://groups.google.com/group/reviewboard?hl=en


--
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


how does post-review know if files changed?

2012-06-07 Thread Dado Feigenblatt
When updating a review, post-review will tell you if the file changed 
since it was last submitted for review.

How does it know that? Does RB save diffs for each request update?
If so, how can I retrieve them via the web api?

Thanks,
Dado

--
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en