Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Ken Giusti

[X] Yes, release Proton 0.9-rc-3 as 0.9 final



- Original Message -
> From: "Rafael Schloming" 
> To: proton@qpid.apache.org, us...@qpid.apache.org
> Sent: Monday, March 16, 2015 4:42:18 PM
> Subject: [VOTE]: Proton 0.9-rc-3
> 
> Hi Everyone,
> 
> Here's a quick respin of 0.9-rc-3. The only changes from rc-2 are exactly
> those two mentioned on the rc-2 vote thread. I've included them at the end
> for reference. You can find the source artifacts in the usual location:
> 
> https://people.apache.org/~rhs/qpid-proton-0.9-rc-3/
> 
> Java binaries are here:
> 
> https://repository.apache.org/content/repositories/orgapacheqpid-1031
> 
> Please check it out and register your vote:
> 
> [   ] Yes, release Proton 0.9-rc-3 as 0.9 final
> [   ] No, because ...
> 
> --Rafael
> 
> ==
> 
> commit 810088b14dedcd12a9474687ba9cd05fc8297188
> Author: Dominic Evans 
> Date:   Mon Mar 16 12:18:20 2015 +
> 
> PROTON-834: further UTF-8 encoder fixes
> 
> After commit c65e897 it turned out there were still some issues with
> strings containing a codepoint >0xDBFF which was being incorrectly
> treated as a surrogate pair in the calculateUTF8Length method.
> 
> Fixed this up and added some more test coverage.
> 
> Closes #13
> 
> (cherry picked from commit 7b9b516d445ab9e86a0313709c77218d901435b1)
> 
> commit c2042d7d26c4383047dac2709d1a2effe0b11419
> Author: Alan Conway 
> Date:   Mon Mar 16 09:51:28 2015 -0400
> 
> PROTON-839: Proton 0.9 RC 2 blocker -
> proton_tests.utils.SyncRequestResponse
> 
> Fix to reactor.py, check for lack of SSL domain.
> 
> (cherry picked from commit e31df015a79d791e62caf9bef3f29bdfd77042ef)
> 

-- 
-K


Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Timothy Bish

On 03/16/2015 04:42 PM, Rafael Schloming wrote:

[   ] Yes, release Proton 0.9-rc-3 as 0.9 final


[X] Yes, release Proton 0.9-rc-3 as 0.9 final


--
Tim Bish
Sr Software Engineer | RedHat Inc.
tim.b...@redhat.com | www.redhat.com
skype: tabish121 | twitter: @tabish121
blog: http://timbish.blogspot.com/



[jira] [Commented] (PROTON-582) Perl language bindings do not properly identify integers from strings when encoding

2015-03-17 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/PROTON-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14365086#comment-14365086
 ] 

ASF subversion and git services commented on PROTON-582:


Commit f32643492ba6763d46caccc59752ce1fb64ced9e in qpid-proton's branch 
refs/heads/master from [~mcpierce]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=f326434 ]

PROTON-582: Added in missing is_float method to Perl bindings.


> Perl language bindings do not properly identify integers from strings when 
> encoding
> ---
>
> Key: PROTON-582
> URL: https://issues.apache.org/jira/browse/PROTON-582
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: perl-binding
>Affects Versions: 0.7
>Reporter: Darryl L. Pierce
>Assignee: Darryl L. Pierce
> Fix For: 0.8
>
>
> Rafi found the code:
>   sub isnum ($) {
>   return 0 if $_[0] eq '';
>   $_[0] ^ $_[0] ? 0 : 1
>   }
> which can differentiate strings from integers. An additional piece can check 
> for a . in a numeric value to differentiate integers from floating point 
> values.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Darryl L. Pierce
On Tue, Mar 17, 2015 at 09:42:18AM +1300, Rafael Schloming wrote:
> Hi Everyone,
> 
> Here's a quick respin of 0.9-rc-3. The only changes from rc-2 are exactly
> those two mentioned on the rc-2 vote thread. I've included them at the end
> for reference. You can find the source artifacts in the usual location:
> 
> https://people.apache.org/~rhs/qpid-proton-0.9-rc-3/
> 
> Java binaries are here:
> 
> https://repository.apache.org/content/repositories/orgapacheqpid-1031
> 
> Please check it out and register your vote:
> 
> [ X ] Yes, release Proton 0.9-rc-3 as 0.9 final
> [   ] No, because ...

Packaged for Fedora. Installed all packages.
Tested across language bindings (Python, Perl, Ruby).
Found a single non-blocking issue in Perl that I've fixed and pushed.

-- 
Darryl L. Pierce, Sr. Software Engineer @ Red Hat, Inc.
Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/



pgpWRFD4rhjeD.pgp
Description: PGP signature


[GitHub] qpid-proton pull request: NO-JIRA: Measure size of encoded data.

2015-03-17 Thread alanconway
Github user alanconway commented on the pull request:

https://github.com/apache/qpid-proton/pull/11#issuecomment-82366307
  
On Mon, 2015-03-16 at 12:31 -0700, Andrew Stitcher wrote:
> I think you new pn_data_size() possibly has the wrong side effects -
> it seems to lose the pn_data cursor position, so that merely querying
> the size will be hard to intermix with appending to the pn_data. It is
> true that this is not something the current code ever does does, but I
> think making the query function have side effects (except caching the
> size itself perhaps) is unexpected.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
Fixed

commit 973bad033ba3a1b700ab80ab4edee209ab81f05a
Author: Alan Conway 
Commit: Alan Conway 

NO-JIRA: Restore data position when measuring size of encoded data.





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Ted Ross

[ X ] Yes, release Proton 0.9-rc-3 as 0.9 final


On 03/16/2015 04:42 PM, Rafael Schloming wrote:

Hi Everyone,

Here's a quick respin of 0.9-rc-3. The only changes from rc-2 are exactly
those two mentioned on the rc-2 vote thread. I've included them at the end
for reference. You can find the source artifacts in the usual location:

 https://people.apache.org/~rhs/qpid-proton-0.9-rc-3/

Java binaries are here:

 https://repository.apache.org/content/repositories/orgapacheqpid-1031

Please check it out and register your vote:

[   ] Yes, release Proton 0.9-rc-3 as 0.9 final
[   ] No, because ...

--Rafael

==

commit 810088b14dedcd12a9474687ba9cd05fc8297188
Author: Dominic Evans 
Date:   Mon Mar 16 12:18:20 2015 +

 PROTON-834: further UTF-8 encoder fixes

 After commit c65e897 it turned out there were still some issues with
 strings containing a codepoint >0xDBFF which was being incorrectly
 treated as a surrogate pair in the calculateUTF8Length method.

 Fixed this up and added some more test coverage.

 Closes #13

 (cherry picked from commit 7b9b516d445ab9e86a0313709c77218d901435b1)

commit c2042d7d26c4383047dac2709d1a2effe0b11419
Author: Alan Conway 
Date:   Mon Mar 16 09:51:28 2015 -0400

 PROTON-839: Proton 0.9 RC 2 blocker -
proton_tests.utils.SyncRequestResponse

 Fix to reactor.py, check for lack of SSL domain.

 (cherry picked from commit e31df015a79d791e62caf9bef3f29bdfd77042ef)



Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Alan Conway
[ +1] Yes, release Proton 0.9-rc-3 as 0.9 final





Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Robbie Gemmell
[ +1 ] Yes, release Proton 0.9-rc-3 as 0.9 final

Robbie

On 16 March 2015 at 20:42, Rafael Schloming  wrote:
> Hi Everyone,
>
> Here's a quick respin of 0.9-rc-3. The only changes from rc-2 are exactly
> those two mentioned on the rc-2 vote thread. I've included them at the end
> for reference. You can find the source artifacts in the usual location:
>
> https://people.apache.org/~rhs/qpid-proton-0.9-rc-3/
>
> Java binaries are here:
>
> https://repository.apache.org/content/repositories/orgapacheqpid-1031
>
> Please check it out and register your vote:
>
> [   ] Yes, release Proton 0.9-rc-3 as 0.9 final
> [   ] No, because ...
>
> --Rafael
>
> ==
>
> commit 810088b14dedcd12a9474687ba9cd05fc8297188
> Author: Dominic Evans 
> Date:   Mon Mar 16 12:18:20 2015 +
>
> PROTON-834: further UTF-8 encoder fixes
>
> After commit c65e897 it turned out there were still some issues with
> strings containing a codepoint >0xDBFF which was being incorrectly
> treated as a surrogate pair in the calculateUTF8Length method.
>
> Fixed this up and added some more test coverage.
>
> Closes #13
>
> (cherry picked from commit 7b9b516d445ab9e86a0313709c77218d901435b1)
>
> commit c2042d7d26c4383047dac2709d1a2effe0b11419
> Author: Alan Conway 
> Date:   Mon Mar 16 09:51:28 2015 -0400
>
> PROTON-839: Proton 0.9 RC 2 blocker -
> proton_tests.utils.SyncRequestResponse
>
> Fix to reactor.py, check for lack of SSL domain.
>
> (cherry picked from commit e31df015a79d791e62caf9bef3f29bdfd77042ef)


Re: [VOTE]: Proton 0.9-rc-3

2015-03-17 Thread Božo Dragojevič
[ +1 ] Yes, release Proton 0.9-rc-3 as 0.9 final

Bozzo
On Mar 16, 2015 9:43 PM, "Rafael Schloming"  wrote:

> Hi Everyone,
>
> Here's a quick respin of 0.9-rc-3. The only changes from rc-2 are exactly
> those two mentioned on the rc-2 vote thread. I've included them at the end
> for reference. You can find the source artifacts in the usual location:
>
> https://people.apache.org/~rhs/qpid-proton-0.9-rc-3/
>
> Java binaries are here:
>
> https://repository.apache.org/content/repositories/orgapacheqpid-1031
>
> Please check it out and register your vote:
>
> [   ] Yes, release Proton 0.9-rc-3 as 0.9 final
> [   ] No, because ...
>
> --Rafael
>
> ==
>
> commit 810088b14dedcd12a9474687ba9cd05fc8297188
> Author: Dominic Evans 
> Date:   Mon Mar 16 12:18:20 2015 +
>
> PROTON-834: further UTF-8 encoder fixes
>
> After commit c65e897 it turned out there were still some issues with
> strings containing a codepoint >0xDBFF which was being incorrectly
> treated as a surrogate pair in the calculateUTF8Length method.
>
> Fixed this up and added some more test coverage.
>
> Closes #13
>
> (cherry picked from commit 7b9b516d445ab9e86a0313709c77218d901435b1)
>
> commit c2042d7d26c4383047dac2709d1a2effe0b11419
> Author: Alan Conway 
> Date:   Mon Mar 16 09:51:28 2015 -0400
>
> PROTON-839: Proton 0.9 RC 2 blocker -
> proton_tests.utils.SyncRequestResponse
>
> Fix to reactor.py, check for lack of SSL domain.
>
> (cherry picked from commit e31df015a79d791e62caf9bef3f29bdfd77042ef)
>