what am I doing wrong?

[spithash]

Code: Select all

{exec cut \-d. \-f1 /proc/uptime} reply; set secs [expr $reply % 60]; set mins [expr $reply / 60 % 60]; set hours [expr $reply / 3600 % 24]; set days [expr $reply / 86400]

Can anyone please let me know how can I fix this? what am I doing wrong?
I'm not a coder, I'm trying to be one. this is from a script I found and I am trying to fix it.

The errors I get are:

Code: Select all

% {exec cut \-d. \-f1 /proc/uptime} reply; set secs [expr $reply % 60]; set mins [expr $reply / 60 % 60]; set hours [expr $reply / 3600 % 24]; set days [expr $reply / 86400]
invalid command name "exec cut \-d. \-f1 /proc/uptime"
%

[caesar]

if you wish to store the result of the exec command in the reply variable, then replace:

Code: Select all

{exec cut \-d. \-f1 /proc/uptime} reply;

with either:

Code: Select all

set reply [exec cut \-d. \-f1 /proc/uptime]

or:

Code: Select all

catch {exec cut \-d. \-f1 /proc/uptime} reply

I would go with first option.

[spithash]

what about this?

Code: Select all

[03:59:38] <@spithash> ;set reply {exec cut -d. -f1 /proc/uptime} set secs [expr {$reply % 60}]; set mins [expr {$reply / 60 % 60}]; set hours [expr {$reply / 3600 % 24}]; set days [expr {$reply / 86400}]
[03:59:40] <@nigger> spithash: #29 Tcl error: can't use non-numeric string as operand of "%"

[caesar]

Before the set secs you forgot to add an ;

Try with:

Code: Select all

set reply [exec cut -d. -f1 /proc/uptime]; set secs [expr {$reply % 60}]; set mins [expr {$reply / 60 % 60}]; set hours [expr {$reply / 3600 % 24}]; set days [expr {$reply / 86400}]