Re: Unicode and Subversion

2014-05-05 Thread Stefan Bohne
Here you go, David.

2014-04-29 10:12:58,425 - DEBUG -  - DiffParser.parse: Beginning parse of 
diff, size = 429
2014-04-29 10:12:58,442 - DEBUG -  - DiffParser.parse: Finished parsing 
diff.
2014-04-29 10:12:58,445 - ERROR - None - stbohne - 
/api/review-requests/2194/diffs/ - Error uploading new diff: 'ascii' codec 
can't decode byte 0xc3 in position 20: ordinal not in range(128)
Traceback (most recent call last):
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/webapi/resources.py,
 
line 2115, in create
request.FILES.get('parent_diff_path'))
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/reviews/forms.py,
 
line 309, in create
history)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/diffviewer/forms.py,
 
line 67, in create
request=self.request)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/diffviewer/managers.py,
 
line 82, in create_from_upload
save=save)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/diffviewer/managers.py,
 
line 103, in create_from_data
check_existence=(not parent_diff_file_contents)))
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/diffviewer/managers.py,
 
line 213, in _process_files
request=request))):
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/models.py,
 
line 211, in get_file_exists
base_commit_id, request)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/models.py,
 
line 333, in _get_file_exists_uncached
exists = self.get_scmtool().file_exists(path, revision)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/core.py,
 
line 80, in file_exists
self.get_file(path, revision)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/svn.py,
 
line 191, in get_file
return self._do_on_path(get_file_data, path, revision)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/svn.py,
 
line 145, in _do_on_path
normpath = self.__normalize_path(path)
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.7.21-py2.7.egg/reviewboard/scmtools/svn.py,
 
line 318, in __normalize_path
if path.startswith(self.repopath):
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 20: 
ordinal not in range(128)


Am Sonntag, 4. Mai 2014 05:56:22 UTC+2 schrieb David Trowbridge:

 Stefan,

 This has improved significantly in the 2.0 RCs, but we should probably 
 still try to fix it in the 1.7.x series. Can you see if you can get a 
 traceback for the UnicodeDecodeError?

 -David


 On Wed, Apr 30, 2014 at 2:52 AM, Stefan Bohne 
 stefan...@sentec-elektronik.de javascript: wrote:

 Review Board 1.7.21

 Am Dienstag, 29. April 2014 22:02:37 UTC+2 schrieb David Trowbridge:

 What version of Review Board are you using?

 -David


 On Tue, Apr 29, 2014 at 3:22 AM, Stefan Bohne stefan...@sentec-
 elektronik.de wrote:

  Thanks, David. That fixed it. But now reviewboard itself seems to 
 stumble over unicode paths.

 $ rbt post --repository-url=http://svn/svn/main/ --username=stbohne 
 --password=XXX -d --server=http://reviews/ 8792

  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55) 
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main/
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry 
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
  Making HTTP GET request to http://reviews/api/review-requests/
  Making HTTP POST request to http://reviews/api/review-requests/
  Making HTTP GET request to http://reviews/api/review-
 requests/2194/diffs/
  Making HTTP POST request to http://reviews/api/review-
 requests/2194/diffs/
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'ascii' codec can't decode 
 byte 0xc3 in position 20: ordinal not in range(128)]}, u'stat': u'fail', 
 u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
 Traceback (most recent call last):
   File /usr/local/bin/rbt, line 9, in module
 

Re: Unicode and Subversion

2014-05-03 Thread David Trowbridge
Stefan,

This has improved significantly in the 2.0 RCs, but we should probably
still try to fix it in the 1.7.x series. Can you see if you can get a
traceback for the UnicodeDecodeError?

-David


On Wed, Apr 30, 2014 at 2:52 AM, Stefan Bohne 
stefan.bo...@sentec-elektronik.de wrote:

 Review Board 1.7.21

 Am Dienstag, 29. April 2014 22:02:37 UTC+2 schrieb David Trowbridge:

 What version of Review Board are you using?

 -David


 On Tue, Apr 29, 2014 at 3:22 AM, Stefan Bohne stefan...@sentec-
 elektronik.de wrote:

  Thanks, David. That fixed it. But now reviewboard itself seems to
 stumble over unicode paths.

 $ rbt post --repository-url=http://svn/svn/main/ --username=stbohne
 --password=XXX -d --server=http://reviews/ 8792

  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main/
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
  Making HTTP GET request to http://reviews/api/review-requests/
  Making HTTP POST request to http://reviews/api/review-requests/
  Making HTTP GET request to http://reviews/api/review-
 requests/2194/diffs/
  Making HTTP POST request to http://reviews/api/review-
 requests/2194/diffs/
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'ascii' codec can't decode byte
 0xc3 in position 20: ordinal not in range(128)]}, u'stat': u'fail',
 u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
 Traceback (most recent call last):
   File /usr/local/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.6', 'console_scripts', 'rbt')()
   File /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.
 7.egg/rbtools/commands/main.py, line 134, in main
 command.run_from_argv([RB_MAIN, command_name] + args)
   File /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.
 7.egg/rbtools/commands/__init__.py, line 422, in run_from_argv
 exit_code = self.main(*args) or 0
   File /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.
 7.egg/rbtools/commands/post.py, line 769, in main
 submit_as=self.options.submit_as)
   File /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.
 7.egg/rbtools/commands/post.py, line 545, in post_request
 raise CommandError('\n'.join(error_msg))
 rbtools.commands.CommandError: Error uploading diff


 One or more fields had errors (HTTP 400, API Error 105)

 path: 'ascii' codec can't decode byte 0xc3 in position 20: ordinal
 not in range(128)

 Your review request still exists, but the diff is not attached.


 Am Dienstag, 29. April 2014 00:01:50 UTC+2 schrieb David Trowbridge:

 I believe that this is because we remove locale configuration, since
 svn likes to localize some of the control strings in the diffs. Could you
 test a prospective code change for this?

  In rbtools/util/process.py, there are two lines:

  env['LC_ALL'] = 'C'
 env['LANGUAGE'] = 'C'

  Can you change 'C' to 'C.UTF-8' and see if it works?

  -David


 On Mon, Apr 28, 2014 at 6:06 AM, Stefan Bohne stefan...@sentec-
 elektronik.de wrote:

 Hello,

 I'm having problems posting a review request from a subversion
 repository with rbt post that contains non-ascii file names. It seems that
 rbt post executes svn diff with a non-unicode locale and that makes svn
 fail. The same svn command works when typed on the command line. But it
 could also be a configuration problem on my side. Any hints would be much
 appreciated. Below is the command line output with debug infos.


 $ rbt post --repository-url=http://svn/svn/main --username=stbohne
 --password=XXX -d --server=http://reviews/ 8792
  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /,
 Supports changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main
  Running: svn info http://svn/svn/main --non-interactive
  Running: 

Re: Unicode and Subversion

2014-04-30 Thread Stefan Bohne
 Review Board 1.7.21

Am Dienstag, 29. April 2014 22:02:37 UTC+2 schrieb David Trowbridge:

 What version of Review Board are you using?

 -David


 On Tue, Apr 29, 2014 at 3:22 AM, Stefan Bohne 
 stefan...@sentec-elektronik.de javascript: wrote:

  Thanks, David. That fixed it. But now reviewboard itself seems to 
 stumble over unicode paths.

 $ rbt post --repository-url=http://svn/svn/main/ --username=stbohne 
 --password=XXX -d --server=http://reviews/ 8792

  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55) 
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main/
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry 
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
  Making HTTP GET request to http://reviews/api/review-requests/
  Making HTTP POST request to http://reviews/api/review-requests/
  Making HTTP GET request to 
 http://reviews/api/review-requests/2194/diffs/
  Making HTTP POST request to 
 http://reviews/api/review-requests/2194/diffs/
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'ascii' codec can't decode byte 
 0xc3 in position 20: ordinal not in range(128)]}, u'stat': u'fail', 
 u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
 Traceback (most recent call last):
   File /usr/local/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.6', 'console_scripts', 'rbt')()
   File 
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/main.py,
  
 line 134, in main
 command.run_from_argv([RB_MAIN, command_name] + args)
   File 
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/__init__.py,
  
 line 422, in run_from_argv
 exit_code = self.main(*args) or 0
   File 
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py,
  
 line 769, in main
 submit_as=self.options.submit_as)
   File 
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py,
  
 line 545, in post_request
 raise CommandError('\n'.join(error_msg))
 rbtools.commands.CommandError: Error uploading diff


 One or more fields had errors (HTTP 400, API Error 105)

 path: 'ascii' codec can't decode byte 0xc3 in position 20: ordinal 
 not in range(128)

 Your review request still exists, but the diff is not attached.


 Am Dienstag, 29. April 2014 00:01:50 UTC+2 schrieb David Trowbridge: 

 I believe that this is because we remove locale configuration, since svn 
 likes to localize some of the control strings in the diffs. Could you test 
 a prospective code change for this? 

  In rbtools/util/process.py, there are two lines:

  env['LC_ALL'] = 'C'
 env['LANGUAGE'] = 'C'
  
  Can you change 'C' to 'C.UTF-8' and see if it works?

  -David
  

 On Mon, Apr 28, 2014 at 6:06 AM, Stefan Bohne stefan...@sentec-
 elektronik.de wrote:

 Hello,

 I'm having problems posting a review request from a subversion 
 repository with rbt post that contains non-ascii file names. It seems that 
 rbt post executes svn diff with a non-unicode locale and that makes svn 
 fail. The same svn command works when typed on the command line. But it 
 could also be a configuration problem on my side. Any hints would be much 
 appreciated. Below is the command line output with debug infos.


 $ rbt post --repository-url=http://svn/svn/main --username=stbohne 
 --password=XXX -d --server=http://reviews/ 8792
  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports 
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry 
 http://svn/svn/main/@8791 

Re: Unicode and Subversion

2014-04-29 Thread Stefan Bohne
Thanks, David. That fixed it. But now reviewboard itself seems to 
stumble over unicode paths.


$ rbt post --repository-url=http://svn/svn/main/ --username=stbohne 
--password=XXX -d --server=http://reviews/ 8792

 RBTools 0.6
 Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
[GCC 4.8.1]
 Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
 Home = /home/stbohne
 Current directory = /home/stbohne
 Checking for a Subversion repository...
 Running: svn info http://svn/svn/main/ --non-interactive
 Running: diff --version
 repository info: Path: http://svn/svn/main, Base path: /, Supports 
changesets: False

 Making HTTP GET request to http://reviews/api/
 Making HTTP GET request to http://reviews/api/info/
 Running: svn log -r 8792 -l 1 --xml http://svn/svn/main/
 Running: svn info http://svn/svn/main/ --non-interactive
 Running: diff --version
 repository info: Path: http://svn/svn/main, Base path: /, Supports 
changesets: False

 Running: svn status --ignore-externals
 Running: svn diff --diff-cmd=diff --notice-ancestry 
http://svn/svn/main/@8791 http://svn/svn/main/@8792

 Making HTTP GET request to http://reviews/api/review-requests/
 Making HTTP POST request to http://reviews/api/review-requests/
 Making HTTP GET request to 
http://reviews/api/review-requests/2194/diffs/
 Making HTTP POST request to 
http://reviews/api/review-requests/2194/diffs/

 Got API Error 105 (HTTP code 400): One or more fields had errors
 Error data: {u'fields': {u'path': [u'ascii' codec can't decode 
byte 0xc3 in position 20: ordinal not in range(128)]}, u'stat': 
u'fail', u'err': {u'msg': u'One or more fields had errors', u'code': 105}}

Traceback (most recent call last):
  File /usr/local/bin/rbt, line 9, in module
load_entry_point('RBTools==0.6', 'console_scripts', 'rbt')()
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/main.py, 
line 134, in main

command.run_from_argv([RB_MAIN, command_name] + args)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/__init__.py, 
line 422, in run_from_argv

exit_code = self.main(*args) or 0
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py, 
line 769, in main

submit_as=self.options.submit_as)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py, 
line 545, in post_request

raise CommandError('\n'.join(error_msg))
rbtools.commands.CommandError: Error uploading diff


One or more fields had errors (HTTP 400, API Error 105)

path: 'ascii' codec can't decode byte 0xc3 in position 20: ordinal 
not in range(128)


Your review request still exists, but the diff is not attached.


Am Dienstag, 29. April 2014 00:01:50 UTC+2 schrieb David Trowbridge:

   I believe that this is because we remove locale configuration, since
   svn likes to localize some of the control strings in the diffs.
   Could you test a prospective code change for this?

   In rbtools/util/process.py, there are two lines:

   env['LC_ALL'] = 'C'
   env['LANGUAGE'] = 'C'

   Can you change 'C' to 'C.UTF-8' and see if it works?

   -David


   On Mon, Apr 28, 2014 at 6:06 AM, Stefan Bohne
   stefan...@sentec-elektronik.de javascript: wrote:

   Hello,

   I'm having problems posting a review request from a subversion
   repository with rbt post that contains non-ascii file names. It
   seems that rbt post executes svn diff with a non-unicode locale
   and that makes svn fail. The same svn command works when typed
   on the command line. But it could also be a configuration
   problem on my side. Any hints would be much appreciated. Below
   is the command line output with debug infos.


   $ rbt post --repository-url=http://svn/svn/main
   http://svn/svn/main --username=stbohne --password=XXX -d
   --server=http://reviews/ 8792
 RBTools 0.6
 Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
   [GCC 4.8.1]
 Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
 Home = /home/stbohne
 Current directory = /home/stbohne
 Checking for a Subversion repository...
 Running: svn info http://svn/svn/main --non-interactive
 Running: diff --version
 repository info: Path: http://svn/svn/main, Base path: /,
   Supports changesets: False
 Making HTTP GET request to http://reviews/api/
 Making HTTP GET request to http://reviews/api/info/
 Running: svn log -r 8792 -l 1 --xml http://svn/svn/main
 Running: svn info http://svn/svn/main --non-interactive
 Running: diff --version
 repository info: Path: http://svn/svn/main, Base path: /,
   Supports changesets: False
 Running: svn status --ignore-externals
 Running: svn diff --diff-cmd=diff --notice-ancestry
   http://svn/svn/main/@8791 http://svn/svn/main/@8792
   Failed to execute command: ['svn', 'diff', 

Re: Unicode and Subversion

2014-04-29 Thread David Trowbridge
What version of Review Board are you using?

-David


On Tue, Apr 29, 2014 at 3:22 AM, Stefan Bohne 
stefan.bo...@sentec-elektronik.de wrote:

  Thanks, David. That fixed it. But now reviewboard itself seems to stumble
 over unicode paths.

 $ rbt post --repository-url=http://svn/svn/main/ --username=stbohne
 --password=XXX -d --server=http://reviews/ 8792

  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main/
  Running: svn info http://svn/svn/main/ --non-interactive

  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
  Making HTTP GET request to http://reviews/api/review-requests/
  Making HTTP POST request to http://reviews/api/review-requests/
  Making HTTP GET request to
 http://reviews/api/review-requests/2194/diffs/
  Making HTTP POST request to
 http://reviews/api/review-requests/2194/diffs/
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'ascii' codec can't decode byte
 0xc3 in position 20: ordinal not in range(128)]}, u'stat': u'fail',
 u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
 Traceback (most recent call last):
   File /usr/local/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.6', 'console_scripts', 'rbt')()
   File
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/main.py,
 line 134, in main
 command.run_from_argv([RB_MAIN, command_name] + args)
   File
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/__init__.py,
 line 422, in run_from_argv
 exit_code = self.main(*args) or 0
   File
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py,
 line 769, in main
 submit_as=self.options.submit_as)
   File
 /usr/local/lib/python2.7/dist-packages/RBTools-0.6-py2.7.egg/rbtools/commands/post.py,
 line 545, in post_request
 raise CommandError('\n'.join(error_msg))
 rbtools.commands.CommandError: Error uploading diff


 One or more fields had errors (HTTP 400, API Error 105)

 path: 'ascii' codec can't decode byte 0xc3 in position 20: ordinal not
 in range(128)

 Your review request still exists, but the diff is not attached.


 Am Dienstag, 29. April 2014 00:01:50 UTC+2 schrieb David Trowbridge:

 I believe that this is because we remove locale configuration, since svn
 likes to localize some of the control strings in the diffs. Could you test
 a prospective code change for this?

  In rbtools/util/process.py, there are two lines:

  env['LC_ALL'] = 'C'
 env['LANGUAGE'] = 'C'

  Can you change 'C' to 'C.UTF-8' and see if it works?

  -David


 On Mon, Apr 28, 2014 at 6:06 AM, Stefan Bohne stefan...@sentec-
 elektronik.de wrote:

 Hello,

 I'm having problems posting a review request from a subversion
 repository with rbt post that contains non-ascii file names. It seems that
 rbt post executes svn diff with a non-unicode locale and that makes svn
 fail. The same svn command works when typed on the command line. But it
 could also be a configuration problem on my side. Any hints would be much
 appreciated. Below is the command line output with debug infos.


 $ rbt post --repository-url=http://svn/svn/main --username=stbohne
 --password=XXX -d --server=http://reviews/ 8792
  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
 Failed to execute command: ['svn', 'diff', '--diff-cmd=diff',
 '--notice-ancestry', 'http://svn/svn/main/@8791', '
 

Re: Unicode and Subversion

2014-04-28 Thread David Trowbridge
I believe that this is because we remove locale configuration, since svn
likes to localize some of the control strings in the diffs. Could you test
a prospective code change for this?

In rbtools/util/process.py, there are two lines:

env['LC_ALL'] = 'C'
env['LANGUAGE'] = 'C'

Can you change 'C' to 'C.UTF-8' and see if it works?

-David


On Mon, Apr 28, 2014 at 6:06 AM, Stefan Bohne 
stefan.bo...@sentec-elektronik.de wrote:

 Hello,

 I'm having problems posting a review request from a subversion repository
 with rbt post that contains non-ascii file names. It seems that rbt post
 executes svn diff with a non-unicode locale and that makes svn fail. The
 same svn command works when typed on the command line. But it could also be
 a configuration problem on my side. Any hints would be much appreciated.
 Below is the command line output with debug infos.


 $ rbt post --repository-url=http://svn/svn/main --username=stbohne
 --password=XXX -d --server=http://reviews/ 8792
  RBTools 0.6
  Python 2.7.5+ (default, Feb 27 2014, 19:39:55)
 [GCC 4.8.1]
  Running on Linux-3.11.0-18-generic-i686-with-Ubuntu-13.10-saucy
  Home = /home/stbohne
  Current directory = /home/stbohne
  Checking for a Subversion repository...
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Making HTTP GET request to http://reviews/api/
  Making HTTP GET request to http://reviews/api/info/
  Running: svn log -r 8792 -l 1 --xml http://svn/svn/main
  Running: svn info http://svn/svn/main --non-interactive
  Running: diff --version
  repository info: Path: http://svn/svn/main, Base path: /, Supports
 changesets: False
  Running: svn status --ignore-externals
  Running: svn diff --diff-cmd=diff --notice-ancestry
 http://svn/svn/main/@8791 http://svn/svn/main/@8792
 Failed to execute command: ['svn', 'diff', '--diff-cmd=diff',
 '--notice-ancestry', 'http://svn/svn/main/@8791', '
 http://svn/svn/main/@8792']
 [svn: E22: Can't convert string from 'UTF-8' to native encoding:\n,
 'svn: E22: Index: Beaglebone Black/Z?\\195?\\188rn/
 branches/silo/var/www/silo/welcome.cgi\n', '=
 ==\n', '\n', 'svn: E175002: Error
 reading spooled REPORT request response\n']
 stbohne@sentec116:~$ locale
 LANG=de_DE.UTF-8
 LANGUAGE=
 LC_CTYPE=de_DE.UTF-8
 LC_NUMERIC=de_DE.UTF-8
 LC_TIME=de_DE.UTF-8
 LC_COLLATE=de_DE.UTF-8
 LC_MONETARY=de_DE.UTF-8
 LC_MESSAGES=de_DE.UTF-8
 LC_PAPER=de_DE.UTF-8
 LC_NAME=de_DE.UTF-8
 LC_ADDRESS=de_DE.UTF-8
 LC_TELEPHONE=de_DE.UTF-8
 LC_MEASUREMENT=de_DE.UTF-8
 LC_IDENTIFICATION=de_DE.UTF-8
 LC_ALL=
 $ svn diff --diff-cmd=diff --notice-ancestry http://svn/svn/main/@8791
 http://svn/svn/main/@8792
 Index: Beaglebone Black/Zürn/branches/silo/var/www/silo/welcome.cgi
 ===
 --- Beaglebone Black/Zürn/branches/silo/var/www/silo/welcome.cgi
  (Revision 8791)
 +++ Beaglebone Black/Zürn/branches/silo/var/www/silo/welcome.cgi
  (Revision 8792)
 @@ -1,7 +1,6 @@
  #!/usr/bin/python
  import cgi
  import cgitb; cgitb.enable()
 -import pexpect
  import os
  import hashlib

 Thanks,
 Stefan

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 --- You received this message because you are subscribed to the Google
 Groups reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.