gparted bug in startup script

[hiro]

startup script uses
if test "z`ps -e | grep gpartedbin`" != "z"; then                   
ps -e doesn't exist on standard tinycorelinux so this will match against itself in the ps output, thus gparted is not started.

[Juanito]

adjusted "ps -e" -> "ps" in sbin/gparted and reposted - thanks

[curaga]

Doesn't it also need "grep -v grep" to filter the grep calls out?

[hiro]

i agree with curaga

[Juanito]

Could you give an example of exactly the line you'd like to see in the script (I realise there are four of them).

I'd suggest you also report this as a bug to gparted.

[hiro]

replace all ps -e | grep bla
with
ps | grep bla | grep -v grep

here it should be:
if test "z`ps | grep gpartedbin | grep -v grep`" != "z"; then

[Juanito]

re-modified and posted

[Misalf]

/usr/local/share/applications/gparted.desktop  seems to have a bad  Exec=  entry.

Code: [Select]

Exec= /usr/local/sbin/gparted %f
should be

Code: [Select]

Exec=sudo /usr/local/sbin/gparted

edit:
And it has no  X-FullPathIcon= .

[Juanito]

There's been a few conversations about this - basically there's no desktop icon to make it a little harder to make a big mistake with gparted - see info file.

[Misalf]

Aha, I see.

[coreplayer2]

There's been a few conversations about this - basically there's no desktop icon to make it a little harder to make a big mistake with gparted - see info file.

But where's the fun in that! :p

[drunk]

The process gpartedbin is already running.

re-modified and posted

Hi!
Could you please fix 'gparted.tcz' for 'v7.2 x86_64' too?

[Juanito]

posted

@Hiro - did you ever report this to gparted?

[hiro]

no, i find the whole script overly badly written, platform specific and rather pointless, so apart from fixing our special case i didn't want to get engaged in this.