]> git.ipfire.org Git - thirdparty/git.git/commitdiff
git-p4: add RCS keyword status message
authorBen Keene <seraphire@gmail.com>
Fri, 14 Feb 2020 14:44:46 +0000 (14:44 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 14 Feb 2020 16:58:53 +0000 (08:58 -0800)
During the p4 submit process, git-p4 will attempt to apply a patch
to the files found in the p4 workspace. However, if P4 uses RCS
keyword expansion, this patch may fail.

When the patch fails, the user is alerted to the failure and that git-p4
will attempt to clear the expanded text from the files and re-apply
the patch. The current version of git-p4 does not tell the user the
result of the re-apply attempt after the RCS expansion has been removed
which can be confusing.

Add a new print statement after the git patch has been successfully
applied when the RCS keywords have been cleansed.

Signed-off-by: Ben Keene <seraphire@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-p4.py

index 8761c70e1c613acb9855b3f3ec01f9ba873695a3..258b9b98b93f92632d2079ecf4b2a321e675cb08 100755 (executable)
--- a/git-p4.py
+++ b/git-p4.py
@@ -2025,6 +2025,9 @@ class P4Submit(Command, P4UserMap):
         applyPatchCmd = patchcmd + "--check --apply -"
         patch_succeeded = True
 
+        if verbose:
+            print("TryPatch: %s" % tryPatchCmd)
+
         if os.system(tryPatchCmd) != 0:
             fixed_rcs_keywords = False
             patch_succeeded = False
@@ -2064,6 +2067,7 @@ class P4Submit(Command, P4UserMap):
                 print("Retrying the patch with RCS keywords cleaned up")
                 if os.system(tryPatchCmd) == 0:
                     patch_succeeded = True
+                    print("Patch succeesed this time with RCS keywords cleaned")
 
         if not patch_succeeded:
             for f in editedFiles: