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/26 16:48:27 UTC
[jira] [Updated] (YETUS-132) javadoc test does not run if javac
test is disabled
[ https://issues.apache.org/jira/browse/YETUS-132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kengo Seki updated YETUS-132:
-----------------------------
Description:
I found it trying YETUS-129.00.patch. In this patch, Geode personality turns off only javac test, but javadoc test is also skipped.
{code}
[sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../incubator-geode --project=geode --resetrepo 22
(snip)
============================================================================
============================================================================
Pre-patch javadoc verification on develop
============================================================================
============================================================================
============================================================================
============================================================================
Applying patch to develop
============================================================================
============================================================================
Applying the patch:
Tue Oct 27 00:41:29 JST 2015
cd /Users/sekikn/incubator-geode
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-27531.22860/patch
Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/cache/EntryOperation.java cleanly.
Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/EntryOperationImpl.java cleanly.
.../com/gemstone/gemfire/cache/EntryOperation.java | 2 +-
.../gemfire/internal/cache/EntryOperationImpl.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
(snip)
============================================================================
============================================================================
Patch javadoc verification
============================================================================
============================================================================
============================================================================
============================================================================
Determining number of patched ASF License errors
============================================================================
============================================================================
(snip)
| Vote | Subsystem | Runtime | Comment
============================================================================
| -1 | @author | 0m 00s | The patch appears to contain 2 @author
| | | | tags which the community has agreed to
| | | | not allow in code contributions.
| -1 | test4tests | 0m 00s | The patch doesn't appear to include any
| | | | new or modified tests. Please justify why
| | | | no new tests are needed for this patch.
| | | | Also please list what manual steps were
| | | | performed to verify this patch.
| +1 | gradleboot | 0m 06s | develop passed
| +1 | gradleboot | 0m 04s | the patch passed
| +1 | whitespace | 0m 00s | Patch has no whitespace issues.
| 0 | asflicense | 0m 05s | ASF License check generated no output?
| | | 0m 32s |
|| Subsystem || Report/Notes ||
============================================================================
| GITHUB PR | https://github.com/apache/incubator-geode/pull/22 |
| Optional Tests | asflicense javadoc unit findbugs |
| 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 | gradle |
| Personality | /Users/sekikn/yetus/dev-support/personality/geode.sh |
| git revision | develop / 7909836 |
| findbugs | not supported by the gradle plugin |
| Powered by | Apache Yetus http://yetus.apache.org |
============================================================================
============================================================================
Finished build.
============================================================================
============================================================================
{code}
This is because $\{JDK_DIR_LIST} is initialised in javac_initialize.
was:
I found it trying YETUS-129.00.patch. In this patch, Geode personality turns off only javac test, but javadoc test is also skipped.
{code}
[sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../incubator-geode --project=geode --resetrepo 22
(snip)
============================================================================
============================================================================
Pre-patch javadoc verification on develop
============================================================================
============================================================================
============================================================================
============================================================================
Applying patch to develop
============================================================================
============================================================================
Applying the patch:
Tue Oct 27 00:41:29 JST 2015
cd /Users/sekikn/incubator-geode
git apply --binary -v --stat --apply -p1 /private/tmp/yetus-27531.22860/patch
Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/cache/EntryOperation.java cleanly.
Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/EntryOperationImpl.java cleanly.
.../com/gemstone/gemfire/cache/EntryOperation.java | 2 +-
.../gemfire/internal/cache/EntryOperationImpl.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
(snip)
============================================================================
============================================================================
Patch javadoc verification
============================================================================
============================================================================
(snip)
| Vote | Subsystem | Runtime | Comment
============================================================================
| -1 | @author | 0m 00s | The patch appears to contain 2 @author
| | | | tags which the community has agreed to
| | | | not allow in code contributions.
| -1 | test4tests | 0m 00s | The patch doesn't appear to include any
| | | | new or modified tests. Please justify why
| | | | no new tests are needed for this patch.
| | | | Also please list what manual steps were
| | | | performed to verify this patch.
| +1 | gradleboot | 0m 06s | develop passed
| +1 | gradleboot | 0m 04s | the patch passed
| +1 | whitespace | 0m 00s | Patch has no whitespace issues.
| 0 | asflicense | 0m 05s | ASF License check generated no output?
| | | 0m 32s |
|| Subsystem || Report/Notes ||
============================================================================
| GITHUB PR | https://github.com/apache/incubator-geode/pull/22 |
| Optional Tests | asflicense javadoc unit findbugs |
| 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 | gradle |
| Personality | /Users/sekikn/yetus/dev-support/personality/geode.sh |
| git revision | develop / 7909836 |
| findbugs | not supported by the gradle plugin |
| Powered by | Apache Yetus http://yetus.apache.org |
============================================================================
============================================================================
Finished build.
============================================================================
============================================================================
{code}
This is because $\{JDK_DIR_LIST} is initialised in javac_initialize.
> javadoc test does not run if javac test is disabled
> ---------------------------------------------------
>
> Key: YETUS-132
> URL: https://issues.apache.org/jira/browse/YETUS-132
> Project: Yetus
> Issue Type: Bug
> Components: Test Patch
> Reporter: Kengo Seki
>
> I found it trying YETUS-129.00.patch. In this patch, Geode personality turns off only javac test, but javadoc test is also skipped.
> {code}
> [sekikn@mobile yetus]$ dev-support/test-patch.sh --basedir=../incubator-geode --project=geode --resetrepo 22
> (snip)
> ============================================================================
> ============================================================================
> Pre-patch javadoc verification on develop
> ============================================================================
> ============================================================================
> ============================================================================
> ============================================================================
> Applying patch to develop
> ============================================================================
> ============================================================================
> Applying the patch:
> Tue Oct 27 00:41:29 JST 2015
> cd /Users/sekikn/incubator-geode
> git apply --binary -v --stat --apply -p1 /private/tmp/yetus-27531.22860/patch
> Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/cache/EntryOperation.java cleanly.
> Applied patch gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/EntryOperationImpl.java cleanly.
> .../com/gemstone/gemfire/cache/EntryOperation.java | 2 +-
> .../gemfire/internal/cache/EntryOperationImpl.java | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
> (snip)
> ============================================================================
> ============================================================================
> Patch javadoc verification
> ============================================================================
> ============================================================================
> ============================================================================
> ============================================================================
> Determining number of patched ASF License errors
> ============================================================================
> ============================================================================
> (snip)
> | Vote | Subsystem | Runtime | Comment
> ============================================================================
> | -1 | @author | 0m 00s | The patch appears to contain 2 @author
> | | | | tags which the community has agreed to
> | | | | not allow in code contributions.
> | -1 | test4tests | 0m 00s | The patch doesn't appear to include any
> | | | | new or modified tests. Please justify why
> | | | | no new tests are needed for this patch.
> | | | | Also please list what manual steps were
> | | | | performed to verify this patch.
> | +1 | gradleboot | 0m 06s | develop passed
> | +1 | gradleboot | 0m 04s | the patch passed
> | +1 | whitespace | 0m 00s | Patch has no whitespace issues.
> | 0 | asflicense | 0m 05s | ASF License check generated no output?
> | | | 0m 32s |
> || Subsystem || Report/Notes ||
> ============================================================================
> | GITHUB PR | https://github.com/apache/incubator-geode/pull/22 |
> | Optional Tests | asflicense javadoc unit findbugs |
> | 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 | gradle |
> | Personality | /Users/sekikn/yetus/dev-support/personality/geode.sh |
> | git revision | develop / 7909836 |
> | findbugs | not supported by the gradle plugin |
> | Powered by | Apache Yetus http://yetus.apache.org |
> ============================================================================
> ============================================================================
> Finished build.
> ============================================================================
> ============================================================================
> {code}
> This is because $\{JDK_DIR_LIST} is initialised in javac_initialize.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)