Re: [PATCH] Update swig INSTALL document for Python 2 bindings

2020-05-19 Thread Daniel Shahaf
Yasuhito FUTATSUKI wrote on Wed, 20 May 2020 10:18 +0900: > [[[ > * INSTALL (I.C.13): Add Note that non-release mode is requires to build > SWIG Python 2 binding. > s/13/12/ s/^ SWIG/ SWIG/. Continuation line should be indented at least two spaces. Most of us indent continuation lines three

Re: [PATCH] Update swig INSTALL document for Python 2 bindings

2020-05-19 Thread Yasuhito FUTATSUKI
Thank you for the review. On 2020/05/14 22:27, Daniel Shahaf wrote: > Yasuhito FUTATSUKI wrote on Thu, 14 May 2020 19:51 +0900: >> Hi, >> >> I overlooked that 'make clean-swig-py' doesn't remove SWIG generated >> source files if build-output.mk is generated for release mode. It is >> need to clean

Re: [PATCH] Update swig INSTALL document for Python 2 bindings

2020-05-14 Thread Daniel Shahaf
Yasuhito FUTATSUKI wrote on Thu, 14 May 2020 19:51 +0900: > Hi, > > I overlooked that 'make clean-swig-py' doesn't remove SWIG generated > source files if build-output.mk is generated for release mode. It is > need to clean them to rebuild source files for Python 2 bindings. > > So I want to upda

[PATCH] Update swig INSTALL document for Python 2 bindings

2020-05-14 Thread Yasuhito FUTATSUKI
Hi, I overlooked that 'make clean-swig-py' doesn't remove SWIG generated source files if build-output.mk is generated for release mode. It is need to clean them to rebuild source files for Python 2 bindings. So I want to update subversion/bindings/swig/INSTALL so that users don't fall into this p