Revert "Remote: Change "sh -c" to "sh -lc" (#36760)"

[maxbrunsfeld]

This reverts commit bf5ed6d.

We believe this may be breaking some users whose shell initialization scripts change the working directory.

Release Notes:

• N/A

[ConradIrwin]

Thanks for figuring this out!

[Aeolun]

Jup, this was definitely breaking my my remote dev. Though to be fair, that was because my .profile spewed out a bunch of garbage. I think it'd probably be helpful if zed looks for something that looks like the output of uname if it's going to use it while also loading the profile.

[achimnol]

I also bumped with this issue, as sh cannot execute eval $(pyenv init --path) commands.
If we want to keep the original commit's intention, it should use the user's default login shell instead of the vanilla sh.

[ConradIrwin]

I think there's probably two things:

• When we're running things like uname, we want the minimal stuff happening.
• When we run shell commands within Zed, we should be putting you in a login shell.

I think there's some middle ground which I don't understand yet, which is some internal Zed commands being run. That might need some more thought, but it sounds like in some cases you need some stuff that's currently happening in sh that should be happening in a different shell?