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

PIP packaging #1602

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

lmr
Copy link
Member

@lmr lmr commented Jun 5, 2018

These are the changes I made to avocado-vt to be pip installable. For self containability purposes, the 2 last commits should be in fact merged, but I wanted to make it easier to spot the trick that makes data_dir able to handle all files inside the avocado_vt package.

So in case people think this is OK the last version should have the last 2 patches merged.

lmr added 5 commits June 5, 2018 09:28
Let's use the same naming standard that other avocado
plugins are using (avocado-framework-plugin-[name]).

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
By adding the proper dependencies into setup.py, pip
installation should pull the necessary packages for
avocado-vt to work.

Also, add the dependency on avocado-framework in requirements.txt.

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
Let's build for the most recent (at the moment), Ubuntu LTS (18.04).

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
Following the work done in avocado and other plugins,
make all extraneous resources self contained. This
way we can use easy_install and install avocado-vt
in a virtualenv.

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
After moving all relevant files to avocado_vt, change
virttest.data_dir to find files on that location.

Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
@lmr lmr requested review from ldoktor and clebergnu June 5, 2018 07:37
@ldoktor
Copy link
Contributor

ldoktor commented Jul 26, 2018

This is related to #1142 some things overlap, some are better treated here, some there... Let's merge those approaches and add the etc handling...

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