Crash on xsel not available. about ipfs-deploy HOT 3 CLOSED

After further investigation the copy to clipboard was never working but the error was ignored with a warning. Now it fails the deploy.

from ipfs-deploy.

@kevincox hmm, that's very odd. Can you try adding the flag -C to see if it fixes it?

After further investigation the copy to clipboard was never working but the error was ignored with a warning. Now it fails the deploy.

Yeah, we were not making use of any errors. I decided that it's best to fail than to hide what's happening. However, this error seems to happen during installation and not runtime so I really don't understand how it is happening now.

However if that behavious is desired there should be a documented way to disable it.

-C. I should add the entire CLI usage on the docs.

I would recomend not copying to the clipboard by default because having your clipboard editied is an unexpected default for a random CLI tool.

I personally also agree with you. The default behavior till v8 was to copy to clipboard and open the browser for you. I already disabled the opening in the web browser because of the same reasons.

I want to keep in mind that this CLI was built to make it very easy for beginners to deploy a website on IPFS so it's arguable whether or not we should disable this kind of features.

Personally, I would disable it by default or remove it entirely. We already print the link to the CLI. In most interactive terminals you can just click on them and access the website. And copying it's not hard either.

Let's just see if that fixes your issue and then see how to proceed.

from ipfs-deploy.

I was actually misunderstanding the failure. IPFS deploy was not actually crashing, my CI was just failing due to the logging change from stderr to stdout. This issue can probably be closed, although #213 does look a bit nicer anyways so maybe it is still worth merging.

The real problem that was crashing my CI was #215

from ipfs-deploy.