Re: swig-py3 str/bytes difference in svn_config_*()

2019-12-13 Thread Yasuhito FUTATSUKI
Though this issue was already resolved... On 2019/12/13 17:28, Yasuhito FUTATSUKI wrote: > On 2019/12/13 16:24, Daniel Shahaf wrote: >> Thanks for the quick fix.  However, it doesn't work on my machine (Debian >> stretch). >> >> With your latest patch, «svn_config_get_user_config_path(None,

"svn patch" and the TAB character

2019-12-13 Thread Doug Robinson
Folks: If I use "svn diff --patch compatible" to produce a patch file. Then edit that patch file using an editor with settings that replace the TAB with a "proper" number of SPACE characters, the "svn patch" command will declare: $ svn patch ../n.txt Skipped missing target: 'TheFile (revision

Re: swig-py3 str/bytes difference in svn_config_*()

2019-12-13 Thread Yasuhito FUTATSUKI
On 2019/12/13 20:44, Yasuhito FUTATSUKI wrote: > On 2019/12/13 19:47, Daniel Shahaf wrote: >> Yasuhito FUTATSUKI wrote on Fri, Dec 13, 2019 at 17:28:18 +0900: >>> Thank you for testing. I also tested and same result on FreeBSD >>> with Python 3.6. It seems "z*" format in PyArg_ParseTuple() doesn't

Re: swig-py3 str/bytes difference in svn_config_*()

2019-12-13 Thread Yasuhito FUTATSUKI
On 2019/12/13 19:47, Daniel Shahaf wrote: > Yasuhito FUTATSUKI wrote on Fri, Dec 13, 2019 at 17:28:18 +0900: >> Thank you for testing. I also tested and same result on FreeBSD >> with Python 3.6. It seems "z*" format in PyArg_ParseTuple() doesn't >> work I expected. >> >> On the other hand, I

Re: [#4840] Merge assertion failure in svn_sort__array_insert

2019-12-13 Thread Julian Foad
Here are the three patches resulting from my work so far: merge-4840-backtrace-4.patch merge-4840-print-empty-range.patch merge-4840-test-1.patch Together, anyone who wants to follow along can reproduce the errors. These are not quite in a clean enough state to commit, but I expect to commit

Re: swig-py3 str/bytes difference in svn_config_*()

2019-12-13 Thread Daniel Shahaf
Yasuhito FUTATSUKI wrote on Fri, Dec 13, 2019 at 17:28:18 +0900: > Thank you for testing. I also tested and same result on FreeBSD > with Python 3.6. It seems "z*" format in PyArg_ParseTuple() doesn't > work I expected. > > On the other hand, I can't find good reason to care typemap for > "const

Re: swig-py3 str/bytes difference in svn_config_*()

2019-12-13 Thread Yasuhito FUTATSUKI
On 2019/12/13 16:24, Daniel Shahaf wrote: Thanks for the quick fix. However, it doesn't work on my machine (Debian stretch). With your latest patch, «svn_config_get_user_config_path(None, None)» calls the C function of the name with «path == NULL» and segfaults immediately, because the