[MediaWiki-commits] [Gerrit] Remove quiet option on Android tests - change (integration/config)

2015-09-10 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged.

Change subject: Remove quiet option on Android tests
..


Remove quiet option on Android tests

When all tests perform well, Gradle's -q option is great to eliminate
unnecessary noise and shave a second or two off build times.
Unfortunately, tests fail and we need to know why quickly. This patch
removes the quiet option from builds, restoring the default verbosity
level.

For example, prior to this patch, the complete output for a ProGuard
failure was:

  FAILURE: Build failed with an exception.

  * What went wrong:
  Execution failed for task ':app:proguardAlphaReleaseAndroidTest'.
  > java.io.IOException: Please correct the above warnings first.

  * Try:
  Run with --stacktrace option to get the stack trace. Run with --info or 
--debug option to get more log output.

After this patch, the failure will show that it is due to ProGuard
warnings and what those warnings are.

Change-Id: Ie0805774064b47bbd516094dee84a1075a93691d
---
M jjb/mobile.yaml
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mholloway: Looks good to me, but someone else must approve
  Hashar: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/jjb/mobile.yaml b/jjb/mobile.yaml
index cc35ed1..ef2b8e6 100644
--- a/jjb/mobile.yaml
+++ b/jjb/mobile.yaml
@@ -36,7 +36,7 @@
 builders:
  - shell: |
  scripts/missing-qq.py
- ./gradlew -q clean checkstyle testAllAlphaRelease
+ ./gradlew clean checkstyle testAllAlphaRelease
 publishers:
  - archive:
  # Capture generated .apk, ProGuard mappings, checkstyle.xml, and test 
results

-- 
To view, visit https://gerrit.wikimedia.org/r/237133
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie0805774064b47bbd516094dee84a1075a93691d
Gerrit-PatchSet: 2
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Niedzielski 
Gerrit-Reviewer: BearND 
Gerrit-Reviewer: Dbrant 
Gerrit-Reviewer: Hashar 
Gerrit-Reviewer: Mholloway 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] Remove quiet option on Android tests - change (integration/config)

2015-09-09 Thread Niedzielski (Code Review) 
Niedzielski has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/237133

Change subject: Remove quiet option on Android tests
..

Remove quiet option on Android tests

When all tests perform well, Gradle's -q option is great to eliminate
unnecessary noise and shave a second or two off build times.
Unfortunately, tests fail and we need to know why quickly. This patch
removes the quiet option from builds, restoring the default verbosity
level.

For example, prior to this patch, the complete output for a ProGuard
failure was:

  FAILURE: Build failed with an exception.

  * What went wrong:
  Execution failed for task ':app:proguardAlphaReleaseAndroidTest'.
  > java.io.IOException: Please correct the above warnings first.

  * Try:
  Run with --stacktrace option to get the stack trace. Run with --info or 
--debug option to get more log output.

After this patch, the failure will show that it is due to ProGuard
warnings and what those warnings are.

Change-Id: Ie0805774064b47bbd516094dee84a1075a93691d
---
M jjb/mobile.yaml
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/integration/config 
refs/changes/33/237133/1

diff --git a/jjb/mobile.yaml b/jjb/mobile.yaml
index cc35ed1..ef2b8e6 100644
--- a/jjb/mobile.yaml
+++ b/jjb/mobile.yaml
@@ -36,7 +36,7 @@
 builders:
  - shell: |
  scripts/missing-qq.py
- ./gradlew -q clean checkstyle testAllAlphaRelease
+ ./gradlew clean checkstyle testAllAlphaRelease
 publishers:
  - archive:
  # Capture generated .apk, ProGuard mappings, checkstyle.xml, and test 
results

-- 
To view, visit https://gerrit.wikimedia.org/r/237133
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie0805774064b47bbd516094dee84a1075a93691d
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Niedzielski 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits