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/11/02 11:32:27 UTC

[jira] [Commented] (YETUS-115) test-patch fails to find relative path

    [ https://issues.apache.org/jira/browse/YETUS-115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14985027#comment-14985027 ] 

Kengo Seki commented on YETUS-115:
----------------------------------

Is this already resolved on master? I couldn't reproduce it.

{code}
[sekikn@mobile yetus]$ precommit/test-patch.sh --build-tool=nobuild --plugins=all --resetrepo ../../../tmp/YETUS-78.02.patch 
/tmp/yetus-26809.25340 has been created
Running in destructive (--resetrepo) developer mode
Processing: ../../../tmp/YETUS-78.02.patch
Patch file ../../../tmp/YETUS-78.02.patch copied to /private/tmp/yetus-26809.25340


============================================================================
============================================================================
                        Confirming git environment
============================================================================
============================================================================


HEAD is now at 06c76a1 YETUS-138. Skipping maven poms that don't package causes testing holes
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Current branch master is up to date.
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Current branch master is up to date.
Testing  patch on master.


============================================================================
============================================================================
                         precommit patch detected
============================================================================
============================================================================




============================================================================
============================================================================
                         Applying patch to master
============================================================================
============================================================================


Applying the patch:
Mon Nov  2 19:28:44 JST 2015
cd /Users/sekikn/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-26809.25340/patch
Applied patch precommit/personality/orc.sh cleanly.
 precommit/personality/orc.sh |   29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)
Running in destructive (--resetrepo) developer mode
Processing: ../../../tmp/YETUS-78.02.patch


============================================================================
============================================================================
                        Confirming git environment
============================================================================
============================================================================


HEAD is now at 06c76a1 YETUS-138. Skipping maven poms that don't package causes testing holes
Removing precommit/personality/orc.sh
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Current branch master is up to date.
Already on 'master'
Your branch is up-to-date with 'origin/master'.
Current branch master is up to date.
Testing  patch on master.


============================================================================
============================================================================
                    Re-exec mode detected. Continuing.
============================================================================
============================================================================




============================================================================
============================================================================
             Checking there are no @author tags in the patch.
============================================================================
============================================================================


There appear to be 0 @author tags in the patch.


============================================================================
============================================================================
           Checking there are new or changed tests in the patch.
============================================================================
============================================================================


Patch does not appear to need new or modified tests.


============================================================================
============================================================================
                        shellcheck plugin: prepatch
============================================================================
============================================================================


Running shellcheck against all identifiable shell scripts


============================================================================
============================================================================
                         Applying patch to master
============================================================================
============================================================================


Applying the patch:
Mon Nov  2 19:28:44 JST 2015
cd /Users/sekikn/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-26809.25340/patch
Applied patch precommit/personality/orc.sh cleanly.
 precommit/personality/orc.sh |   29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)
Mon Nov  2 19:28:52 JST 2015
cd /Users/sekikn/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-26809.25340/patch
Applied patch precommit/personality/orc.sh cleanly.
 precommit/personality/orc.sh |   29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)


============================================================================
============================================================================
                       shellcheck plugin: postpatch
============================================================================
============================================================================


Running shellcheck against all identifiable shell scripts


============================================================================
============================================================================
                Checking for whitespace at the end of lines
============================================================================
============================================================================




============================================================================
============================================================================
             Determining number of patched ASF License errors
============================================================================
============================================================================



Total Elapsed time:   0m 15s



+1 overall

 ____                              _ 
/ ___| _   _  ___ ___ ___  ___ ___| |
\___ \| | | |/ __/ __/ _ \/ __/ __| |
 ___) | |_| | (_| (_|  __/\__ \__ \_|
|____/ \__,_|\___\___\___||___/___(_)
                                     


| Vote |      Subsystem |  Runtime   | Comment
============================================================================
|   0  |        reexec  |  0m 07s    | precommit patch detected. 
|  +1  |       @author  |  0m 00s    | The patch does not contain any @author 
|      |                |            | tags.
|  +1  |    shellcheck  |  0m 00s    | There were no new shellcheck issues. 
|  +1  |    whitespace  |  0m 00s    | Patch has no whitespace issues. 
|      |                |  0m 15s    | 


|| Subsystem || Report/Notes ||
============================================================================
| Optional Tests |  asflicense  shellcheck  |
| uname | Darwin mobile.local 15.0.0 Darwin Kernel Version 15.0.0: Sat Sep 19 15:53:46 PDT 2015; root:xnu-3247.10.11~1/RELEASE_X86_64 x86_64 |
| Build tool | nobuild |
| git revision | master / 06c76a1 |
| Default Java | 1.7.0_80 |
| shellcheck | v0.4.1 |
| modules | C:  U:  |
| Powered by | Apache Yetus   http://yetus.apache.org |


============================================================================
============================================================================
                              Finished build.
============================================================================
============================================================================
{code}

> test-patch fails to find relative path
> --------------------------------------
>
>                 Key: YETUS-115
>                 URL: https://issues.apache.org/jira/browse/YETUS-115
>             Project: Yetus
>          Issue Type: Bug
>          Components: Test Patch
>    Affects Versions: 0.1.0
>            Reporter: Sean Busbey
>
> At least in non-docker reexec mode, attempting to feed ourself a relative path fails, while giving a path resolved by the shell succeeds. example runs to follow in comments.



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