Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

!! notation expands if using bash weak quoting (double quotes) #19

Open
rogerdubbs opened this issue May 20, 2020 · 7 comments
Open

!! notation expands if using bash weak quoting (double quotes) #19

rogerdubbs opened this issue May 20, 2020 · 7 comments
Labels
question Further information is requested

Comments

@rogerdubbs
Copy link

rogerdubbs commented May 20, 2020

Example:

> p4 submit -d "R!! inline method"

This will expand to

> p4 submit -d "Rmylastcommand inline method"

or something.

I guess I need to learn to use single quotes to use this.

@arlobelshee
Copy link
Contributor

Ugh. Yeah, that's a problem. I never use command-line for messages, so never though of shell escapes when creating it.

Do you have any alternatives you might recommend?

The main thing I like is that it stands out. It is 3 chars wide, with the first indicating what's going on and the next two screaming RISK IS HERE!!

@arlobelshee
Copy link
Contributor

What do people think about **

F**
B**
R**

Hm. Looks like I'm cussing. I don't particularly like that.

@arlobelshee arlobelshee added the question Further information is requested label Jul 2, 2021
@arlobelshee
Copy link
Contributor

I'm still open to a proposal that avoids terminal history expansion. I don't have an idea, so can anyone make a good proposal?

@JayBazuzi
Copy link
Collaborator

There are so many different shells out there with so many different rules for which characters are special in which contexts. It's hard to design any system that won't trip up any shell.

Here's another example:

> git commit -am "r   rename `t` -> `u`"

In PowerShell backtick is the escape character, so backtick-t becomes a TAB. So in PowerShell I try to always use single quotes unless I know I want special shell expansion things to happen in my string.

Meanwhile, in Python ' and " have the same treatment. Sigh.

@JayBazuzi
Copy link
Collaborator

Some more character ideas for unsafe:
R++
R@@

Or maybe we change the layout:
!R!
*R*

Maybe we also switch the safe changes to .r. which would address the multiple-space-collapse issue (#26). . and ! have a nice visual relationship.

@jmasonlee
Copy link

I like the layout change. I feel like it makes things more useable.

@arlobelshee
Copy link
Contributor

#43 Considers an alternate layout. Addresses this concern and also makes risk more clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants