Skip to content

Python libs: update linking#558

Merged
MaxRayskiy merged 9 commits intomasterfrom
macos-pip
Oct 11, 2022
Merged

Python libs: update linking#558
MaxRayskiy merged 9 commits intomasterfrom
macos-pip

Conversation

@MaxRayskiy
Copy link
Contributor

No description provided.

@MaxRayskiy MaxRayskiy added the full-ci run all steps label Sep 29, 2022
@MaxRayskiy MaxRayskiy changed the title fix python depend Python libs: update linking Sep 29, 2022

- name: Create and fix Wheel
if: ${{ inputs.UPLOAD_ARTIFACTS && matrix.config == 'Release' }}
if: ${{ matrix.config == 'Release' }}
Copy link
Contributor Author

@MaxRayskiy MaxRayskiy Oct 11, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

debug. remove the step

- name: Upload Wheel to Artifacts
if: ${{false}}
#if: ${{ inputs.UPLOAD_ARTIFACTS && matrix.config == 'Release' }}
if: ${{ matrix.config == 'Release' }}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also remove

@MaxRayskiy MaxRayskiy requested a review from Grantim October 11, 2022 14:37
CMakeLists.txt Outdated
include_directories(${PYBIND11_DIR})
ENDIF()

message("=============${PYTHON_VERSION}")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rm

macos-build-test:
needs: [ versioning-and-release-url ]
uses: MeshInspector/MeshLib/.github/workflows/build-test-macos.yml@master
uses: MeshInspector/MeshLib/.github/workflows/build-test-macos.yml@macos-pip
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reset on merge?

@MaxRayskiy MaxRayskiy merged commit a278520 into master Oct 11, 2022
@MaxRayskiy MaxRayskiy deleted the macos-pip branch October 11, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci run all steps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants