Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-14 Thread Tim Bell
On 11/14/12 01:22, Erik Joelsson wrote: Yes, it's intended. What I really would like to express is (in pseudo code): if (OS=windows AND BIN_FILE in ACCEPTED_SIZE_DIFF AND (DIFF_SIZE_NUM = 512 OR DIFF_SIZE_NUM = -512)) I wasn't sure I could write this in bash, but maybe I'm wrong. I figured th

Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-14 Thread Erik Joelsson
On 2012-11-13 23:06, Tim Bell wrote: Hi Erik: This is the open part of this change. It only changes the compare script to handle the deploy build artifacts. http://cr.openjdk.java.net/~erikj/8002026/webrev.root.01/ I was getti

Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-13 Thread Tim Bell
Hi Erik: This is the open part of this change. It only changes the compare script to handle the deploy build artifacts. http://cr.openjdk.java.net/~erikj/8002026/webrev.root.01/ I was getting ready to push this for you when I not

Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-12 Thread Kelly O'Hair
Looks ok. -kto On Nov 12, 2012, at 6:44 AM, Erik Joelsson wrote: > This is the open part of this change. It only changes the compare script to > handle the deploy build artifacts. > > http://cr.openjdk.java.net/~erikj/8002026/webrev.root.01/ >

Review Request: 8002026: build-infra: deploy repository building

2012-11-12 Thread Erik Joelsson
This is the open part of this change. It only changes the compare script to handle the deploy build artifacts. http://cr.openjdk.java.net/~erikj/8002026/webrev.root.01/ /Erik