Tiny Core Linux

Tiny Core Extensions => TCE Bugs => Topic started by: hiro on August 30, 2016, 05:01:11 PM

Title: gparted bug in startup script
Post by: hiro on August 30, 2016, 05:01:11 PM
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.
Title: Re: gparted bug in startup script
Post by: Juanito on August 31, 2016, 07:47:28 AM
adjusted "ps -e" -> "ps" in sbin/gparted and reposted - thanks
Title: Re: gparted bug in startup script
Post by: curaga on August 31, 2016, 12:22:34 PM
Doesn't it also need "grep -v grep" to filter the grep calls out?
Title: Re: gparted bug in startup script
Post by: hiro on August 31, 2016, 06:57:48 PM
i agree with curaga
Title: Re: gparted bug in startup script
Post by: Juanito on September 01, 2016, 01:48:38 AM
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.
Title: Re: gparted bug in startup script
Post by: hiro on September 01, 2016, 05:00:06 AM
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
Title: Re: gparted bug in startup script
Post by: Juanito on September 01, 2016, 10:13:22 AM
re-modified and posted
Title: Re: gparted bug in startup script
Post by: Misalf on September 04, 2016, 11:18:56 AM
/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= .
Title: Re: gparted bug in startup script
Post by: Juanito on September 04, 2016, 01:01:18 PM
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.
Title: Re: gparted bug in startup script
Post by: Misalf on September 04, 2016, 01:20:30 PM
Aha, I see.
Title: Re: gparted bug in startup script
Post by: coreplayer2 on September 04, 2016, 04:45:25 PM
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
Title: Re: gparted bug in startup script
Post by: drunk on December 17, 2016, 04:31:37 AM
Quote
The process gpartedbin is already running.
Quote
re-modified and posted

Hi!
Could you please fix 'gparted.tcz' for 'v7.2 x86_64' too?
Title: Re: gparted bug in startup script
Post by: Juanito on December 17, 2016, 08:01:18 AM
posted

@Hiro - did you ever report this to gparted?
Title: Re: gparted bug in startup script
Post by: hiro on December 17, 2016, 09:22:03 AM
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.