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.

Sign up for GitHub

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

Fix certain hash miss matches in case of non weel installations #7594

Merged
merged 16 commits into from
Mar 7, 2023
Merged

Fix certain hash miss matches in case of non weel installations #7594

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5b16309
Adjusted Chef.prepare() to allways write to a temporary directory. Ot…
quantum-byte Mar 2, 2023
cb35709
Fixed formatting and linting issues
quantum-byte Mar 2, 2023
cead0bb
First solution with broken test
quantum-byte Mar 4, 2023
65c36b2
Fix the test by also providing the non_wheel archive hash
quantum-byte Mar 4, 2023
6143b4e
remove validate flag for now
quantum-byte Mar 4, 2023
87181dc
Implement review suggestions to also make add the hash for cached non…
quantum-byte Mar 5, 2023
bca4289
make mypy happy and improve some code comments
quantum-byte Mar 5, 2023
5bc6c03
Fix test_executor_should_write_pep610_url_references_for_non_wheel_ur…
quantum-byte Mar 6, 2023
e672978
make flake happy and removed some print debug statements :)
quantum-byte Mar 6, 2023
9029971
make flake really happy
quantum-byte Mar 6, 2023
62d92ba
Code review improvements
quantum-byte Mar 6, 2023
ede8dbb
Further code review improvements and minor test improvements (cleanup…
quantum-byte Mar 6, 2023
1d844c0
Improved tests further
quantum-byte Mar 6, 2023
1772a6a
refactor test mocking logic
quantum-byte Mar 7, 2023
41e2727
Additional Review improvements
quantum-byte Mar 7, 2023
1e1c331
coding style
radoering Mar 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix the test by also providing the non_wheel archive hash
  • Loading branch information
@quantum-byte
quantum-byte committed Mar 5, 2023
commit 65c36b20c8cb56176da541bcfb0da1cd744686f0
7 changes: 1 addition & 6 deletions src/poetry/installation/executor.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -716,7 +716,7 @@ def _download_link(self, operation: Install | Update, link: Link) -> Path:
# validate the original (downloaded or cached) archive only if we can. e.g. if
# we use the cached wheel or the actual downloaded archive
if package.files and link.filename == archive.name:
self._validate_archive_hash(archive, package)
self._populate_hashes_dict(archive, package)

if archive.suffix != ".whl":
message = (
Expand All @@ -727,11 +727,6 @@ def _download_link(self, operation: Install | Update, link: Link) -> Path:

archive = self._chef.prepare(archive, output_dir=output_dir)

elif link.is_wheel:
self._populate_hashes_dict(
archive, package, validate=False
) # already validated

return archive

def _populate_hashes_dict(
Expand Down