Skip to content

Commit

Permalink
Update how PL is installed (#7005)
Browse files Browse the repository at this point in the history
**Context:**
setuptools has had a release recently that normalizes package names:

References:
- pypa/setuptools#4766
-
https://packaging.python.org/en/latest/specifications/name-normalization/#name-normalization


**Description of the Change:**
Due setuptools 75.8.0 and 75.8.1 outputting completely different package
names, we cannot support both, so CI now "pins" to >=75.8.1.

**Benefits:**
Fix CI failures.

**Possible Drawbacks:**
We may need to update docs.

This is also another situation where moving away from setup.py to
pyproject would yield benefits. As then you can specify the minimum
setuptools version as part of the build-backend configuration.

**Related GitHub Issues:**
None.
[sc-85195](https://app.shortcut.com/xanaduai/story/85195/investigate-issue-with-pennylane-ci-failing-with-uv-build-outputting-package-that-doesn-t-match-name-in-setup-py)
  • Loading branch information
rashidnhm authored Feb 25, 2025
1 parent 29688ee commit af65460
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/unit-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ jobs:
done <<< "$ADDITIONAL_OS_PACKAGES"
- name: Upgrade PIP and install wheel
run: uv pip install --upgrade pip wheel
run: uv pip install --upgrade pip wheel setuptools>=75.8.1

- name: Install PennyLane dependencies
run: |
Expand All @@ -158,7 +158,7 @@ jobs:
shell: bash
run: |
uv build --wheel
uv pip install dist/PennyLane*.whl
uv pip install dist/pennylane*.whl
- name: Install additional PIP packages (Post PennyLane Install)
shell: bash
Expand Down

0 comments on commit af65460

Please sign in to comment.