handle arguments when passed with filename

[cryptid11]

so if I do "filename arg1 arg2" it works as expected, this doesn't manage to work when there are spaces in filenames, but this is good since spaces in filenames are really bad practice XD

[radare]

will be good to document this in the pydoc and add a example script. @a1ext what do you think about this pr?

[a1ext]

will be good to document this in the pydoc and add a example script. @a1ext what do you think about this pr?

Agree with you. @cryptid11 do you think it is good to just check for a whitespace?

P.S. @radare, can you set up dummy appveyor/travisci for this repo? I'm planning to add some tests.

[cryptid11]

in theory should be, except file with whitespace in name, right? anyway that was really useful for me, obviously if this create damage more than good should not be included.

anyway integrate that with other checks if you feel, probably the code can also be cleaner, or other shapes you prefer (i prefer one line for example like cmd = cmd[:1] + flags + cmd[1:-1] + [cmd[-1].split(' ')[0]] + ['-Rarg'+str(x+1)+'='+arg for x,arg in enumerate(filename.split(' ')[1:])]), etc

[radare]

what about checking if file is an array and handle r2pipe.open(['ls', '-l', '/'], ['-d']) or so?

…

On 14 Jan 2019, at 22:02, cryptid11 ***@***.***> wrote: in theory should be, except file with whitespace in name, right? anyway that was really useful for me, obviously if this create damage more than good should not be included. anyway integrate that with other checks if you feel, probably the code can also be cleaner, or other shapes you prefer (i prefer one line for example like cmd = cmd[:1] + flags + cmd[1:-1] + [cmd[-1].split(' ')[0]] + ['-Rarg'+str(x+1)+'='+arg for x,arg in enumerate(filename.split(' ')[1:])]), etc — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#91 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3-lmb5azygwf53PfMIXsUNsSxtSeIBks5vDPB5gaJpZM4Z_LVg>.

[a1ext]

what about checking if file is an array and handle r2pipe.open(['ls', '-l', '/'], ['-d']) or so?

good idea

[radare]

can you update the PR? do you agree with the change?

[a1ext]

@cryptid11 Could you adjust the code as @radare proposed:

what about checking if file is an array and handle r2pipe.open(['ls', '-l', '/'], ['-d']) or so?

[cryptid11]

honestly I forget this stuff, but if you tell me what to change in what I will do that surely

[radare]

The idea is to also handle the case where the function receives an array insteaad of a string

[trufae]

please resolve conflicts