lcitool: Don't import crypt

[libvirt] [jenkins-ci PATCH] lcitool: Don't import crypt

Posted by Andrea Bolognani 5 years, 9 months ago

It hasn't been needed since dbe55924f90d, and flake8 rightfully
complains about that.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
 guests/lcitool | 1 -
 1 file changed, 1 deletion(-)

diff --git a/guests/lcitool b/guests/lcitool
index 759eff6..e8263da 100755
--- a/guests/lcitool
+++ b/guests/lcitool
@@ -17,7 +17,6 @@
 # with this program. If not, see <https://www.gnu.org/licenses/>.
 
 import argparse
-import crypt
 import fnmatch
 import json
 import os
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [jenkins-ci PATCH] lcitool: Don't import crypt

Posted by Cole Robinson 5 years, 9 months ago

On 2/14/19 4:39 AM, Andrea Bolognani wrote:
> It hasn't been needed since dbe55924f90d, and flake8 rightfully
> complains about that.
> 
> Signed-off-by: Andrea Bolognani <abologna@redhat.com>
> ---
>  guests/lcitool | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/guests/lcitool b/guests/lcitool
> index 759eff6..e8263da 100755
> --- a/guests/lcitool
> +++ b/guests/lcitool
> @@ -17,7 +17,6 @@
>  # with this program. If not, see <https://www.gnu.org/licenses/>.
>  
>  import argparse
> -import crypt
>  import fnmatch
>  import json
>  import os
> 

Reviewed-by: Cole Robinson <crobinso@redhat.com>

- Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list