Author: rinrab
Date: Tue Jul  2 19:41:00 2024
New Revision: 1918855

URL: http://svn.apache.org/viewvc?rev=1918855&view=rev
Log:
On the 'cmake' branch: Follow-up to r1918852: Fix msvc_export for RA and
FS modules.

There was a little problem in the previous code that the msvc_export varaible
was initializing for targets with type of TargetLib but not TargetRaModule and
TargetFsModule, which checks were done in theirs elifs before.

* build/generator/gen_cmake.py
  (Generator.write): Fix sequence of `elif` conditions to correctly initialize
   msvc_export variable for RA and FS modules.

Modified:
    subversion/branches/cmake/build/generator/gen_cmake.py

Modified: subversion/branches/cmake/build/generator/gen_cmake.py
URL: 
http://svn.apache.org/viewvc/subversion/branches/cmake/build/generator/gen_cmake.py?rev=1918855&r1=1918854&r2=1918855&view=diff
==============================================================================
--- subversion/branches/cmake/build/generator/gen_cmake.py (original)
+++ subversion/branches/cmake/build/generator/gen_cmake.py Tue Jul  2 19:41:00 
2024
@@ -100,7 +100,6 @@ class Generator(gen_base.GeneratorBase):
       target: gen_base.Target
       group = None
       enable_condition = "TRUE"
-      msvc_export = []
 
       if isinstance(target, gen_base.TargetScript):
         # there is nothing to build
@@ -120,7 +119,9 @@ class Generator(gen_base.GeneratorBase):
         group = "SVN_FS_MODULES"
       elif isinstance(target, gen_base.TargetApacheMod):
         pass
-      elif isinstance(target, gen_base.TargetLib):
+
+      msvc_export = []
+      if isinstance(target, gen_base.TargetLib):
         for export in target.msvc_export:
           path = "subversion/include/" + export.replace("\\", "/")
           msvc_export.append(path)


Reply via email to