Skip to content

python3Packages.stac-validator: fix pyproject name - #547349

Open
imincik wants to merge 1 commit into
NixOS:masterfrom
imincik:stac-validator-fix-pyproject-name
Open

python3Packages.stac-validator: fix pyproject name#547349
imincik wants to merge 1 commit into
NixOS:masterfrom
imincik:stac-validator-fix-pyproject-name

Conversation

@imincik

@imincik imincik commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Things done

We could also update Nix pname to stac_valid to match pyproject name,
but stac_validator looks more appropriate/correct.

This PR is needed to unbreak the package.

We could also update Nix pname to `stac_valid` to match pyproject name,
but `stac_validator` looks more appropriate/correct.
@imincik
imincik requested review from autra, l0b0 and sikmir July 30, 2026 08:16
@nixpkgs-ci
nixpkgs-ci Bot requested a review from a team July 30, 2026 08:21
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 6.topic: python Python is a high-level, general-purpose programming language. labels Jul 30, 2026
@l0b0

l0b0 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Shouldn't this be upstreamed instead?

@imincik

imincik commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Shouldn't this be upstreamed instead?

Yes, we should submit patch upstream as well, but this PR unbreaks the package in the meanwhile.

@imincik

imincik commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Upstream PR - StacLabs/stac-validator#305

@imincik

imincik commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review-gha (d38b08f)

Command: nixpkgs-review pr 547349
Commit: 63d365c462a00f3a4b0da2fe31ac61ad2720dadf (subsequent changes)
Merge: 4354a1a353be52e44580a9a6492e7d4e32bded4f

Logs: https://github.com/imincik/nixpkgs-review-gha/actions/runs/30530434115/attempts/1

Download packages from cache:
  • x86_64-linux
    nix-store -r \
      --option binary-caches 'https://cache.nixos.org/ https://imincik-nixpkgs-review-gha.cachix.org' \
      --option trusted-public-keys '
        cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
        imincik-nixpkgs-review-gha.cachix.org-1:vbkvABMrxmQ+ObRDwmF2qaLIEhGHdbF4MwXtOPU8Jmk=
      ' \
      /nix/store/7g0jr8302b64micr2jy0p945xa32nxn5-python3.13-stac-validator-4.4.0 \
      /nix/store/0kibrr02bcdsp51xlx5hfaxwsy2d3wah-python3.13-stac-validator-4.4.0-dist \
      /nix/store/zbwra4vrr753c4ik32alls5zj49b60c0-python3.14-stac-validator-4.4.0-dist \
      /nix/store/kw32wd1y44591pfbjl640ix1p51zzxla-python3.14-stac-validator-4.4.0 \
      /nix/store/qd4i27lmjm9mwj7h0rd2ixvp6rmy7jvb-python3.14-stac-validator-4.4.0-dist \
      /nix/store/mwr0xcfra72s7rb4hyfkj0s0sm83ma36-python3.14-stac-validator-4.4.0
  • aarch64-linux
    nix-store -r \
      --option binary-caches 'https://cache.nixos.org/ https://imincik-nixpkgs-review-gha.cachix.org' \
      --option trusted-public-keys '
        cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
        imincik-nixpkgs-review-gha.cachix.org-1:vbkvABMrxmQ+ObRDwmF2qaLIEhGHdbF4MwXtOPU8Jmk=
      ' \
      /nix/store/gfd38hnr5cwqw1sda0ms7laxzd3ficzh-python3.14-stac-validator-4.4.0-dist \
      /nix/store/7i4y56pz427z64y1rmjlkm77wzyrhjaw-python3.14-stac-validator-4.4.0-dist \
      /nix/store/l0262jj3nrw46cfas6y4z1f34704n6gc-python3.13-stac-validator-4.4.0 \
      /nix/store/4qpz59vr1c3nqbxzpfa666h313i6b7d4-python3.14-stac-validator-4.4.0 \
      /nix/store/w625ldbha1w9z9r9dka6ym2vw10dgkw4-python3.14-stac-validator-4.4.0 \
      /nix/store/9ps8a2gxp4jhhzv74s5d58q3z7y4a21c-python3.13-stac-validator-4.4.0-dist
  • aarch64-darwin
    nix-store -r \
      --option binary-caches 'https://cache.nixos.org/ https://imincik-nixpkgs-review-gha.cachix.org' \
      --option trusted-public-keys '
        cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
        imincik-nixpkgs-review-gha.cachix.org-1:vbkvABMrxmQ+ObRDwmF2qaLIEhGHdbF4MwXtOPU8Jmk=
      ' \
      /nix/store/m4qphjcl2cs0wxrhvayss3hq78q2rdyj-python3.14-stac-validator-4.4.0 \
      /nix/store/imxc6j3lyfs5dqamizyc76404cf2bniv-python3.14-stac-validator-4.4.0-dist \
      /nix/store/f66l3swzxqp86ib11v6spshbs2k9ri6y-python3.13-stac-validator-4.4.0 \
      /nix/store/1jjanqvfj9zhgdx513qwhm864ifk047z-python3.14-stac-validator-4.4.0-dist \
      /nix/store/91j3hgq90adk2csw2r91cgx5fqib2mdf-python3.14-stac-validator-4.4.0 \
      /nix/store/6xm2b76csmxmpx7lcs655gsg6falvn0i-python3.13-stac-validator-4.4.0-dist

x86_64-linux

✅ 3 packages built:
  • python313Packages.stac-validator
  • python314Packages.stac-validator
  • stac-validator

aarch64-linux

✅ 3 packages built:
  • python313Packages.stac-validator
  • python314Packages.stac-validator
  • stac-validator

x86_64-darwin

No rebuilds


aarch64-darwin (sandbox = relaxed)

✅ 3 packages built:
  • python313Packages.stac-validator
  • python314Packages.stac-validator
  • stac-validator

@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Jul 30, 2026
@autra

autra commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

We could also update Nix pname to stac_valid to match pyproject name,

I believe this will be easier for upstream: if they change their package name in pyproject, they'll have to change their name in pypi too, right?

@imincik

imincik commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

It looks like upstream changed the pyproject name from stac_validator to stac_valid recently. Looking at the commit message, I am not really sure if this change was intentional.

@imincik

imincik commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

pypi is also showing a warning message.

@Sigmanificient

Copy link
Copy Markdown
Member

I think we should move the package to correspond to pname :p

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. 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. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants