From 55f2b7defa934f006ef7160745654b4438d3aeeb Mon Sep 17 00:00:00 2001
From: liym27 <33742067+liym27@users.noreply.github.com>
Date: Tue, 10 Dec 2019 09:28:08 +0800
Subject: [PATCH] Polish CI error message. test=develop (#21642)

---
 tools/check_api_approvals.sh | 4 +---
 tools/check_op_desc.py       | 1 -
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/tools/check_api_approvals.sh b/tools/check_api_approvals.sh
index 3c70dbb6c8..782d45635c 100644
--- a/tools/check_api_approvals.sh
+++ b/tools/check_api_approvals.sh
@@ -75,7 +75,7 @@ fi
 
 op_desc_diff=`python ${PADDLE_ROOT}/tools/check_op_desc.py ${PADDLE_ROOT}/paddle/fluid/OP_DESC_DEV.spec  ${PADDLE_ROOT}/paddle/fluid/OP_DESC_PR.spec`
 if [ "$op_desc_diff" != "" ]; then
-    echo_line="You must have one RD (liym27 (Recommend), zhhsplendid, Aurelius84, lanxianghit or phlrain) approval for the changes of Inputs/Output/Attrs of OPs. The changes of OPs will cause that the new version inference fails to load model trained by the old version.\n"
+    echo_line="You must have one RD (liym27 (Recommend), zhhsplendid, Aurelius84, lanxianghit or phlrain) approval for the changes of Inputs/Output/Attrs of OPs. The changes of OPs will cause that the new version inference fails to load model trained by the old version. Please modify your code. \n For more details, please click [https://github.com/PaddlePaddle/Paddle/wiki/OP-Input-Output-Attribute-Compatibility-Modification].\n${op_desc_diff}\n"
     check_approval 1 33742067 7913861 9301846 47554610 43953930
 fi
 
@@ -173,8 +173,6 @@ fi
 
 python ${PADDLE_ROOT}/tools/diff_api.py ${PADDLE_ROOT}/paddle/fluid/API_DEV.spec  ${PADDLE_ROOT}/paddle/fluid/API_PR.spec
 python ${PADDLE_ROOT}/tools/check_op_register_type.py ${PADDLE_ROOT}/paddle/fluid/OP_TYPE_DEV.spec  ${PADDLE_ROOT}/paddle/fluid/OP_TYPE_PR.spec
-python ${PADDLE_ROOT}/tools/check_op_desc.py ${PADDLE_ROOT}/paddle/fluid/OP_DESC_DEV.spec  ${PADDLE_ROOT}/paddle/fluid/OP_DESC_PR.spec
-
 if [ -n "${echo_list}" ]; then
   exit 1
 fi
diff --git a/tools/check_op_desc.py b/tools/check_op_desc.py
index ae3e3cafda..465d7cf02d 100644
--- a/tools/check_op_desc.py
+++ b/tools/check_op_desc.py
@@ -245,7 +245,6 @@ if len(sys.argv) == 3:
     if error:
         print("-" * 30)
         print_error_message(error_message)
-        print_repeat_process()
         print("-" * 30)
 else:
     print("Usage: python check_op_desc.py OP_DESC_DEV.spec OP_DESC_PR.spec")
-- 
GitLab