oysandvik94 / curl.nvim

Run requests with curl, formatted with jq, and persisted commands according to your own workflow
95 stars 3 forks source link

bash: cannot set terminal process group (-1): Inappropriate ioctl for device #36

Closed iamxiaojianzheng closed 2 weeks ago

iamxiaojianzheng commented 4 weeks ago

My environment is git bash + neovim 0.10.1, is it possible to solve this problem?

bash: cannot set terminal process group (-1): Inappropriate ioctl for device bash: no job control in this shell bash: "curl https://jsonplaceholder.typicode.com/todos/1 -p -sSL": No such file or directory 
oysandvik94 commented 3 weeks ago

I'll take a look!

oysandvik94 commented 2 weeks ago

@iamxiaojianzheng Can you try it with quotes around the URL? As in: curl "https://jsonplaceholder.typicode.com/todos/1" -p

iamxiaojianzheng commented 2 weeks ago

@iamxiaojianzheng Can you try it with quotes around the URL? As in: curl "https://jsonplaceholder.typicode.com/todos/1" -p

image

iamxiaojianzheng commented 2 weeks ago

I can use rest.nvim and kulala.nvim normally.

I guess this is all a wrapper around the curl command. Is there anything different about curl.nvim ?

oysandvik94 commented 2 weeks ago

That's right, it's using your underlying shell which is why it proably doesnt work. Those other plugins probably use libraries which are supported regardless of shell.

I'm guessing you get the same error if you just paste the curl command in to git bash?

iamxiaojianzheng commented 2 weeks ago

no problem in gitbash.

image

oysandvik94 commented 2 weeks ago

Could you try with the "-sSL" option as well?

iamxiaojianzheng commented 2 weeks ago

Result like this.

image

oysandvik94 commented 2 weeks ago

Interesting, thanks! I'll try and set it up on my windows PC to debug.

iamxiaojianzheng commented 2 weeks ago

Interesting, thanks! I'll try and set it up on my windows PC to debug.

This is a strange question. 😂

iamxiaojianzheng commented 2 weeks ago

@oysandvik94 I have tried to solve this problem, and I hope you can verify again whether this change is reasonable.

oysandvik94 commented 2 weeks ago

Fixed in #39