Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Remove dependency version constraints #259

Merged
merged 2 commits into from
Oct 10, 2019
Merged

Conversation

alwa-nordic
Copy link
Contributor

Remove most dependency version constraints from requirements.txt, which
is read by setup.py. This was done so that any project that depends on
this one can choose dependency versions as freely as possible and is
reposible for testing and freezing for its releases. This change also
introduces a requirements-freeze.txt which freezes the dependency
environment to a known good state for nrfutil that will be used for
builds of nrfutil standalone executable.

Sadly this means that installing nrfutil manually trough pip can break
without warning. A possible solution for this is to create a dedicated
package for nrfutil with the frozen dependencies. This would be meant to
be run in an isolated environment using pipx or a simmilar tool.

Remove most dependency version constraints from requirements.txt, which
is read by setup.py. This was done so that any project that depends on
this one can choose dependency versions as freely as possible and is
reposible for testing and freezing for its releases. This change also
introduces a requirements-freeze.txt which freezes the dependency
environment to a known good state for nrfutil that will be used for
builds of nrfutil standalone executable.

Sadly this means that installing nrfutil manually trough pip can break
without warning. A possible solution for this is to create a dedicated
package for nrfutil with the frozen dependencies. This would be ment to
be run in a isolated environment using pipx or a simmilar tool.
@alwa-nordic alwa-nordic requested a review from bihanssen October 9, 2019 10:27
bihanssen
bihanssen previously approved these changes Oct 9, 2019
Copy link
Contributor

@bihanssen bihanssen left a comment

Choose a reason for hiding this comment

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

👍

@alwa-nordic alwa-nordic merged commit c0d7858 into master Oct 10, 2019
@alwa-nordic alwa-nordic deleted the requirements-rework branch October 10, 2019 09:01
@alwa-nordic alwa-nordic restored the requirements-rework branch October 14, 2019 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants