You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Sean Busbey (JIRA)" <ji...@apache.org> on 2015/10/21 10:41: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=14966451#comment-14966451 ]
Sean Busbey commented on YETUS-115:
-----------------------------------
relative path failure:
{code}
yetus busbey$ ./dev-support/test-patch.sh --plugins="all" --build-tool=nobuild --resetrepo ../patches/YETUS-78.00.patch
/tmp/yetus-2412.13981 has been created
Running in destructive (--resetrepo) developer mode
Processing: ../patches/YETUS-78.00.patch
Patch file ../patches/YETUS-78.00.patch copied to /private/tmp/yetus-2412.13981
============================================================================
============================================================================
Confirming git environment
============================================================================
============================================================================
HEAD is now at 23c6253 YETUS-114 add no-op build tool
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
Current branch master is up to date.
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
Current branch master is up to date.
Testing patch on master.
============================================================================
============================================================================
precommit patch detected
============================================================================
============================================================================
============================================================================
============================================================================
Applying patch to master
============================================================================
============================================================================
Applying the patch:
Wed Oct 21 03:36:16 CDT 2015
cd /Users/busbey/projects/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-2412.13981/patch
/private/tmp/yetus-2412.13981/patch:41: new blank line at EOF.
+
Applied patch dev-support/personality/orc.sh cleanly.
warning: 1 line adds whitespace errors.
dev-support/personality/orc.sh | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
Running in destructive (--resetrepo) developer mode
Processing: ../patches/YETUS-78.00.patch
ERROR: Unsure how to process ../patches/YETUS-78.00.patch.
============================================================================
============================================================================
Finished build.
============================================================================
============================================================================
{code}
Giving a path resolved by the shell to the same file works:
{code}
yetus busbey$ ./dev-support/test-patch.sh --plugins="all" --build-tool=nobuild --resetrepo ~/projects/patches/YETUS-78.00.patch
/tmp/yetus-19386.20329 has been created
Running in destructive (--resetrepo) developer mode
Processing: /Users/busbey/projects/patches/YETUS-78.00.patch
Patch file /Users/busbey/projects/patches/YETUS-78.00.patch copied to /private/tmp/yetus-19386.20329
============================================================================
============================================================================
Confirming git environment
============================================================================
============================================================================
HEAD is now at 23c6253 YETUS-114 add no-op build tool
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
Current branch master is up to date.
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
Current branch master is up to date.
Testing patch on master.
============================================================================
============================================================================
precommit patch detected
============================================================================
============================================================================
============================================================================
============================================================================
Applying patch to master
============================================================================
============================================================================
Applying the patch:
Wed Oct 21 03:38:08 CDT 2015
cd /Users/busbey/projects/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-19386.20329/patch
/private/tmp/yetus-19386.20329/patch:41: new blank line at EOF.
+
Applied patch dev-support/personality/orc.sh cleanly.
warning: 1 line adds whitespace errors.
dev-support/personality/orc.sh | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
Running in destructive (--resetrepo) developer mode
Processing: /Users/busbey/projects/patches/YETUS-78.00.patch
============================================================================
============================================================================
Confirming git environment
============================================================================
============================================================================
HEAD is now at 23c6253 YETUS-114 add no-op build tool
Removing dev-support/personality/orc.sh
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
Current branch master is up to date.
Already on 'master'
Your branch is ahead of 'busbey/master' by 29 commits.
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:
Wed Oct 21 03:38:08 CDT 2015
cd /Users/busbey/projects/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-19386.20329/patch
/private/tmp/yetus-19386.20329/patch:41: new blank line at EOF.
+
Applied patch dev-support/personality/orc.sh cleanly.
warning: 1 line adds whitespace errors.
dev-support/personality/orc.sh | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
Wed Oct 21 03:38:10 CDT 2015
cd /Users/busbey/projects/yetus
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-19386.20329/patch
/private/tmp/yetus-19386.20329/patch:41: new blank line at EOF.
+
Applied patch dev-support/personality/orc.sh cleanly.
warning: 1 line adds whitespace errors.
dev-support/personality/orc.sh | 26 ++++++++++++++++++++++++++
1 file changed, 26 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 05s
+1 overall
____ _
/ ___| _ _ ___ ___ ___ ___ ___| |
\___ \| | | |/ __/ __/ _ \/ __/ __| |
___) | |_| | (_| (_| __/\__ \__ \_|
|____/ \__,_|\___\___\___||___/___(_)
| Vote | Subsystem | Runtime | Comment
============================================================================
| 0 | reexec | 0m 02s | 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 05s |
|| Subsystem || Report/Notes ||
============================================================================
| Optional Tests | asflicense shellcheck |
| uname | Darwin busbey2-MBA.local 12.6.0 Darwin Kernel Version 12.6.0: Wed Mar 18 16:23:48 PDT 2015; root:xnu-2050.48.19~1/RELEASE_X86_64 x86_64 |
| Build tool | nobuild |
| git revision | master / 23c6253 |
| Default Java | 1.7.0_51 |
| shellcheck | v0.3.7 |
| 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)