Skip to content

Comments

poetry: 1.4.2 -> 1.5.1#233393

Merged
K900 merged 4 commits intoNixOS:masterfrom
dotlambda:poetry-1.5.0
Jul 9, 2023
Merged

poetry: 1.4.2 -> 1.5.1#233393
K900 merged 4 commits intoNixOS:masterfrom
dotlambda:poetry-1.5.0

Conversation

@dotlambda
Copy link
Member

@dotlambda dotlambda commented May 22, 2023

Description of changes

Diff: python-poetry/poetry@refs/tags/1.4.2...1.5.1

Changelog: https://github.com/python-poetry/poetry/blob/refs/tags/1.5.1/CHANGELOG.md

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

closes #241458

@dotlambda dotlambda requested a review from K900 May 22, 2023 11:36
@ofborg ofborg bot requested a review from jbaum98 May 22, 2023 12:01
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels May 22, 2023
Copy link
Contributor

@Dessix Dessix left a comment

Choose a reason for hiding this comment

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

Result of nixpkgs-review pr 233393 run on x86_64-linux 1

2 packages failed to build:
  • poetryPlugins.poetry-plugin-up
  • poetryPlugins.poetry-plugin-up.dist
4 packages built:
  • poetry
  • poetry.dist
  • poetryPlugins.poetry-audit-plugin
  • poetryPlugins.poetry-audit-plugin.dist
Reviewed points
  • changes are backward compatible
  • removed options are declared with mkRemovedOptionModule
  • changes that are not backward compatible are documented in release notes
  • module tests succeed on ARCHITECTURE
  • options types are appropriate
  • options description is set
  • options example is provided
  • documentation affected by the changes is updated
Possible improvements

Documenting how to invoke with and without plugins from a nixos configuration or flake would be helpful.

Comments

Fails to build on x86_64-linux due to plugin dependency build failures.

Failures are, ironically, due to AttributeError: 'PyProjectTOML' object has no attribute 'file' during the test_e2e.py portion of the pytestCheckPhase, rather than failure to actually build the plugin. May be simple incompatibility in toml?

Diagnosis may be complicated by apparent failure building (optional) pandas dependency for python-3.10 in latest nixpkgs commits.

@natsukium
Copy link
Member

The patch, #220252, seems to cause the x86_64-darwin build to fail.
However, without the patch, I got the same problem again.

@considerate
Copy link
Member

I encountered a build failure the first time I built this on x86_64-linux but running the build again I wasn't able to reproduce the failure and instead I got a working version of poetry.

@Dessix
Copy link
Contributor

Dessix commented Jun 12, 2023

error: attribute 'fetchPypi' missing

       at /home/dessix/.cache/nixpkgs-review/pr-233393-5/nixpkgs/pkgs/tools/package-management/poetry/default.nix:14:15:

           13|         version = "3.12.0";
           14|         src = self.fetchPypi {
             |               ^

Looks like we've got a new type of failure on the latest nixpkgs caused by #234043.

@Dessix Dessix mentioned this pull request Jul 4, 2023
12 tasks
Dessix added a commit to Dessix/nixpkgs that referenced this pull request Jul 6, 2023
@github-actions github-actions bot added the 6.topic: python Python is a high-level, general-purpose programming language. label Jul 9, 2023
@dotlambda dotlambda changed the title poetry: 1.4.2 -> 1.5.0 poetry: 1.4.2 -> 1.5.1 Jul 9, 2023
@dotlambda dotlambda force-pushed the poetry-1.5.0 branch 2 times, most recently from 398e5b2 to 69415b9 Compare July 9, 2023 03:18
@ofborg ofborg bot added 8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package labels Jul 9, 2023
@ofborg ofborg bot requested a review from mweinelt July 9, 2023 03:36
@dotlambda dotlambda requested a review from Dessix July 9, 2023 03:47
Copy link
Contributor

@Dessix Dessix left a comment

Choose a reason for hiding this comment

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

Looks great. Let's merge it and close #241458.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: python Python is a high-level, general-purpose programming language. 8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants