Fixed sentry_patched_popen_init regarding args passed as iterators

[lorinkoz]

Closes #436

Didn't want to write another function like _init_argument so added a force_setdefault param that enacts setting default even if there is a value already.

_init_argument with a setdefault_callback was currently failing because a is a tuple and don't support item assignment. Added conversion to list before calling the first _init_argument.

In some cases, args is not even a list but a string. Wrapped in list so as not to break the formatting of span.description.

[lorinkoz]

🛎 😃

[untitaker]

Hey, I am on vacation atm. Will check in two weeks, sorry for delay

…

On Tue, Jul 23, 2019, 13:38 Lorenzo Peña ***@***.***> wrote: 🛎 😃 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#440?email_source=notifications&email_token=AAGMPRMTYMQ4T2MQQ25YMXTQA3USZA5CNFSM4IFNTUKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2S2TJA#issuecomment-514173348>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAGMPRMEG4H5QGGQ2FGMY23QA3USZANCNFSM4IFNTUKA> .

[untitaker]

Closing in favor of #449, see also the updates in #436. This is technically a bug in your subprocess invocations. It uses the subprocess API in a way that just happens to work on CPython but does not adhere to Python docs (and fails on PyPy in theory).