Re: Python3 work [was: The run up to Subversion 1.13.0]

2019-09-22 Thread Johan Corveleyn
On Sat, Sep 21, 2019 at 10:28 AM Yasuhito FUTATSUKI  wrote:
>
> On 2019/09/20 0:44, Daniel Shahaf wrote:
> > Yasuhito FUTATSUKI wrote on Thu, 19 Sep 2019 06:41 +00:00:
>
> >> The FAIL on test_merge_peg3 is a newline style issue on reading file
> >> in raw mode, which is introduced by my patch.
> >> (https://svn.apache.org/viewvc/subversion/branches/swig-py3/subversion/bindings/swig/python/tests/client.py?view=annotate#l438)
> >>
> >> Rest of those errors and failures are in the tests that were added only
> >> to swig-py, and it seems most of the causes of them are confusion of
> >> paths in platform specific format and Subversion's canoical format,
> >> as the comment in SubversionClientTestCase.test_update4() says.
> >>
> >> (I'll try to fix those issues on check-swig-py later, hopefully
> >> next weekend.)
> >
> > Thanks!  No rush.
>
> Now I re-sync swig-py3 branch with trunk and make a patch addressing
> issues on testing on other than Unix/Linux platform.
>
> I've tested on FreeBSD 11.2, with Python 2.7 and Python 3.6.
>
> Could anyone please test it on Windows ?

Tested on Windows 7, and yes, this fixes the previous test failures
with Python 2.7.16. Thanks!

Test output:
[[[
Testing Release configuration on local repository.
-- Running Swig Python tests --
..s.
.
--
Ran 153 tests in 63.214s

OK (skipped=1)
]]]


Building with Python 3.7.4 still fails with the same error though (no
problem, I know your patch wasn't addressing that, just mentioning it
here for completeness). Consequently I haven't been able to run the
swig-python tests with python 3.7 yet.

[[[
c:\python37\include\pytime.h(123): error C4115: 'timeval': named type
definition in parentheses
[C:\research\svn\dev\swig-py3\build\win32\vcnet-vcproj\libsvn_swig_py.vcxproj
]]]

-- 
Johan


Re: Python3 work [was: The run up to Subversion 1.13.0]

2019-09-22 Thread Yasuhito FUTATSUKI

On 2019/09/23 6:16, Johan Corveleyn wrote:

On Sat, Sep 21, 2019 at 10:28 AM Yasuhito FUTATSUKI  wrote:


On 2019/09/20 0:44, Daniel Shahaf wrote:

Yasuhito FUTATSUKI wrote on Thu, 19 Sep 2019 06:41 +00:00:



The FAIL on test_merge_peg3 is a newline style issue on reading file
in raw mode, which is introduced by my patch.
(https://svn.apache.org/viewvc/subversion/branches/swig-py3/subversion/bindings/swig/python/tests/client.py?view=annotate#l438)

Rest of those errors and failures are in the tests that were added only
to swig-py, and it seems most of the causes of them are confusion of
paths in platform specific format and Subversion's canoical format,
as the comment in SubversionClientTestCase.test_update4() says.

(I'll try to fix those issues on check-swig-py later, hopefully
next weekend.)


Thanks!  No rush.


Now I re-sync swig-py3 branch with trunk and make a patch addressing
issues on testing on other than Unix/Linux platform.

I've tested on FreeBSD 11.2, with Python 2.7 and Python 3.6.

Could anyone please test it on Windows ?


Tested on Windows 7, and yes, this fixes the previous test failures
with Python 2.7.16. Thanks!

Test output:
[[[
Testing Release configuration on local repository.
-- Running Swig Python tests --
..s.
.
--
Ran 153 tests in 63.214s

OK (skipped=1)
]]]



I've commited it, as r1866362. Thanks!



Building with Python 3.7.4 still fails with the same error though (no
problem, I know your patch wasn't addressing that, just mentioning it
here for completeness). Consequently I haven't been able to run the
swig-python tests with python 3.7 yet.

[[[
c:\python37\include\pytime.h(123): error C4115: 'timeval': named type
definition in parentheses
[C:\research\svn\dev\swig-py3\build\win32\vcnet-vcproj\libsvn_swig_py.vcxproj
]]]


Here is a patch not to treat C4115 as error, globally. If this makes
it possible to build with Python 3.7 on Windows, then next step can be
to limit to apply this relaxation of compile option to files which
contains "#include " (most of them are generated by swig...).

--
Yasuhito FUTATSUKI 
Index: build/generator/templates/vcnet_vcproj.ezt
===
--- build/generator/templates/vcnet_vcproj.ezt  (revision 1867212)
+++ build/generator/templates/vcnet_vcproj.ezt  (working copy)
@@ -60,7 +60,7 @@

DisableSpecificWarnings="4100;4127;4206;4512;4701;4706;4800"
Detect64BitPortabilityProblems="FALSE"
AdditionalOptions="
-   /we4002 /we4003 /we4013 /we4020 /we4022 /we4024 
/we4028 /we4029 /we4030 /we4031 /we4033 /we4047 /we4089 /we4113 /we4115 /we4204 
/we4715"
+   /we4002 /we4003 /we4013 /we4020 /we4022 /we4024 
/we4028 /we4029 /we4030 /we4031 /we4033 /we4047 /we4089 /we4113 /we4204 /we4715"
DebugInformationFormat="3"

ProgramDataBaseFileName="$(IntDir)\[target.output_pdb]"
[if-any 
configs.forced_include_files]ForcedIncludeFiles="[for 
configs.forced_include_files][configs.forced_include_files][if-index 
configs.forced_include_files last][else];[end][end]"
Index: build/generator/templates/vcnet_vcxproj.ezt
===
--- build/generator/templates/vcnet_vcxproj.ezt (revision 1867212)
+++ build/generator/templates/vcnet_vcxproj.ezt (working copy)
@@ -65,7 +65,7 @@
   Level4
   ProgramDatabase
   
4100;4127;4206;4512;4701;4706;4800;%(DisableSpecificWarnings)
-  
4002;4003;4013;4020;4022;4024;4028;4029;4030;4031;4033;4047;4089;4113;4115;4133;4204;4700;4715;4789;%(TreatSpecificWarningsAsErrors)
+  
4002;4003;4013;4020;4022;4024;4028;4029;4030;4031;4033;4047;4089;4113;4133;4204;4700;4715;4789;%(TreatSpecificWarningsAsErrors)
 [if-any configs.forced_include_files]  [for 
configs.forced_include_files][configs.forced_include_files];[end]%(ForcedIncludeFiles)
 [end]