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

Added pypi:testresources required by one of the tockloader dependencies; #376

Merged
merged 1 commit into from
Sep 10, 2021

Conversation

mxi1
Copy link

@mxi1 mxi1 commented Sep 10, 2021

Signed-off-by: minjun xi.minjun@gmail.com

Fixes #343

It's a good idea to open an issue first for discussion.

Running setup.sh on ubuntu versions (at least 20.04, 20.10), there will be one unmet dependency error message, indicating "launchpadlib" requires "testresources".

The pypi package "testresources" requires to be installed, but it doesn't.

The solution is easy here: install testresouces pypi package, before installing "tockloader", "intelhex", etc.

  • [Y] Tests pass
  • Appropriate changes to README are included in PR

@google-cla google-cla bot added the cla: yes label Sep 10, 2021
@kaczmarczyck kaczmarczyck changed the base branch from bugfix to stable September 10, 2021 07:20
@kaczmarczyck kaczmarczyck changed the base branch from stable to bugfix September 10, 2021 07:20
@kaczmarczyck
Copy link
Collaborator

Thanks for the contribution to make OpenSK work on more platforms!
Sorry, our bugfix branch was behind. If you rebase onto the new bugfix I just merged, the checks should work.

kaczmarczyck
kaczmarczyck previously approved these changes Sep 10, 2021
@kaczmarczyck
Copy link
Collaborator

I went ahead and rebased, to see if it works :)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 49.018% when pulling f908988 on mxi1:bugfix into 2f24f52 on google:bugfix.

@kaczmarczyck kaczmarczyck merged commit 04bd0b2 into google:bugfix Sep 10, 2021
kaczmarczyck pushed a commit to kaczmarczyck/OpenSK that referenced this pull request Nov 19, 2021
kaczmarczyck pushed a commit that referenced this pull request Nov 19, 2021
…es; (#376)

Signed-off-by: minjun <xi.minjun@gmail.com>
kaczmarczyck pushed a commit to kaczmarczyck/OpenSK that referenced this pull request Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants