From 83bacfdf1692839dcbda8d785f305c2c122fbc36 Mon Sep 17 00:00:00 2001
From: Sergey Yakubov <sergey.yakubov@desy.de>
Date: Tue, 20 Aug 2019 13:11:35 +0200
Subject: [PATCH] fix paths

---
 producer/api/python/CMakeLists_Windows.cmake     |  6 +++---
 .../python/binary_dist_windows/CMakeLists.txt    | 12 ++++++------
 .../api/python/binary_dist_windows/setup.py.in   |  6 +++---
 .../api/python/source_dist_linux/CMakeLists.txt  | 16 +++++++---------
 .../api/python/source_dist_linux/setup.py.in     |  9 ++++-----
 worker/api/python/asapo_worker.pyx.in            |  2 +-
 .../python/binary_dist_windows/CMakeLists.txt    | 12 ++++++------
 .../api/python/source_dist_linux/CMakeLists.txt  |  8 ++++----
 worker/api/python/source_dist_linux/setup.py.in  |  3 +--
 9 files changed, 35 insertions(+), 39 deletions(-)

diff --git a/producer/api/python/CMakeLists_Windows.cmake b/producer/api/python/CMakeLists_Windows.cmake
index 95dd07a0d..c9b58cda8 100644
--- a/producer/api/python/CMakeLists_Windows.cmake
+++ b/producer/api/python/CMakeLists_Windows.cmake
@@ -7,15 +7,15 @@ message ("   Python includes:" ${Python3_INCLUDE_DIRS})
 message ("   Numpy:" ${PYTHON_NUMPY_INCLUDE_DIR})
 
 
-add_custom_command(OUTPUT asapo_worker.cpp
+add_custom_command(OUTPUT asapo_producer.cpp
         COMMAND ${Python3_EXECUTABLE} cythonize.py
         DEPENDS asapo-producer)
 
 
-set(TARGET_NAME asapo_worker)
+set(TARGET_NAME asapo_producer)
 
 set(SOURCE_FILES
-        asapo_worker.cpp)
+        asapo_producer.cpp)
 
 add_library(${TARGET_NAME} SHARED ${SOURCE_FILES})
 set_target_properties(${TARGET_NAME} PROPERTIES SUFFIX ".pyd")
diff --git a/producer/api/python/binary_dist_windows/CMakeLists.txt b/producer/api/python/binary_dist_windows/CMakeLists.txt
index ac46814f2..120f157ae 100644
--- a/producer/api/python/binary_dist_windows/CMakeLists.txt
+++ b/producer/api/python/binary_dist_windows/CMakeLists.txt
@@ -1,16 +1,16 @@
 configure_file(setup.py.in setup.py @ONLY)
 
-ADD_CUSTOM_TARGET(windows-bdist ALL
+ADD_CUSTOM_TARGET(windows-bdist-producer ALL
         COMMAND ${Python3_EXECUTABLE} setup.py bdist_wheel
         )
 
 
-ADD_CUSTOM_TARGET(copy_python_bdist ALL
-        COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_CURRENT_BINARY_DIR}/asapo_worker
-        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo_worker> ${CMAKE_CURRENT_BINARY_DIR}/asapo_worker/
+ADD_CUSTOM_TARGET(copy_python_bdist-producer ALL
+        COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_CURRENT_BINARY_DIR}/asapo_producer
+        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo_producer> ${CMAKE_CURRENT_BINARY_DIR}/asapo_producer/
         )
 
 
-ADD_DEPENDENCIES(copy_python_bdist asapo_worker)
+ADD_DEPENDENCIES(copy_python_bdist-producer asapo_producer)
 
-ADD_DEPENDENCIES(windows-bdist copy_python_bdist)
+ADD_DEPENDENCIES(windows-bdist-producer copy_python_bdist-producer)
diff --git a/producer/api/python/binary_dist_windows/setup.py.in b/producer/api/python/binary_dist_windows/setup.py.in
index cc896317e..1053f8e94 100644
--- a/producer/api/python/binary_dist_windows/setup.py.in
+++ b/producer/api/python/binary_dist_windows/setup.py.in
@@ -1,12 +1,12 @@
 import setuptools
 
 setuptools.setup(
-      name ="asapo_worker",
+      name ="asapo_producer",
       version = "@ASAPO_VERSION_PYTHON@",
       install_requires=['numpy'],
       include_package_data=True,
-      packages=['asapo_worker'],
+      packages=['asapo_producer'],
       package_data={
-        'asapo_worker': ['asapo_worker.pyd'],
+        'asapo_producer': ['asapo_producer.pyd'],
       },
 )
diff --git a/producer/api/python/source_dist_linux/CMakeLists.txt b/producer/api/python/source_dist_linux/CMakeLists.txt
index 48fe09828..2b1f56c78 100644
--- a/producer/api/python/source_dist_linux/CMakeLists.txt
+++ b/producer/api/python/source_dist_linux/CMakeLists.txt
@@ -2,25 +2,23 @@ configure_file(setup.py.in setup.py @ONLY)
 configure_file(MANIFEST.in MANIFEST.in @ONLY)
 
 
-ADD_CUSTOM_TARGET(python-dist ALL
+ADD_CUSTOM_TARGET(python-dist-producer ALL
         COMMAND python setup.py sdist
         )
 
-#get_property(ASAPO_WORKER_LIB TARGET asapo-worker PROPERTY LOCATION)
-
-ADD_CUSTOM_TARGET(copy_python_dist ALL
+ADD_CUSTOM_TARGET(copy_python_dist-producer ALL
         COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/../asapo_wrappers.h ${CMAKE_CURRENT_BINARY_DIR}/.
-        COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/../asapo_worker.cpp ${CMAKE_CURRENT_BINARY_DIR}/.
-        COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_SOURCE_DIR}/worker/api/cpp/include ${CMAKE_CURRENT_BINARY_DIR}/include
+        COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/../asapo_producer.cpp ${CMAKE_CURRENT_BINARY_DIR}/.
+        COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_SOURCE_DIR}/producer/api/cpp/include ${CMAKE_CURRENT_BINARY_DIR}/include
         COMMAND ${CMAKE_COMMAND} -E copy_directory ${ASAPO_CXX_COMMON_INCLUDE_DIR}/common ${CMAKE_CURRENT_BINARY_DIR}/include/common
         COMMAND ${CMAKE_COMMAND} -E remove ${CMAKE_CURRENT_BINARY_DIR}/include/common/version.h.in
         COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_CURRENT_BINARY_DIR}/lib
         COMMAND ${CMAKE_COMMAND} -E copy ${CURL_LIBRARIES} ${CMAKE_CURRENT_BINARY_DIR}/lib
-        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo-worker> ${CMAKE_CURRENT_BINARY_DIR}/lib
+        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo-producer> ${CMAKE_CURRENT_BINARY_DIR}/lib
 
         )
 
 
-ADD_DEPENDENCIES(copy_python_dist python-lib3)
+ADD_DEPENDENCIES(copy_python_dist-producer python-lib3-producer)
 
-ADD_DEPENDENCIES(python-dist copy_python_dist)
+ADD_DEPENDENCIES(python-dist-producer copy_python_dist-producer)
diff --git a/producer/api/python/source_dist_linux/setup.py.in b/producer/api/python/source_dist_linux/setup.py.in
index 9fc10832c..8d8509b7a 100644
--- a/producer/api/python/source_dist_linux/setup.py.in
+++ b/producer/api/python/source_dist_linux/setup.py.in
@@ -3,8 +3,8 @@ from distutils.core import Extension
 import numpy
 
 ext_modules = [
-    Extension("asapo_worker", ["asapo_worker.cpp"],
-              extra_objects=['lib/libasapo-worker.a'],
+    Extension("asapo_producer", ["asapo_producer.cpp"],
+              extra_objects=['lib/libasapo-producer.a'],
 					   libraries = ['curl'],
                        include_dirs=["include/common","include",numpy.get_include()],
                        extra_compile_args=@EXTRA_COMPILE_ARGS@,
@@ -13,10 +13,9 @@ ext_modules = [
 ]
 
 setup(
-      name ="asapo_worker",
+      name ="asapo_producer",
       ext_modules = ext_modules,
       version = "@ASAPO_VERSION_PYTHON@",
       setup_requires=["numpy"],
-      install_requires=["numpy"],
-      platforms=["aaa"]
+      install_requires=["numpy"]
       )
diff --git a/worker/api/python/asapo_worker.pyx.in b/worker/api/python/asapo_worker.pyx.in
index a8af17551..9a76b93ed 100644
--- a/worker/api/python/asapo_worker.pyx.in
+++ b/worker/api/python/asapo_worker.pyx.in
@@ -34,8 +34,8 @@ cdef class PyDataBroker:
     def _op(self, op, group_id, meta_only, uint64_t id):
         cdef FileInfo info
         cdef string b_group_id = _bytes(group_id)
-        cdef FileData* p_data =  <FileData*>NULL if meta_only else &data
         cdef FileData data
+        cdef FileData* p_data =  <FileData*>NULL if meta_only else &data
         cdef Error err
         cdef np.npy_intp dims[1]
         if op == "next":
diff --git a/worker/api/python/binary_dist_windows/CMakeLists.txt b/worker/api/python/binary_dist_windows/CMakeLists.txt
index de6a95e2c..ac46814f2 100644
--- a/worker/api/python/binary_dist_windows/CMakeLists.txt
+++ b/worker/api/python/binary_dist_windows/CMakeLists.txt
@@ -1,16 +1,16 @@
 configure_file(setup.py.in setup.py @ONLY)
 
-ADD_CUSTOM_TARGET(windows-bdist-producer ALL
+ADD_CUSTOM_TARGET(windows-bdist ALL
         COMMAND ${Python3_EXECUTABLE} setup.py bdist_wheel
         )
 
 
-ADD_CUSTOM_TARGET(copy_python_bdist_producer ALL
-        COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_CURRENT_BINARY_DIR}/asapo_producer
-        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo_producer> ${CMAKE_CURRENT_BINARY_DIR}/asapo_producer/
+ADD_CUSTOM_TARGET(copy_python_bdist ALL
+        COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_CURRENT_BINARY_DIR}/asapo_worker
+        COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:asapo_worker> ${CMAKE_CURRENT_BINARY_DIR}/asapo_worker/
         )
 
 
-ADD_DEPENDENCIES(copy_python_bdist_producer asapo_producer)
+ADD_DEPENDENCIES(copy_python_bdist asapo_worker)
 
-ADD_DEPENDENCIES(windows-bdist-producer copy_python_bdist_producer)
+ADD_DEPENDENCIES(windows-bdist copy_python_bdist)
diff --git a/worker/api/python/source_dist_linux/CMakeLists.txt b/worker/api/python/source_dist_linux/CMakeLists.txt
index 83b420eee..48fe09828 100644
--- a/worker/api/python/source_dist_linux/CMakeLists.txt
+++ b/worker/api/python/source_dist_linux/CMakeLists.txt
@@ -2,13 +2,13 @@ configure_file(setup.py.in setup.py @ONLY)
 configure_file(MANIFEST.in MANIFEST.in @ONLY)
 
 
-ADD_CUSTOM_TARGET(python-dist-producer ALL
+ADD_CUSTOM_TARGET(python-dist ALL
         COMMAND python setup.py sdist
         )
 
 #get_property(ASAPO_WORKER_LIB TARGET asapo-worker PROPERTY LOCATION)
 
-ADD_CUSTOM_TARGET(copy_python_dist_producer ALL
+ADD_CUSTOM_TARGET(copy_python_dist ALL
         COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/../asapo_wrappers.h ${CMAKE_CURRENT_BINARY_DIR}/.
         COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_BINARY_DIR}/../asapo_worker.cpp ${CMAKE_CURRENT_BINARY_DIR}/.
         COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_SOURCE_DIR}/worker/api/cpp/include ${CMAKE_CURRENT_BINARY_DIR}/include
@@ -21,6 +21,6 @@ ADD_CUSTOM_TARGET(copy_python_dist_producer ALL
         )
 
 
-ADD_DEPENDENCIES(copy_python_dist_producer python-lib3-producer)
+ADD_DEPENDENCIES(copy_python_dist python-lib3)
 
-ADD_DEPENDENCIES(python-dist-producer copy_python_dist_producer)
+ADD_DEPENDENCIES(python-dist copy_python_dist)
diff --git a/worker/api/python/source_dist_linux/setup.py.in b/worker/api/python/source_dist_linux/setup.py.in
index 9fc10832c..d4df8d1c8 100644
--- a/worker/api/python/source_dist_linux/setup.py.in
+++ b/worker/api/python/source_dist_linux/setup.py.in
@@ -17,6 +17,5 @@ setup(
       ext_modules = ext_modules,
       version = "@ASAPO_VERSION_PYTHON@",
       setup_requires=["numpy"],
-      install_requires=["numpy"],
-      platforms=["aaa"]
+      install_requires=["numpy"]
       )
-- 
GitLab