You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Kengo Seki (JIRA)" <ji...@apache.org> on 2015/10/17 08:17:05 UTC

[jira] [Updated] (YETUS-57) empty patch_dir if not in re-exec mode?

     [ https://issues.apache.org/jira/browse/YETUS-57?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kengo Seki updated YETUS-57:
----------------------------
    Attachment: YETUS-57.00.patch

Attaching a patch.

{code}
[sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../hadoop --patch-dir=/Users/sekikn/p --plugins=all --project=hadoop --resetrepo HADOOP-12070 # first time
/Users/sekikn/p has been created

(snip)

Applying the patch:
Sat Oct 17 15:08:26 JST 2015
cd /Users/sekikn/hadoop
git apply --binary -v --stat --apply -p0 /Users/sekikn/p/patch
Applied patch hadoop-common-project/hadoop-common/src/main/bin/hadoop.cmd cleanly.
 .../hadoop-common/src/main/bin/hadoop.cmd          |   26 ++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

(snip)

[sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../hadoop --patch-dir=/Users/sekikn/p --plugins=all --project=hadoop --resetrepo HADOOP-12070 # second time

(snip)

Applying the patch:
Sat Oct 17 15:08:26 JST 2015
cd /Users/sekikn/hadoop
git apply --binary -v --stat --apply -p0 /Users/sekikn/p/patch
Applied patch hadoop-common-project/hadoop-common/src/main/bin/hadoop.cmd cleanly.
 .../hadoop-common/src/main/bin/hadoop.cmd          |   26 ++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)
Sat Oct 17 15:09:07 JST 2015
cd /Users/sekikn/hadoop
git apply --binary -v --stat --apply -p0 /Users/sekikn/p/patch
Applied patch hadoop-common-project/hadoop-common/src/main/bin/hadoop.cmd cleanly.
 .../hadoop-common/src/main/bin/hadoop.cmd          |   26 ++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

(snip)

[sekikn@mobile yetus]$ dev-support/smart-apply-patch.sh ~/YETUS-57.00.patch 
Processing: /Users/sekikn/YETUS-57.00.patch
Patch file /Users/sekikn/YETUS-57.00.patch copied to /tmp/yetus-28695.995
Applying the patch:
Sat Oct 17 15:10:48 JST 2015
cd /Users/sekikn/yetus
git apply --binary -v --stat --apply -p1 /tmp/yetus-28695.995/patch
Applied patch dev-support/test-patch.sh cleanly.
 dev-support/test-patch.sh |    8 ++++++++
 1 file changed, 8 insertions(+)
[sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../hadoop --patch-dir=/Users/sekikn/p --plugins=all --project=hadoop --resetrepo HADOOP-12070
Removing contents in /Users/sekikn/p

(snip)

Applying the patch:
Sat Oct 17 15:11:18 JST 2015
cd /Users/sekikn/hadoop
git apply --binary -v --stat --apply -p0 /Users/sekikn/p/patch
Applied patch hadoop-common-project/hadoop-common/src/main/bin/hadoop.cmd cleanly.
 .../hadoop-common/src/main/bin/hadoop.cmd          |   26 ++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

(snip)
{code}

> empty patch_dir if not in re-exec mode?
> ---------------------------------------
>
>                 Key: YETUS-57
>                 URL: https://issues.apache.org/jira/browse/YETUS-57
>             Project: Yetus
>          Issue Type: Bug
>          Components: Test Patch
>            Reporter: Allen Wittenauer
>         Attachments: YETUS-57.00.patch
>
>
> test-patch behaves in surprising ways if the patch processing directory isn't empty on execution.  This should probably be fixed, keeping in mind that reexec mode will place some things there.  We should probably check if we are re-execing and if not, either error or clean it out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)