# [PATCH v4 0/2] git-p4: handle "Translation of file content failed" 14 messages from 2015-09-21 to 2015-09-23. Participants: larsxschneider@gmail.com, Junio C Hamano, Lars Schneider, Eric Sunshine, Michael Blume. Thread: https://gitlist.dev/t/40394 ## larsxschneider@gmail.com, 2015-09-21 10:01 Subject: [PATCH v4 0/2] git-p4: handle "Translation of file content failed" Message-ID: <1442829701-2347-1-git-send-email-larsxschneider@gmail.com> URL: https://gitlist.dev/e/1442829701-2347-1-git-send-email-larsxschneider%40gmail.com ``` From: Lars Schneider diff to v3: * replace non portable "sed -i" call in test case (thanks Luke and Torsten!) * use "test_expect_failure" in the first commit that adds the test case, flip it to "test_expect_success" in subsequent commit (thanks Eric and Luke!) * rename test case from t9824... to t9825... to avoid clashes with "git-p4: add Git LFS backend for large file system" patch Cheers, Lars Lars Schneider (2): git-p4: add test case for "Translation of file content failed" error git-p4: handle "Translation of file content failed" git-p4.py | 27 +++++++++------- t/t9825-git-p4-handle-utf16-without-bom.sh | 50 ++++++++++++++++++++++++++++++ 2 files changed, 66 insertions(+), 11 deletions(-) create mode 100755 t/t9825-git-p4-handle-utf16-without-bom.sh -- 2.5.1 ``` ## larsxschneider@gmail.com, 2015-09-21 10:01 Subject: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: <1442829701-2347-2-git-send-email-larsxschneider@gmail.com> URL: https://gitlist.dev/e/1442829701-2347-2-git-send-email-larsxschneider%40gmail.com In-Reply-To: <1442829701-2347-1-git-send-email-larsxschneider@gmail.com> ``` From: Lars Schneider 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. More info here: http://answers.perforce.com/articles/KB/3117 Signed-off-by: Lars Schneider --- t/t9825-git-p4-handle-utf16-without-bom.sh | 50 ++++++++++++++++++++++++++++++ 1 file changed, 50 insertions(+) create mode 100755 t/t9825-git-p4-handle-utf16-without-bom.sh diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh b/t/t9825-git-p4-handle-utf16-without-bom.sh new file mode 100755 index 0000000..65c3c4e --- /dev/null +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh @@ -0,0 +1,50 @@ +#!/bin/sh + +test_description='git p4 handling of UTF-16 files without BOM' + +. ./lib-git-p4.sh + +UTF16="\227\000\227\000" + +test_expect_success 'start p4d' ' + start_p4d +' + +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' + ( + cd "$cli" && + printf "$UTF16" >file1 && + p4 add -t utf16 file1 && + p4 submit -d "file1" + ) && + + ( + cd "db" && + p4d -jc && + # P4D automatically adds a BOM. Remove it here to make the file invalid. + sed -e "$ d" depot/file1,v >depot/file1,v.new && + mv -- depot/file1,v.new depot/file1,v && + printf "@$UTF16@" >>depot/file1,v && + p4d -jrF checkpoint.1 + ) +' + +test_expect_failure 'clone depot with invalid UTF-16 file in verbose mode' ' + git p4 clone --dest="$git" --verbose //depot && + test_when_finished cleanup_git && + ( + cd "$git" && + printf "$UTF16" >expect && + test_cmp_bin expect file1 + ) +' + +test_expect_failure 'clone depot with invalid UTF-16 file in non-verbose mode' ' + git p4 clone --dest="$git" //depot +' + +test_expect_success 'kill p4d' ' + kill_p4d +' + +test_done -- 2.5.1 ``` ## larsxschneider@gmail.com, 2015-09-21 10:01 Subject: [PATCH v4 2/2] git-p4: handle "Translation of file content failed" Message-ID: <1442829701-2347-3-git-send-email-larsxschneider@gmail.com> URL: https://gitlist.dev/e/1442829701-2347-3-git-send-email-larsxschneider%40gmail.com In-Reply-To: <1442829701-2347-1-git-send-email-larsxschneider@gmail.com> ``` From: Lars Schneider 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. More info here: http://answers.perforce.com/articles/KB/3117 Fix this by detecting this error and retrieving the file as binary instead. The result in Git is the same. Known issue: This works only if git-p4 is executed in verbose mode. In normal mode no exceptions are thrown and git-p4 just exits. Signed-off-by: Lars Schneider --- git-p4.py | 27 ++++++++++++++++----------- t/t9825-git-p4-handle-utf16-without-bom.sh | 2 +- 2 files changed, 17 insertions(+), 12 deletions(-) diff --git a/git-p4.py b/git-p4.py index 073f87b..5ae25a6 100755 --- a/git-p4.py +++ b/git-p4.py @@ -134,13 +134,11 @@ def read_pipe(c, ignore_error=False): sys.stderr.write('Reading pipe: %s\n' % str(c)) expand = isinstance(c,basestring) - p = subprocess.Popen(c, stdout=subprocess.PIPE, shell=expand) - pipe = p.stdout - val = pipe.read() - if p.wait() and not ignore_error: - die('Command failed: %s' % str(c)) - - return val + p = subprocess.Popen(c, stdout=subprocess.PIPE, stderr=subprocess.PIPE, shell=expand) + (out, err) = p.communicate() + if p.returncode != 0 and not ignore_error: + die('Command failed: %s\nError: %s' % (str(c), err)) + return out def p4_read_pipe(c, ignore_error=False): real_cmd = p4_build_cmd(c) @@ -2186,10 +2184,17 @@ class P4Sync(Command, P4UserMap): # them back too. This is not needed to the cygwin windows version, # just the native "NT" type. # - text = p4_read_pipe(['print', '-q', '-o', '-', "%s@%s" % (file['depotFile'], file['change']) ]) - if p4_version_string().find("/NT") >= 0: - text = text.replace("\r\n", "\n") - contents = [ text ] + try: + text = p4_read_pipe(['print', '-q', '-o', '-', '%s@%s' % (file['depotFile'], file['change'])]) + except Exception as e: + if 'Translation of file content failed' in str(e): + type_base = 'binary' + else: + raise e + else: + if p4_version_string().find('/NT') >= 0: + text = text.replace('\r\n', '\n') + contents = [ text ] if type_base == "apple": # Apple filetype files will be streamed as a concatenation of diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh b/t/t9825-git-p4-handle-utf16-without-bom.sh index 65c3c4e..fd2edce 100755 --- a/t/t9825-git-p4-handle-utf16-without-bom.sh +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh @@ -29,7 +29,7 @@ test_expect_success 'init depot with UTF-16 encoded file and artificially remove ) ' -test_expect_failure 'clone depot with invalid UTF-16 file in verbose mode' ' +test_expect_success 'clone depot with invalid UTF-16 file in verbose mode' ' git p4 clone --dest="$git" --verbose //depot && test_when_finished cleanup_git && ( -- 2.5.1 ``` ## Junio C Hamano, 2015-09-21 18:09 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/xmqqio73abl0.fsf%40gitster.mtv.corp.google.com In-Reply-To: <1442829701-2347-2-git-send-email-larsxschneider@gmail.com> ``` larsxschneider@gmail.com writes: > From: Lars Schneider > > 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. > > More info here: http://answers.perforce.com/articles/KB/3117 > > Signed-off-by: Lars Schneider > --- > t/t9825-git-p4-handle-utf16-without-bom.sh | 50 ++++++++++++++++++++++++++++++ > 1 file changed, 50 insertions(+) > create mode 100755 t/t9825-git-p4-handle-utf16-without-bom.sh > > diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh > b/t/t9825-git-p4-handle-utf16-without-bom.sh > new file mode 100755 > index 0000000..65c3c4e > --- /dev/null > +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh > @@ -0,0 +1,50 @@ > +#!/bin/sh > + > +test_description='git p4 handling of UTF-16 files without BOM' > + > +. ./lib-git-p4.sh > + > +UTF16="\227\000\227\000" > + > +test_expect_success 'start p4d' ' > + start_p4d > +' > + > +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' > + ( > + cd "$cli" && > + printf "$UTF16" >file1 && > + p4 add -t utf16 file1 && > + p4 submit -d "file1" > + ) && > + > + ( > + cd "db" && > + p4d -jc && > + # P4D automatically adds a BOM. Remove it here to make the file invalid. > + sed -e "$ d" depot/file1,v >depot/file1,v.new && Do you need the space between the address $ (i.e. the last line) and operation 'd' (i.e. delete it)? I am asking because that looks very unusual at least in our codebase. > + mv -- depot/file1,v.new depot/file1,v && > + printf "@$UTF16@" >>depot/file1,v && > + p4d -jrF checkpoint.1 > + ) > +' > + > +test_expect_failure 'clone depot with invalid UTF-16 file in verbose mode' ' > + git p4 clone --dest="$git" --verbose //depot && > + test_when_finished cleanup_git && > + ( > + cd "$git" && > + printf "$UTF16" >expect && > + test_cmp_bin expect file1 > + ) > +' > + > +test_expect_failure 'clone depot with invalid UTF-16 file in non-verbose mode' ' > + git p4 clone --dest="$git" //depot > +' > + > +test_expect_success 'kill p4d' ' > + kill_p4d > +' > + > +test_done ``` ## Lars Schneider, 2015-09-21 23:03 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/E47DE9F0-6017-4E96-AC29-E6C60C4D85CB%40gmail.com In-Reply-To: ``` On 21 Sep 2015, at 20:09, Junio C Hamano wrote: > larsxschneider@gmail.com writes: > >> From: Lars Schneider >> >> 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. >> >> More info here: http://answers.perforce.com/articles/KB/3117 >> >> Signed-off-by: Lars Schneider >> --- >> t/t9825-git-p4-handle-utf16-without-bom.sh | 50 ++++++++++++++++++++++++++++++ >> 1 file changed, 50 insertions(+) >> create mode 100755 t/t9825-git-p4-handle-utf16-without-bom.sh >> >> diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh >> b/t/t9825-git-p4-handle-utf16-without-bom.sh >> new file mode 100755 >> index 0000000..65c3c4e >> --- /dev/null >> +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh >> @@ -0,0 +1,50 @@ >> +#!/bin/sh >> + >> +test_description='git p4 handling of UTF-16 files without BOM' >> + >> +. ./lib-git-p4.sh >> + >> +UTF16="\227\000\227\000" >> + >> +test_expect_success 'start p4d' ' >> + start_p4d >> +' >> + >> +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' >> + ( >> + cd "$cli" && >> + printf "$UTF16" >file1 && >> + p4 add -t utf16 file1 && >> + p4 submit -d "file1" >> + ) && >> + >> + ( >> + cd "db" && >> + p4d -jc && >> + # P4D automatically adds a BOM. Remove it here to make the file invalid. >> + sed -e "$ d" depot/file1,v >depot/file1,v.new && > > Do you need the space between the address $ (i.e. the last line) and > operation 'd' (i.e. delete it)? I am asking because that looks very > unusual at least in our codebase. Well, I am no “sed” pro. I have to admit that I found this snippet on the Internet and it just worked. If I remove the space then it does not work. I was not yet able to figure out why… anyone an idea? Thanks, Lars > >> + mv -- depot/file1,v.new depot/file1,v && >> + printf "@$UTF16@" >>depot/file1,v && >> + p4d -jrF checkpoint.1 >> + ) >> +' >> + >> +test_expect_failure 'clone depot with invalid UTF-16 file in verbose mode' ' >> + git p4 clone --dest="$git" --verbose //depot && >> + test_when_finished cleanup_git && >> + ( >> + cd "$git" && >> + printf "$UTF16" >expect && >> + test_cmp_bin expect file1 >> + ) >> +' >> + >> +test_expect_failure 'clone depot with invalid UTF-16 file in non-verbose mode' ' >> + git p4 clone --dest="$git" //depot >> +' >> + >> +test_expect_success 'kill p4d' ' >> + kill_p4d >> +' >> + >> +test_done ``` ## Eric Sunshine, 2015-09-21 23:54 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/CAPig%2BcRV-RCdcmAHG%2BbRL6_yYYNCRqQPQ%2Bv3KCXwC81StGKibg%40mail.gmail.com In-Reply-To: ``` On Mon, Sep 21, 2015 at 7:03 PM, Lars Schneider wrote: > On 21 Sep 2015, at 20:09, Junio C Hamano wrote: >> larsxschneider@gmail.com writes: >>> +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' >>> + ( >>> + cd "db" && >>> + p4d -jc && >>> + # P4D automatically adds a BOM. Remove it here to make the file invalid. >>> + sed -e "$ d" depot/file1,v >depot/file1,v.new && >> >> Do you need the space between the address $ (i.e. the last line) and >> operation 'd' (i.e. delete it)? I am asking because that looks very >> unusual at least in our codebase. > > Well, I am no “sed” pro. I have to admit that I found this snippet > on the Internet and it just worked. If I remove the space then it > does not work. I was not yet able to figure out why… anyone an idea? Yes, it's because $d is a variable reference, even within double quotes. Typically, one uses single quotes around the sed argument to suppress this sort of undesired behavior. Since the entire test body is already within single quotes, however, changing the sed argument to use single quotes, rather than double, will require escaping them: sed -e \'$d\' depot/file... Aside: You could also drop the unnecessary quotes from the 'cd' argument. ``` ## Junio C Hamano, 2015-09-22 01:10 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/xmqqbncv6yym.fsf%40gitster.mtv.corp.google.com In-Reply-To: ``` Eric Sunshine writes: > Yes, it's because $d is a variable reference, even within double > quotes. s/even/especially/ ;-) Here is what I queued as SQUASH??? diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh b/t/t9825-git-p4-handle-utf16-without-bom.sh index 65c3c4e..735c0bb 100644 --- a/t/t9825-git-p4-handle-utf16-without-bom.sh +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh @@ -22,8 +22,8 @@ test_expect_success 'init depot with UTF-16 encoded file and artificially remove cd "db" && p4d -jc && # P4D automatically adds a BOM. Remove it here to make the file invalid. - sed -e "$ d" depot/file1,v >depot/file1,v.new && - mv -- depot/file1,v.new depot/file1,v && + sed -e "\$d" depot/file1,v >depot/file1,v.new && + mv depot/file1,v.new depot/file1,v && printf "@$UTF16@" >>depot/file1,v && p4d -jrF checkpoint.1 ) ``` ## Lars Schneider, 2015-09-22 10:09 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: <9F835973-7045-4AA7-A0B0-D3D3C6F25D73@gmail.com> URL: https://gitlist.dev/e/9F835973-7045-4AA7-A0B0-D3D3C6F25D73%40gmail.com In-Reply-To: ``` On 22 Sep 2015, at 03:10, Junio C Hamano wrote: > Eric Sunshine writes: > >> Yes, it's because $d is a variable reference, even within double >> quotes. > > s/even/especially/ ;-) > > Here is what I queued as SQUASH??? > > diff --git a/t/t9825-git-p4-handle-utf16-without-bom.sh b/t/t9825-git-p4-handle-utf16-without-bom.sh > index 65c3c4e..735c0bb 100644 > --- a/t/t9825-git-p4-handle-utf16-without-bom.sh > +++ b/t/t9825-git-p4-handle-utf16-without-bom.sh > @@ -22,8 +22,8 @@ test_expect_success 'init depot with UTF-16 encoded file and artificially remove > cd "db" && > p4d -jc && > # P4D automatically adds a BOM. Remove it here to make the file invalid. > - sed -e "$ d" depot/file1,v >depot/file1,v.new && > - mv -- depot/file1,v.new depot/file1,v && > + sed -e "\$d" depot/file1,v >depot/file1,v.new && > + mv depot/file1,v.new depot/file1,v && > printf "@$UTF16@" >>depot/file1,v && > p4d -jrF checkpoint.1 > ) This works. I even tested successfully this one: sed \$d depot/file1,v >depot/file1,v.new && Do we need the “-e” option? Thanks, Lars ``` ## Lars Schneider, 2015-09-22 10:12 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/FAA838A1-CBD8-4716-ADDE-6CC912C19BE6%40gmail.com In-Reply-To: ``` On 22 Sep 2015, at 01:54, Eric Sunshine wrote: > On Mon, Sep 21, 2015 at 7:03 PM, Lars Schneider > wrote: >> On 21 Sep 2015, at 20:09, Junio C Hamano wrote: >>> larsxschneider@gmail.com writes: >>>> +test_expect_success 'init depot with UTF-16 encoded file and artificially remove BOM' ' >>>> + ( >>>> + cd "db" && >>>> + p4d -jc && >>>> + # P4D automatically adds a BOM. Remove it here to make the file invalid. >>>> + sed -e "$ d" depot/file1,v >depot/file1,v.new && >>> >>> Do you need the space between the address $ (i.e. the last line) and >>> operation 'd' (i.e. delete it)? I am asking because that looks very >>> unusual at least in our codebase. >> >> Well, I am no “sed” pro. I have to admit that I found this snippet >> on the Internet and it just worked. If I remove the space then it >> does not work. I was not yet able to figure out why… anyone an idea? > > Yes, it's because $d is a variable reference, even within double > quotes. Typically, one uses single quotes around the sed argument to > suppress this sort of undesired behavior. Since the entire test body > is already within single quotes, however, changing the sed argument to > use single quotes, rather than double, will require escaping them: > > sed -e \'$d\' depot/file... > > Aside: You could also drop the unnecessary quotes from the 'cd' argument. Thanks for the explanation. Plus you are correct with the quotes around “db”… just a habit. @Junio: If it is no extra work for you, you can remove the quotes around “db”. I can also create a new patch roll including the sed change and the quote change if it is easier for you. Best, Lars ``` ## Junio C Hamano, 2015-09-22 16:02 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/xmqq8u7y5toe.fsf%40gitster.mtv.corp.google.com In-Reply-To: <9F835973-7045-4AA7-A0B0-D3D3C6F25D73@gmail.com> ``` Lars Schneider writes: > This works. OK, and thanks; as I don't do perforce, the squash was without any testing. > Do we need the “-e” option? In syntactic sense, no, but our codebase tends to prefer to have one, because it is easier to spot which ones are the instructions if you consistently have "-e" even when you give only one. ``` ## Junio C Hamano, 2015-09-22 16:02 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/xmqq4mim5tn1.fsf%40gitster.mtv.corp.google.com In-Reply-To: ``` Lars Schneider writes: > If it is no extra work for you, you can remove the quotes around > “db”. I can also create a new patch roll including the sed change > and the quote change if it is easier for you. Now you've tested the SQUASH??? for me, I can just squash that into your original without resend. Thanks. ``` ## Michael Blume, 2015-09-22 19:11 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/CAO2U3QgehMcBrDUtChLLrn5VrH4jLE0CF5xDSShY72yycLryCg%40mail.gmail.com In-Reply-To: ``` I'm seeing test failures non-executable tests: t9825-git-p4-handle-utf16-without-bom.sh ls -l shows that all the other tests are executable but t9825 isn't. On Tue, Sep 22, 2015 at 9:02 AM, Junio C Hamano wrote: > Lars Schneider writes: > >> This works. > > OK, and thanks; as I don't do perforce, the squash was without any > testing. > >> Do we need the “-e” option? > > In syntactic sense, no, but our codebase tends to prefer to have > one, because it is easier to spot which ones are the instructions if > you consistently have "-e" even when you give only one. > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ``` ## Junio C Hamano, 2015-09-22 19:17 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: URL: https://gitlist.dev/e/CAPc5daXm9sBGAgrqz12d5a%3DzhR3PUXbFpPvOkBCoNQcQVhyOhw%40mail.gmail.com In-Reply-To: ``` Yup, this was privately reported and I just squashed a fix in right now ;-) Thanks. "cd t && make test-lint" would have caught it. On Tue, Sep 22, 2015 at 12:11 PM, Michael Blume wrote: > I'm seeing test failures > > non-executable tests: t9825-git-p4-handle-utf16-without-bom.sh > > ls -l shows that all the other tests are executable but t9825 isn't. > > On Tue, Sep 22, 2015 at 9:02 AM, Junio C Hamano wrote: >> Lars Schneider writes: >> >>> This works. >> >> OK, and thanks; as I don't do perforce, the squash was without any >> testing. >> >>> Do we need the “-e” option? >> >> In syntactic sense, no, but our codebase tends to prefer to have >> one, because it is easier to spot which ones are the instructions if >> you consistently have "-e" even when you give only one. >> -- >> To unsubscribe from this list: send the line "unsubscribe git" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html ``` ## Lars Schneider, 2015-09-23 07:34 Subject: Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error Message-ID: <3E330347-0F89-4E6F-8663-694AD3A559CC@gmail.com> URL: https://gitlist.dev/e/3E330347-0F89-4E6F-8663-694AD3A559CC%40gmail.com In-Reply-To: ``` Thanks a lot for taking care of this! - Lars On 22 Sep 2015, at 21:17, Junio C Hamano wrote: > Yup, this was privately reported and I just squashed a fix in right now ;-) > > Thanks. "cd t && make test-lint" would have caught it. > > On Tue, Sep 22, 2015 at 12:11 PM, Michael Blume wrote: >> I'm seeing test failures >> >> non-executable tests: t9825-git-p4-handle-utf16-without-bom.sh >> >> ls -l shows that all the other tests are executable but t9825 isn't. >> >> On Tue, Sep 22, 2015 at 9:02 AM, Junio C Hamano wrote: >>> Lars Schneider writes: >>> >>>> This works. >>> >>> OK, and thanks; as I don't do perforce, the squash was without any >>> testing. >>> >>>> Do we need the “-e” option? >>> >>> In syntactic sense, no, but our codebase tends to prefer to have >>> one, because it is easier to spot which ones are the instructions if >>> you consistently have "-e" even when you give only one. >>> -- >>> To unsubscribe from this list: send the line "unsubscribe git" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html ```