Skip to content

Prevent already downloaded dependencies from hanging the download - #209

Merged
moelasmar merged 6 commits into
aws:developfrom
egalev:ignore-downloaded-dependencies
Mar 26, 2021
Merged

Prevent already downloaded dependencies from hanging the download#209
moelasmar merged 6 commits into
aws:developfrom
egalev:ignore-downloaded-dependencies

Conversation

@egalev

@egalev egalev commented Nov 5, 2020

Copy link
Copy Markdown
Contributor

Prevent already downloaded dependencies from hanging the download process (it's waiting for user input)

Issue #, if available:

Description of changes:
If your build depends on two private repos, for example your requirements.txt has:

git+ssh://git@github.com/org/package_1.git#package_1
git+ssh://git@github.com/org/package_2.git#package_2

and package_2 depends on package_1 in it's own requirements.txt, the build process hangs. It waits for user input, to decide how to deal with the existing downloaded package_1. This commit should provide a sensible default behaviour.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@moelasmar
moelasmar merged commit 165f92f into aws:develop Mar 26, 2021
aahung pushed a commit to aahung/aws-lambda-builders that referenced this pull request May 21, 2021
…s#209)

* Prevent already downloaded dependencies from hanging the download process (it's waiting for user input)

* Fixed test

* Fixed tests

* Formatted

Co-authored-by: Mohamed Elasmar <71043312+moelasmar@users.noreply.github.com>
aahung pushed a commit that referenced this pull request May 21, 2021
* Prevent already downloaded dependencies from hanging the download process (it's waiting for user input)

* Fixed test

* Fixed tests

* Formatted

Co-authored-by: Mohamed Elasmar <71043312+moelasmar@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants