fix pypi_test

pinpoint-apm · Oct 20, 2023 · 02b8c7d · 02b8c7d
1 parent 4a7ee75
commit 02b8c7d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/.github/workflows/build-wheels.yml b/.github/workflows/build-wheels.yml
@@ -6,17 +6,19 @@ on:
       release-testpypi:
         description: Do you want to release to test-pypi
         required: true
-        default: "true"
+        default: true
+        type: boolean
       release:
         description: Do you want to release to pypi
         required: true
-        default: "false"
+        default: false
+        type: boolean
 
 jobs:
   build_wheels_pypitest:
-    if: ${{ github.event.inputs.release-testpypi }} == "true"
     name: Build wheels on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
+    if: ${{ inputs.release-testpypi }}
     strategy:
       matrix:
         os: [ubuntu-20.04, macos-latest]
@@ -82,9 +84,9 @@ jobs:
           repository_url: https://test.pypi.org/legacy/          
 
   build_wheels:
-    if: ${{ github.event.inputs.release }} == "true"
     name: Build wheels on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
+    if: ${{ inputs.release }}
     strategy:
       matrix:
         os: [ubuntu-20.04, macos-latest]

diff --git a/setup_pypi_test.py b/setup_pypi_test.py
@@ -19,7 +19,7 @@
 ###############################################
 
 extFiles = [
-    'src/PY/_pinpoint_py.c',
+    'src/PY/_pinpoint_py.cpp',
 ]
 
 # add pinpoint-common
@@ -43,11 +43,11 @@
                     extFiles,
                     include_dirs=['common/include',
                                   'common/jsoncpp/include', 'common/src'],
-                    libraries=agent_libraries
+                    libraries=agent_libraries,
+                    extra_compile_args=['-std=c++11']
                     )
       ],
       package_dir={'': 'plugins/PY'},
-      #   packages=find_packages('plugins/PY'),
       packages=find_namespace_packages(
           'plugins/PY', include=['pinpointPy.*', 'pinpointPy']),
       )