txAWS

Merge lp:~therve/txaws/ec2-unicode into lp:txaws

  1. ec2-unicode
  2. Merge into trunk
Proposed by Thomas Herve
Status: Merged
Approved by: Thomas Herve
Approved revision: 111
Merged at revision: 112
Proposed branch: lp:~therve/txaws/ec2-unicode
Merge into: lp:txaws
Diff against target: 0 lines
To merge this branch: bzr merge lp:~therve/txaws/ec2-unicode
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+89859@code.launchpad.net

Description of the change

A fairly simple localized change I hope, which fixes the problem we have in Landscape and CloudDeck with 2.7.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

I'm not really sure what the rules are for voting on txaws merge proposals.

10 + strings, but unicode strings are supported and will be encode in UTF-8.
*encoded ?

other than that, this looks like a reasonable change to me.

Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
Thomas Herve (therve) wrote :

We have a one review rule for now, so thanks a lot for your review! :)

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

+1

[1]

+ L{Signature.encode} accepts unicode strings and encode them un UTF-8.

Typo un->to.

lp:~therve/txaws/ec2-unicode updated
112. By Thomas Herve

Typo

Preview Diff

Empty

Subscribers

People subscribed via source and target branches

to all changes: