Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rye uv #47

Closed
wants to merge 7 commits into from
Closed

rye uv #47

wants to merge 7 commits into from

Conversation

voluntas
Copy link
Member

No description provided.

@@ -122,7 +122,10 @@ jobs:
with:
python-version: "${{ matrix.python_version }}"
cache: "pip"
- run: pip install -r requirements-dev.lock
- name: Remove Editable Install
run: (Get-Content -Path "requirements-dev.lock") -notmatch "-e" | Set-Content -Path "requirements-dev.tmp"
Copy link
Contributor

Choose a reason for hiding this comment

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

ここで、requirements-dev.lock ファイルにある -e file:. の行を削除した requirements を生成しています。

Copy link
Member Author

Choose a reason for hiding this comment

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

@zztkm もうちょい説明ほしい

Copy link
Contributor

@zztkm zztkm Feb 16, 2024

Choose a reason for hiding this comment

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

@voluntas
short
editable install 時にエラー(No module named 'wheel')が発生していましたが、そもそも build 時にeditable install は不要であると判断したので、-e file:. を削除し editable install 自体を回避するようにしました。

long
今回エラーになっていた原因-e file:. の pip install 時に build editable のステップが走って、そこで ModuleNotFoundError: No module named 'wheel' が発生していました。

これを回避するという視点で考えたときに2個回避方法があって

  • 事前に pip install wheel する
  • build するときに editable install する必要はないので、この行を削除してそもそも build editable されないようにする

がありました。

この Actions の目的はビルドだと思ったので、今回は後者を選びました。

Copy link
Member Author

Choose a reason for hiding this comment

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

対応内容がハックになっているので、
そもそも rye uv 使うとなぜ発生するのか、rye uv のバージョンを上がるのを待つべきなのかが知りたい。

ハックが不要になるにはどうするべきなのか、rye / uv 側に PR を出して解決するの?

Copy link
Contributor

Choose a reason for hiding this comment

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

追加調査して、原因の深掘りをした上で改めて対応方法を考えてみます。

@voluntas
Copy link
Member Author

develop で rye uv で問題無くビルドできることがわかったので閉じる。

@voluntas voluntas closed this Feb 19, 2024
@voluntas voluntas deleted the feature/rye-uv branch February 19, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants