Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package yast2-installation for openSUSE:Factory checked in at 2021-05-20 19:24:43 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-installation (Old) and /work/SRC/openSUSE:Factory/.yast2-installation.new.2988 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-installation" Thu May 20 19:24:43 2021 rev:470 rq:894091 version:4.4.7 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-installation/yast2-installation.changes 2021-05-12 19:31:48.775138094 +0200 +++ /work/SRC/openSUSE:Factory/.yast2-installation.new.2988/yast2-installation.changes 2021-05-20 19:25:03.053944820 +0200 @@ -1,0 +2,7 @@ +Tue May 18 12:39:25 UTC 2021 - Knut Anderssen <kanders...@suse.com> + +- Fallback to ncurses when a X display is not opened after 15 + seconds (bsc#1185095) +- 4.4.7 + +------------------------------------------------------------------- Old: ---- yast2-installation-4.4.6.tar.bz2 New: ---- yast2-installation-4.4.7.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-installation.spec ++++++ --- /var/tmp/diff_new_pack.dBmpNO/_old 2021-05-20 19:25:03.597942589 +0200 +++ /var/tmp/diff_new_pack.dBmpNO/_new 2021-05-20 19:25:03.597942589 +0200 @@ -17,7 +17,7 @@ Name: yast2-installation -Version: 4.4.6 +Version: 4.4.7 Release: 0 Summary: YaST2 - Installation Parts License: GPL-2.0-only ++++++ yast2-installation-4.4.6.tar.bz2 -> yast2-installation-4.4.7.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-installation-4.4.6/package/yast2-installation.changes new/yast2-installation-4.4.7/package/yast2-installation.changes --- old/yast2-installation-4.4.6/package/yast2-installation.changes 2021-05-09 21:52:22.000000000 +0200 +++ new/yast2-installation-4.4.7/package/yast2-installation.changes 2021-05-18 17:00:33.000000000 +0200 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Tue May 18 12:39:25 UTC 2021 - Knut Anderssen <kanders...@suse.com> + +- Fallback to ncurses when a X display is not opened after 15 + seconds (bsc#1185095) +- 4.4.7 + +------------------------------------------------------------------- Thu Apr 29 15:59:52 UTC 2021 - Martin Vidner <mvid...@suse.com> - Allow memory profiling of the main installer process, via diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-installation-4.4.6/package/yast2-installation.spec new/yast2-installation-4.4.7/package/yast2-installation.spec --- old/yast2-installation-4.4.6/package/yast2-installation.spec 2021-05-09 21:52:22.000000000 +0200 +++ new/yast2-installation-4.4.7/package/yast2-installation.spec 2021-05-18 17:00:33.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2-installation -Version: 4.4.6 +Version: 4.4.7 Release: 0 Summary: YaST2 - Installation Parts License: GPL-2.0-only diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-installation-4.4.6/startup/YaST2.call new/yast2-installation-4.4.7/startup/YaST2.call --- old/yast2-installation-4.4.6/startup/YaST2.call 2021-05-09 21:52:22.000000000 +0200 +++ new/yast2-installation-4.4.7/startup/YaST2.call 2021-05-18 17:00:33.000000000 +0200 @@ -37,6 +37,9 @@ # the server is ready for requests using this function # --- server_running=0 + # Accepts a max number of attempts to connect to the X-Server. In case that the number is + # reached the X-Server will be killed. By default it tries forever. (bsc#1185095) + open_display_attempts="${1:-0}" TESTX=/usr/lib/YaST2/bin/testX while kill -0 $xserver_pid 2>/dev/null ; do sleep 1 @@ -47,6 +50,12 @@ if test $err = 1;then log "\tTestX: XOpenDisplay failed" server_running=0 + ((open_display_attempts-=1)) + if [ "$open_display_attempts" = 0 ];then + log "\tXOpenDisplay failed too many times (killing XServer)" + kill_xserver + break + fi continue fi # server is running, detach oom-killer from it @@ -78,7 +87,8 @@ export DISPLAY=:0 Xorg -noreset -br -nolisten tcp -deferglyphs 16 2>/dev/tty8 1>&2 vt07 & xserver_pid=$! - wait_for_x11 + # wait 15 seconds for the X-Server before killing it. + wait_for_x11 15 if [ "$server_running" = 1 ];then log "\tX-Server is ready: $xserver_pid" fi @@ -186,6 +196,17 @@ set_inst_qt_env } +#----[ kill_xserver ]----# +function kill_xserver () { + if [ -n "$xserver_pid" ];then + sleep 1 && kill $xserver_pid + while kill -0 $xserver_pid 2>/dev/null ; do + sleep 1 + done + unset xserver_pid + fi +} + #----[ check_x11 ]----# function check_x11 () { #------------------------------------------------------ @@ -532,8 +553,10 @@ log "\tMemory requirement > $MEM_NEEDED not fulfilled -> Medium Qt disabled" MEDIUM[0]=0 fi + # 3.1.4) Check if we need to start our own X11 server... if [ -z "$DISPLAY" ];then + log "\tX-Server is needed" NEED_XSERVER=1 fi @@ -709,10 +732,7 @@ # 9.2) kill X-Server... if [ "$server_running" = 1 ];then - sleep 1 && kill $xserver_pid - while kill -0 $xserver_pid 2>/dev/null ; do - sleep 1 - done + kill_xserver fi if [ -s /etc/yast.inf ];then