Re: [PATCH 1/2] docs: Import print_function in reformat-news.py

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 15, 2018 at 06:00:37PM +0100, Andrea Bolognani wrote:
> The script already works perfectly fine with Python 2, but that's
> more by chance than by design: we have a single occurrence of
> print(), and it just so happens that its only argument is an
> expression. Importing print_function makes the script more future,
> err, past proof.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  docs/reformat-news.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/docs/reformat-news.py b/docs/reformat-news.py
> index 89f7ccb3dc..fe08bf6d91 100755
> --- a/docs/reformat-news.py
> +++ b/docs/reformat-news.py
> @@ -21,6 +21,8 @@
>  # Authors:
>  #     Andrea Bolognani <abologna@xxxxxxxxxx>
>  
> +from __future__ import print_function
> +
>  import sys
>  
>  COLUMNS = 80

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux