Re: [PATCH] squash! diff: add --diff-line-prefix option for passing in a prefix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin <johannes.schindelin@xxxxxx> writes:

> -	test=$(echo "$cmd" | sed -e 's|[/ ][/ ]*|_|g')
> +	test=$(echo "$cmd" | sed -e 's|[/ ][/ ]*|_|g' -e 'y/>/_/')

The existing sed scriptlet says "we cannot have slash and do not
want to have space in filename, so we squash runs of them to a
single underscore".  If you have more characters that you do not
want, you should add that to the existing set instead.

While you are at it, it may be sensible to add a colon to that set,
too, no?

Something like this, perhaps?

> -	test=$(echo "$cmd" | sed -e 's|[/ ][/ ]*|_|g')
> +	test=$(echo "$cmd" | sed -e 's|[/ <:>][/ <:>]*|_|g')
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]