Bug 19000

Summary:	6_s1: plasma /usr/bin/startkde: line 286: [: /usr/share: integer expression expected
Product:	Mageia	Reporter:	Bit Twister <bittwister2>
Component:	RPM Packages	Assignee:	Nicolas Lécureuil <mageia>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	marja11
Version:	Cauldron
Target Milestone:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Source RPM:	plasma-workspace-5.7.2-1.mga6.src.rpm	CVE:
Status comment:
Bug Depends on:
Bug Blocks:	17523

Description Bit Twister 2016-07-20 16:28:54 CEST

Description of problem:

localhost being added to access control list
Loading stage  "initial" 346
startkde: Starting up...
/usr/bin/startkde: line 286: [: /usr/share: integer expression expected
dbus-update-activation-environment: warning: error sending to systemd: org.freedesktop.DBus.Error.Spawn.ChildExited: Process org.freedesktop.systemd1 exited with status 1

Version-Release number of selected component (if applicable):


How reproducible: always


Steps to Reproduce:
1. grep /usr/bin/startkde ~/.xsession-errors

Marja Van Waes 2016-07-20 18:26:22 CEST

CC: (none) => marja11
Blocks: (none) => 17523
Assignee: bugsquad => mageia

Comment 1 Nicolas Lécureuil 2016-07-20 19:21:44 CEST

please provide  rpm -qf  /usr/bin/startkde

Comment 2 Bit Twister 2016-07-20 20:23:19 CEST

(In reply to Nicolas Lécureuil from comment #1)
> please provide  rpm -qf  /usr/bin/startkde

$ rpm -qf  /usr/bin/startkde
plasma-workspace-5.7.2-1.mga6

Comment 3 Nicolas Lécureuil 2016-07-22 19:07:53 CEST

Is it better now ?

Comment 4 Bit Twister 2016-07-22 19:38:05 CEST

(In reply to Nicolas Lécureuil from comment #3)
> Is it better now ?

Only because I fixed it 
$ dir /usr/bin/startkde
-rwxr-xr-x 1 root root 14703 Jul 20 09:30 /usr/bin/startkde

Problem still in Jul 19 17:21 /usr/bin/startkde on last night's network install plus this morning's ~9am updates.

Comment 5 Nicolas Lécureuil 2016-07-22 19:56:34 CEST

when you fix this is nice to provide patches :)

Comment 6 Bit Twister 2016-07-22 20:24:12 CEST

(In reply to Nicolas Lécureuil from comment #5)
> when you fix this is nice to provide patches :)

You have obviously confused me with someone who knows how to fix the rpm. :)

The workaround is:

change    if [ "/usr/share" -eq "/usr/share" ]; then
to        if [ "/usr/share" = "/usr/share" ]; then

at line 286 in /usr/bin/startkde

Comment 7 Nicolas Lécureuil 2016-07-22 20:29:49 CEST

this is what i done in the last rpm :(

Comment 8 Rémi Verschelde 2016-07-22 20:36:39 CEST

Actually, in the last rpm you removed the trailing /: http://svnweb.mageia.org/packages/cauldron/plasma-workspace/current/SOURCES/plasma-workspace-5.7.1-fix-XDG_DATA_DIRS.patch?r1=1042353&r2=1042369

Here Bit Twister mentions that apparently you need to do 's/ -eq/=/'

Comment 9 Nicolas Lécureuil 2016-07-22 20:40:20 CEST

**$$mqsd$qsdm

i added on svn "locally" and forgot to commit :(
i think i need vacations :p

Index: SOURCES/plasma-workspace-5.7.1-fix-XDG_DATA_DIRS.patch
===================================================================
--- SOURCES/plasma-workspace-5.7.1-fix-XDG_DATA_DIRS.patch      (révision 1042964)
+++ SOURCES/plasma-workspace-5.7.1-fix-XDG_DATA_DIRS.patch      (copie de travail)
@@ -6,7 +6,7 @@
  # /usr/local.
  if test -z "$XDG_DATA_DIRS"; then
 -    XDG_DATA_DIRS="@CMAKE_INSTALL_PREFIX@/@SHARE_INSTALL_PREFIX@:/usr/share:/usr/local/share"
-+       if [ "@SHARE_INSTALL_PREFIX@"  -eq "/usr/share" ]; then
++       if [ "@SHARE_INSTALL_PREFIX@"  == "/usr/share" ]; then
 +               XDG_DATA_DIRS="/usr/share:/usr/local/share"
 +       else
 +               XDG_DATA_DIRS="@SHARE_INSTALL_PREFIX@:/usr/share:/usr/local/share

Comment 10 Nicolas Lécureuil 2016-07-22 20:41:20 CEST

sorry , should be better on next rpm.

Comment 11 Nicolas Lécureuil 2016-07-22 22:22:10 CEST

Fixed in new plasma-workspace

Status: NEW => RESOLVED
Resolution: (none) => FIXED