Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jewel: cli: ceph command line tool chokes on ceph –w (the dash is unicode 'en dash' &ndash, copy-paste to reproduce) #10420

Merged
3 commits merged into from Aug 16, 2016

Commits on Jul 24, 2016

  1. Make usage of builtins in ceph_argparse compatible with Python 3

    Signed-off-by: Oleh Prypin <oleh@pryp.in>
    (cherry picked from commit ddf0604)
    oprypin authored and smithfarm committed Jul 24, 2016
    Copy the full SHA
    bb2dc95 View commit details
    Browse the repository at this point in the history
  2. Fix tabs->whitespace in ceph_argparse

    Signed-off-by: Oleh Prypin <oleh@pryp.in>
    (cherry picked from commit bef2c83)
    oprypin authored and smithfarm committed Jul 24, 2016
    Copy the full SHA
    b01af21 View commit details
    Browse the repository at this point in the history
  3. pybind/ceph_argparse: handle non ascii unicode args

    we raise UnicodeDecodeError at seeing non-ascii args if we fail to match
    it with any command signatures. instead, we should use a unicode string
    for representing the error in that case. please note, the exception is
    not printed at all in real-world. =)
    
    Fixes: http://tracker.ceph.com/issues/12287
    Signed-off-by: Kefu Chai <kchai@redhat.com>
    (cherry picked from commit 5864626)
    tchaikov authored and smithfarm committed Jul 24, 2016
    Copy the full SHA
    3da251f View commit details
    Browse the repository at this point in the history