Skip to content

Commit

Permalink
Merge pull request #208 from QuTech-Delft/fix-git-hub-actions-for-dev…
Browse files Browse the repository at this point in the history
…elop

Fix GitHub Actions JS and Python macos/x64 jobs
  • Loading branch information
rturrado authored Mar 25, 2024
2 parents 5525091 + 7c79ab0 commit 4a012b9
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 18 deletions.
3 changes: 2 additions & 1 deletion .github/actions/python-tests/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ runs:
- name: Get latest CMake
uses: lukka/get-cmake@latest
- name: Install Python dependencies
run: python -m pip install --upgrade pip conan pytest setuptools wheel
run: |
python -m pip install --upgrade pip conan pytest setuptools wheel
shell: ${{ inputs.shell }}
- name: Build
run: python -m pip install --verbose .
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -135,9 +135,10 @@ jobs:
- name: Install node
uses: actions/setup-node@v4
with:
node-version: 16.20.0
node-version: 16.20.2
- name: Install conan
run: python3 -m pip install --upgrade pip conan
run: |
pipx install conan
shell: bash
- name: Get latest CMake
uses: lukka/get-cmake@latest
Expand Down
3 changes: 3 additions & 0 deletions conan/profiles/debug
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,6 @@ libqasm/*:build_type=Debug
[options]
libqasm/*:asan_enabled=False
libqasm/*:build_tests=False

[replace_requires]
nodejs/*: nodejs/16.20.2
3 changes: 3 additions & 0 deletions conan/profiles/release
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,6 @@ libqasm/*:build_type=Release
[options]
libqasm/*:asan_enabled=False
libqasm/*:build_tests=False

[replace_requires]
nodejs/*: nodejs/16.20.2
15 changes: 0 additions & 15 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,6 @@ def run(self):

['-o']['libqasm/*:build_python=True']
['-o']['libqasm/*:build_tests=True']
# The Python library needs the compatibility headers
['-o']['libqasm/*:compat=True']
['-o']['libqasm/*:cqasm_python_dir=' + re.escape(os.path.dirname(cqasm_target))]
['-o']['libqasm/*:python_dir=' + re.escape(os.path.dirname(target))]
['-o']['libqasm/*:python_ext=' + re.escape(os.path.basename(target))]
Expand Down Expand Up @@ -144,12 +142,6 @@ def run(self):
if platform.system() == "Darwin":
os.environ['MACOSX_DEPLOYMENT_TARGET'] = '10.10'
_bdist_wheel.run(self)
impl_tag, abi_tag, plat_tag = self.get_tag()
archive_basename = "{}-{}-{}-{}".format(self.wheel_dist_name, impl_tag, abi_tag, plat_tag)
wheel_path = os.path.join(self.dist_dir, archive_basename + '.whl')
if platform.system() == "Darwin":
from delocate.delocating import delocate_wheel
delocate_wheel(wheel_path)


class sdist(_sdist):
Expand All @@ -175,17 +167,10 @@ def initialize_options(self):

classifiers=[
'License :: OSI Approved :: Apache Software License',

'Operating System :: POSIX :: Linux',
'Operating System :: MacOS',
'Operating System :: Microsoft :: Windows',

'Programming Language :: Python :: 3 :: Only',
'Programming Language :: Python :: 3.5',
'Programming Language :: Python :: 3.6',
'Programming Language :: Python :: 3.7',
'Programming Language :: Python :: 3.8',

'Topic :: Scientific/Engineering'
],

Expand Down

0 comments on commit 4a012b9

Please sign in to comment.