On 09/14/2015 06:55 PM, larsxschneider@xxxxxxxxx wrote:
From: Lars Schneider <larsxschneider@xxxxxxxxx> A P4 repository can get into a state where it contains a file with type UTF-16 that does not contain a valid UTF-16 BOM. If git-p4 attempts to retrieve the file then the process crashes with a "Translation of file content failed" error. Signed-off-by: Lars Schneider <larsxschneider@xxxxxxxxx> --- t/t9824-git-p4-handle-utf16-without-bom.sh | 47 ++++++++++++++++++++++++++++++ 1 file changed, 47 insertions(+) create mode 100755 t/t9824-git-p4-handle-utf16-without-bom.sh diff --git a/t/t9824-git-p4-handle-utf16-without-bom.sh b/t/t9824-git-p4-handle-utf16-without-bom.sh new file mode 100755 index 0000000..fa8043b --- /dev/null +++ b/t/t9824-git-p4-handle-utf16-without-bom.sh @@ -0,0 +1,47 @@ +#!/bin/sh + +test_description='git p4 handle UTF-16 without BOM' + +. ./lib-git-p4.sh + +UTF16="\\x97\\x0\\x97\\x0" + +test_expect_success 'start p4d' ' + start_p4d +' + +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' + ( + cd "$cli" && + echo "file1 -text" > .gitattributes &&
Please no space between '>' and the filename, (this is our coding standard, and the same further down)
+ perl -e "printf \"$UTF16\"" >file1 &&
Ehh, do we need perl here ? This will invoke a process-fork, which costs time and cpu load. The following works for me: printf '\227\000\227\000' >file1 -- 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