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

Update flake, fix pyrad build #42

Merged
merged 1 commit into from
Jan 25, 2025
Merged

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented Jan 25, 2025

The build fails for me with

  File "/nix/store/3wbw03q2z5d7ys1pzp30rmzn6qcxnyrp-python3.12-poetry-core-2.0.0/lib/python3.12/site-packages/poetry/core/masonry/metadata.py", line 112, in from_package
    if name == "repository" and url == package.urls["Repository"]:
                                       ~~~~~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'Repository'
error: subprocess-exited-with-error

Making the repository key in the urls section of pyproject.toml fixes the build.

There's a pending upstream PR doing the same: pyradius/pyrad#209

The build fails for me with

      File "/nix/store/3wbw03q2z5d7ys1pzp30rmzn6qcxnyrp-python3.12-poetry-core-2.0.0/lib/python3.12/site-packages/poetry/core/masonry/metadata.py", line 112, in from_package
        if name == "repository" and url == package.urls["Repository"]:
                                           ~~~~~~~~~~~~^^^^^^^^^^^^^^
    KeyError: 'Repository'
    error: subprocess-exited-with-error

Making the `repository` key in the `urls` section of `pyproject.toml`
fixes the build.

There's a pending upstream PR doing the same: pyradius/pyrad#209
Copy link
Member

@WilliButz WilliButz left a comment

Choose a reason for hiding this comment

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

Thank you! :)

@WilliButz WilliButz merged commit 5470f39 into nix-community:main Jan 25, 2025
1 check passed
@Ma27 Ma27 deleted the flake-update branch January 25, 2025 14:05
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