Skip to content

[py-thinc] fixed checksum#18571

Merged
adamjstewart merged 1 commit intospack:developfrom
qwertos:qwertos/py-thinc
Sep 9, 2020
Merged

[py-thinc] fixed checksum#18571
adamjstewart merged 1 commit intospack:developfrom
qwertos:qwertos/py-thinc

Conversation

@qwertos
Copy link
Copy Markdown
Contributor

@qwertos qwertos commented Sep 8, 2020

No description provided.

url = "https://pypi.io/packages/source/t/thinc/thinc-7.4.1.tar.gz"

version('7.4.1', sha256='523e9be1bfaa3ed1d03d406ce451b6b4793a9719d5b83d2ea6b3398b96bc58b8')
version('7.4.1', sha256='0139fa84dc9b8d88af15e648fc4ae13d899b8b5e49cb26a8f4a0604ee9ad8a9e')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any idea where the previous checksum came from? As far as I know, it's not possible for PyPI checksums to change. Can you contact the developers?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm guessing the original hash was a typo since the hash for 7.4.1 and 7.4.0 were originally the same? I'll poke @aweits

Copy link
Copy Markdown
Contributor

@aweits aweits Sep 9, 2020

Choose a reason for hiding this comment

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

yep, cut-n-paste typo, still trying to backtrack to how this got through a build...

@adamjstewart adamjstewart added the checksum Tarball checksum mismatches. label Sep 9, 2020
@qwertos qwertos requested a review from adamjstewart September 9, 2020 16:18
@adamjstewart adamjstewart merged commit f7d1f84 into spack:develop Sep 9, 2020
@qwertos qwertos deleted the qwertos/py-thinc branch October 23, 2020 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

checksum Tarball checksum mismatches.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants