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

[xen staging] xen: Fix the figlet banner



commit 5bf3818c5e0321e4acdbd91a4c9456ecc40a400a
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Mon Jul 22 13:42:25 2024 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Mon Jul 22 16:25:07 2024 +0100

    xen: Fix the figlet banner
    
    4.20 is just wide enough to cause figlet to split the banner onto two lines:
    
       __  __
       \ \/ /___ _ __
        \  // _ \ '_ \
        /  \  __/ | | |
       /_/\_\___|_| |_|
    
       _  _    ____   ___                     _        _     _
      | || |  |___ \ / _ \    _   _ _ __  ___| |_ __ _| |__ | | ___
      | || |_   __) | | | |__| | | | '_ \/ __| __/ _` | '_ \| |/ _ \
      |__   _| / __/| |_| |__| |_| | | | \__ \ || (_| | |_) | |  __/
         |_|(_)_____|\___/    \__,_|_| |_|___/\__\__,_|_.__/|_|\___|
    
    Instruct figlet to use 100 column width, rather than 80, resulting in:
    
       __  __            _  _    ____   ___                     _        _     _
       \ \/ /___ _ __   | || |  |___ \ / _ \    _   _ _ __  ___| |_ __ _| |__ | 
| ___
        \  // _ \ '_ \  | || |_   __) | | | |__| | | | '_ \/ __| __/ _` | '_ \| 
|/ _ \
        /  \  __/ | | | |__   _| / __/| |_| |__| |_| | | | \__ \ || (_| | |_) | 
|  __/
       /_/\_\___|_| |_|    |_|(_)_____|\___/    \__,_|_| 
|_|___/\__\__,_|_.__/|_|\___|
    
    While fixing this, also fix a rendering error in the non-figlet case; while 
a
    leading space looks better for figlet, it looks very wrong for the simple
    one-line case.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
---
 xen/build.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/xen/build.mk b/xen/build.mk
index 0f490ca71b..942ff3867f 100644
--- a/xen/build.mk
+++ b/xen/build.mk
@@ -1,9 +1,9 @@
 quiet_cmd_banner = BANNER  $@
 define cmd_banner
     if command -v figlet >/dev/null 2>&1 ; then \
-       echo " Xen $(XEN_FULLVERSION)" | figlet -f $< > $@.tmp; \
+       echo " Xen $(XEN_FULLVERSION)" | figlet -w 100 -f $< > $@.tmp; \
     else \
-       echo " Xen $(XEN_FULLVERSION)" > $@.tmp; \
+       echo "Xen $(XEN_FULLVERSION)" > $@.tmp; \
     fi; \
     mv -f $@.tmp $@
 endef
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.