Bug 19000 - 6_s1: plasma /usr/bin/startkde: line 286: [: /usr/share: integer expression expected
Summary: 6_s1: plasma /usr/bin/startkde: line 286: [: /usr/share: integer expression ...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Nicolas Lécureuil
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 17523
  Show dependency treegraph
 
Reported: 2016-07-20 16:28 CEST by Bit Twister
Modified: 2016-07-22 22:22 CEST (History)
1 user (show)

See Also:
Source RPM: plasma-workspace-5.7.2-1.mga6.src.rpm
CVE:
Status comment:


Attachments

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


Note You need to log in before you can comment on or make changes to this bug.