long text command isn't working

[codingbond]

Hi guys,

I found that screenutils isn't running long commands, may be i am doing something wrong.
Here is an example -

s.send_commands('python streamdata.py '10.0.0.0/24' or 10.1.0.0/24'....................or 'xx.xx.xx.xx' ')

I got a screen but it just didn't run command in that screen. I need to run command manually.
I there anyway i can fix that.

Thanx in advance.

[Christophe31]

Do you use simple quotes inside simple quote? Do you have an exception? What is your platform? (Macos, bsd or linux?)

[powercode007]

hi chris thanx same thing happening with me too,
i have tried two things

s.send_commands('python streamdata.py '10.0.0.0/24' or 10.1.0.0/24'....................or 'xx.xx.xx.xx' ')
s.send_commands("python streamdata.py '10.0.0.0/24' or 10.1.0.0/24'....................or 'xx.xx.xx.xx' '")

I am using linux CentOS 7, also tried Ubuntu 14.04. Both shows same thing, no error/warning/exception, but only long commands aren't running inside screen.

[Christophe31]

s.exists is True?
does direct commands works?
screen -x yourid -X stuff "your_command"
screen -x yourid -X eval "stuff \015"

[powercode007]

direct command screen -x yourid -X stuff "your_command" isn't working

[Christophe31]

you adapted the id? how does it behaves? is the exists atrubute True?

[powercode007]

First test -

[root@localhost ~]# screen -S stuff
[root@localhost ~]# screen -x 25088 -X stuff "pwd"
[root@localhost ~]# screen -r 25088 -> [root@localhost ~]# pwd
[detached from 25088.stuff]

Second Test -
[root@localhost ~]# screen -S stuff2
[root@localhost ~]# screen -x 25648 -X stuff2 "python streamdata.py '10.0.0.0/24' or 10.1.0.0/24'..............(this very list string)................or 'xx.xx.xx.xx' "
[root@localhost ~]# screen -r 25648 -> [root@localhost ~]#
[detached from 25648.stuff2]

I couldn't understand exist attribute completely, if you are talking about screenutils script i haven't set s.exist true.

[Christophe31]

It's an atribute, you don't have to set it, it gives you a computed value.
You should not change to stuff2 in your second example, stuff is a screen command…

[powercode007]

i am sharing my code snippet, probably clear more things...

# line_filter is dict key for screen name and value for screen command
for k, v in line_filter.iteritems():
    k = Screen(str(k))
    k.initialize()
    print k.exists
    if k.exists:
        k.send_commands(v)

list_screens()

[root@localhost ~]# screen -S hello
[detached from 25483.hello]
[root@localhost ~]# screen -x 25483 -X stuff "p645555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555wd"
[root@localhost ~]# screen -r hello -> [root@localhost ~]#
[detached from 25483.hello]
[root@localhost ~]# screen -x 25088 -X stuff "pwd"
No screen session found.
[root@localhost ~]# screen -x 25483 -X stuff "pwd"
[root@localhost ~]# screen -r 25483 -> [root@localhost ~]# pwd
[detached from 25483.hello]
[root@localhost ~]# 

[Christophe31]

You don't need to initialize if the screen already exist but that's not the issue, I don't know what kind of script you are doing but if you don't have tons of dependencies, you may consider using python3 (which in this snippet would transfor .iteritems() in items() and andd parenthesis to print), you may want to use Screen(str(k), True) to skip initialization, and as you just instanciated the screen, you don't need to check if it exists, (I agree).

[Christophe31]

screen -x 25648 -X stuff "python streamdata.py '10.0.0.0/24' or 10.1.0.0/24'..............(this very long string)................or 'xx.xx.xx.xx' "
what appen when you launch that? (stuff iis not the screen name, but a command)

[powercode007]

yes i got ur point and i tried as u ask
i create a screen with name hello and pid 25483, then run screen stuff command with long string command. command executed nicely. but when i attach hello screen result is command didn't get copied to detached command. while command get copied in case of pwd command inside detached screen.

[Christophe31]

If you have tons of screens, you must avoid name collisions. (you should never have a screen name equals a an other screen name starts)