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

fix get_uris() bug introduced at #91 #92

Merged

Conversation

varhub
Copy link
Contributor

@varhub varhub commented Jan 29, 2016

fix get_uris() bug introduced at #91

Rationale:
Assume that file directory is equal to filename is a misconcept
Checksum was getting bad values and this broke aria2
result: hang forever at "Working... this may take a while."

Patch:
Use package naming convection to extract version from package
name. This patch relies on format: .deb

solves introduced bug at #91
add initial support for filesize

Rationale:
Assume that file directory is equal to filename is a misconcept
Checksum was getting bad values and this broke aria2
result: hang forever at "Working... this may take a while."

Patch:
Use package naming convection to extract version from package
name. This patch relies on format: <name>_<version>_<arch>.deb

solves introduced bug at ilikenwf#91
add initial support for filesize
@varhub
Copy link
Contributor Author

varhub commented Jan 29, 2016

Please @ilikenwf, merge it as soon as possible because I broke upstream.

Eventually, a discussion for verify how md5 is being patched or move to a downloader with sha256 is mandatory. Alternatively, ask canonical to serve an apt-get --print-uris option that forces md5 is another way to avoid above workaround.

ilikenwf pushed a commit that referenced this pull request Jan 29, 2016
@ilikenwf ilikenwf merged commit 44e4336 into ilikenwf:master Jan 29, 2016
@varhub varhub mentioned this pull request Feb 2, 2016
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