You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@yetus.apache.org by aw...@apache.org on 2015/10/29 16:02:48 UTC
[1/7] yetus git commit: YETUS-143. add support for JMeter
Repository: yetus
Updated Branches:
refs/heads/YETUS-83 67f42f1ae -> c3a206998 (forced update)
YETUS-143. add support for JMeter
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/2b40ae27
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/2b40ae27
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/2b40ae27
Branch: refs/heads/YETUS-83
Commit: 2b40ae27010db83ca885b698206148bf4b79491e
Parents: 2138831
Author: Kengo Seki <se...@gmail.com>
Authored: Wed Oct 28 23:19:27 2015 +0900
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Wed Oct 28 10:00:06 2015 -0700
----------------------------------------------------------------------
precommit/personality/jmeter.sh | 59 ++++++++++++++++++++++++++++++++++++
1 file changed, 59 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/2b40ae27/precommit/personality/jmeter.sh
----------------------------------------------------------------------
diff --git a/precommit/personality/jmeter.sh b/precommit/personality/jmeter.sh
new file mode 100755
index 0000000..0af744c
--- /dev/null
+++ b/precommit/personality/jmeter.sh
@@ -0,0 +1,59 @@
+#!/usr/bin/env bash
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+personality_plugins "all,-javadoc,-findbugs,-asflicense"
+
+function personality_globals
+{
+ # shellcheck disable=SC2034
+ PATCH_BRANCH_DEFAULT=trunk
+ # shellcheck disable=SC2034
+ BUILDTOOL=ant
+ # shellcheck disable=SC2034
+ GITHUB_REPO="apache/jmeter"
+ # shellcheck disable=SC2034
+ JMETER_DOWNLOAD_JARS=false
+}
+
+add_test_type jmeter
+
+function jmeter_usage
+{
+ echo "JMeter specific:"
+ echo "--jmeter-download-jars=[false|true] download third-party jars needed by ant build"
+}
+
+function jmeter_parse_args
+{
+ declare i
+
+ for i in "$@"; do
+ case ${i} in
+ --jmeter-download-jars=*)
+ JMETER_DOWNLOAD_JARS=${i#*=}
+ ;;
+ esac
+ done
+}
+
+function jmeter_precheck
+{
+ if [[ ${JMETER_DOWNLOAD_JARS} = true ]]; then
+ pushd "${BASEDIR}" >/dev/null
+ echo_and_redirect "${PATCH_DIR}/jmeter-branch-download-jars.txt" "${ANT}" download_jars
+ popd >/dev/null
+ fi
+}
[2/7] yetus git commit: YETUS-141 make audience annotation jdiff
doclet optional.
Posted by aw...@apache.org.
YETUS-141 make audience annotation jdiff doclet optional.
- jdiff module is only included when profile 'include-jdiff-module' is active
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/1bab736c
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/1bab736c
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/1bab736c
Branch: refs/heads/YETUS-83
Commit: 1bab736c81f501be9992da37e15b31dd774f9e00
Parents: 2b40ae2
Author: Sean Busbey <bu...@apache.org>
Authored: Sat Oct 24 01:43:13 2015 -0500
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Wed Oct 28 10:08:49 2015 -0700
----------------------------------------------------------------------
audience-annotations-component/pom.xml | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/1bab736c/audience-annotations-component/pom.xml
----------------------------------------------------------------------
diff --git a/audience-annotations-component/pom.xml b/audience-annotations-component/pom.xml
index c187de2..76bde1a 100644
--- a/audience-annotations-component/pom.xml
+++ b/audience-annotations-component/pom.xml
@@ -35,6 +35,13 @@
<modules>
<module>audience-annotations</module>
- <module>audience-annotations-jdiff</module>
</modules>
+ <profiles>
+ <profile>
+ <id>include-jdiff-module</id>
+ <modules>
+ <module>audience-annotations-jdiff</module>
+ </modules>
+ </profile>
+ </profiles>
</project>
[4/7] yetus git commit: YETUS-93. tap needs to verify directories
Posted by aw...@apache.org.
YETUS-93. tap needs to verify directories
Signed-off-by: Sean Busbey <bu...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/09ae5616
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/09ae5616
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/09ae5616
Branch: refs/heads/YETUS-83
Commit: 09ae5616ab249ffa8dbbadc18a2cbdbf2d830cec
Parents: d45ae41
Author: Allen Wittenauer <aw...@apache.org>
Authored: Mon Oct 19 10:54:40 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Oct 29 07:54:44 2015 -0700
----------------------------------------------------------------------
precommit/test-patch.d/tap.sh | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/09ae5616/precommit/test-patch.d/tap.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/tap.sh b/precommit/test-patch.d/tap.sh
index 7ced908..8de68f1 100755
--- a/precommit/test-patch.d/tap.sh
+++ b/precommit/test-patch.d/tap.sh
@@ -19,7 +19,7 @@ add_test_format tap
TAP_FAILED_TESTS=""
TAP_LOG_DIR="target/tap"
-function tap_process_args
+function tap_parse_args
{
declare i
@@ -49,7 +49,9 @@ function tap_process_tests
declare module_failed_tests
declare filenames
- filenames=$(find "${TAP_LOG_DIR}" -type f -exec "${GREP}" -l -E "^not ok" {} \;)
+ if [[ -d "${TAP_LOG_DIR}" ]]; then
+ filenames=$(find "${TAP_LOG_DIR}" -type f -exec "${GREP}" -l -E "^not ok" {} \;)
+ fi
if [[ -n "${filenames}" ]]; then
module_failed_tests=$(echo "${filenames}" \
[5/7] yetus git commit: YETUS-94. docker cleanup invalid date
Posted by aw...@apache.org.
YETUS-94. docker cleanup invalid date
Signed-off-by: Sean Busbey <bu...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/d1a27cc4
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/d1a27cc4
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/d1a27cc4
Branch: refs/heads/YETUS-83
Commit: d1a27cc41ed093240d790a6f37beb24a9a8e00da
Parents: 09ae561
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Oct 20 16:55:02 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Oct 29 07:55:47 2015 -0700
----------------------------------------------------------------------
.../test-patch-docker/test-patch-docker.sh | 34 +++++++++++---------
1 file changed, 19 insertions(+), 15 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/d1a27cc4/precommit/test-patch-docker/test-patch-docker.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch-docker/test-patch-docker.sh b/precommit/test-patch-docker/test-patch-docker.sh
index 8b19502..ca910ff 100755
--- a/precommit/test-patch-docker/test-patch-docker.sh
+++ b/precommit/test-patch-docker/test-patch-docker.sh
@@ -94,7 +94,7 @@ function stop_exited_containers
# stop *all* containers that are in exit state for
# more than > 8 hours
- while read line; do
+ while read -r line; do
id=$(echo "${line}" | cut -f1 -d' ')
value=$(echo "${line}" | cut -f2 -d' ')
size=$(echo "${line}" | cut -f3 -d' ')
@@ -127,16 +127,22 @@ function stop_exited_containers
## @param args
function rm_old_containers
{
- local line
- local id
- local value
- local size
+ declare line
+ declare id
+ declare value
+ declare size
+ declare running
- while read line; do
+ while read -r line; do
id=$(echo "${line}" | cut -f1 -d, )
- state=$(echo "${line}" | cut -f2 -d, )
+ running=$(echo "${line}" | cut -f2 -d, )
stoptime=$(echo "${line}" | cut -f3 -d, | cut -f1 -d. )
+ if [[ ${running} = true ]]; then
+ yetus_debug "${id} is still running, skipping."
+ continue
+ fi
+
# believe it or not, date is not even close to standardized...
if [[ $(uname -s) == Linux ]]; then
@@ -148,13 +154,11 @@ function rm_old_containers
stoptime=$(date -j -f "%Y-%m-%dT%H:%M:%S" "${stoptime}" "+%s")
fi
- if [[ ${state} == false ]]; then
- curtime=$(date "+%s")
- ((difftime = curtime - stoptime))
- if [[ ${difftime} -gt 86400 ]]; then
- echo "Removing docker ${id}"
- dockercmd rm "${id}"
- fi
+ curtime=$(date "+%s")
+ ((difftime = curtime - stoptime))
+ if [[ ${difftime} -gt 86400 ]]; then
+ echo "Removing docker ${id}"
+ dockercmd rm "${id}"
fi
done < <(
# see https://github.com/koalaman/shellcheck/issues/375
@@ -187,7 +191,7 @@ function remove_old_tagged_images
local id
local created
- while read line; do
+ while read -r line; do
id=$(echo "${line}" | awk '{print $1}')
created=$(echo "${line}" | awk '{print $5}')
[6/7] yetus git commit: YETUS-77. add bugzilla support
Posted by aw...@apache.org.
YETUS-77. add bugzilla support
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/417c3564
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/417c3564
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/417c3564
Branch: refs/heads/YETUS-83
Commit: 417c3564a4326d192d9f96aa4353cc82675b1284
Parents: d1a27cc
Author: Allen Wittenauer <aw...@apache.org>
Authored: Thu Oct 15 10:17:49 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Oct 29 07:59:09 2015 -0700
----------------------------------------------------------------------
.../latest/precommit-bugsystems.md | 36 +++++
precommit/test-patch.d/bugzilla.sh | 154 +++++++++++++++++++
2 files changed, 190 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/417c3564/asf-site-src/source/documentation/latest/precommit-bugsystems.md
----------------------------------------------------------------------
diff --git a/asf-site-src/source/documentation/latest/precommit-bugsystems.md b/asf-site-src/source/documentation/latest/precommit-bugsystems.md
index 46f5111..823dce6 100644
--- a/asf-site-src/source/documentation/latest/precommit-bugsystems.md
+++ b/asf-site-src/source/documentation/latest/precommit-bugsystems.md
@@ -49,3 +49,39 @@ add_bugsystem <pluginname>
* pluginname\_finalreport
- Write the final result table to the bug system.
+
+# Bugzilla Specific
+
+Currently, Bugzilla support is read-only. To use it, the Bug ID must be preferenced with 'BZ:'. For example:
+
+```bash
+$ test-patch.sh (other options) BZ:4
+```
+
+... will pull down Bugzilla ID #4.
+
+Using the `--bugzilla-base-url` on the command line or BUGZILLA\_BASE\_URL in a project's personality will define the location of the Bugzilla instance. By default, it is https://bz.apache.org/bugzilla .
+
+# GitHub Specific
+
+GitHub supports the full range of functionality, including putting comments on individual lines. Be aware, however, that test-patch.sh will require that GitHub PRs be fully rebased (i.e., a single commit) in many circumstances.
+
+By default, the GitHub plug-in assumes that https://github.com is the base URL for GitHub. Enterprise users may override this with the `--github-base-url` for the normal web user interface and `--github-api-url` for the API URL. Personalities may use GITHUB\_API\_URL and GITHUB\_BASE\_URL.
+
+The specific repository on GitHub is defined with either `--github-repo` on the command line or GITHUB\_REPO in a personality. It should take the form of "user/repo".
+
+In order to comment on issues or, depending upon the security setup of the repo, authentication credentials. The GitHub plug-in supports two types:
+
+ * Token-based: `--github-token` or GITHUB\_TOKEN
+
+ * Username/password: `--github-user`/ GITHUB\_USER , `--github-password` / GITHUB\_PASSWD
+
+Pull requests that are made off of a specific branch will switch the test repo to that branch, if permitted. If the pull request references a JIRA issue that matches the given JIRA issue regexp in the Subject, the JIRA plug-in will also be invoked as needed.
+
+# JIRA Specific
+
+JIRA support allows both patch downloads and summary writes. It also supports branch detection-based upon the name of the attached patch file.
+
+JIRA issues are invoked by matching the command line option to a specific regular expression as given by the `--jira-issue-re` option or via the JIRA\_ISSUE\_RE personality variable. By default, the plug-in uses https://issues.apache.org/jira as the JIRA instance to use. However that may be overwritten via the `--jira-base-url` option or personalities may define via JIRA\_URL.
+
+In order to write information on the issue, JIRA requires username and password authentication using the `--jira-user`/`--jira-password` options or the JIRA\_USER and JIRA\_PASSWORD variables in a personality.
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/yetus/blob/417c3564/precommit/test-patch.d/bugzilla.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/bugzilla.sh b/precommit/test-patch.d/bugzilla.sh
new file mode 100755
index 0000000..2ca4395
--- /dev/null
+++ b/precommit/test-patch.d/bugzilla.sh
@@ -0,0 +1,154 @@
+#!/usr/bin/env bash
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+add_bugsystem bugzilla
+
+# personalities can override the following settings:
+BUGZILLA_BASE_URL="https://bz.apache.org/bugzilla"
+
+function bugzilla_usage
+{
+ echo "Bugzilla Options:"
+ echo "--bugzilla-base-url=<url> The URL of the bugzilla server"
+}
+
+function bugzilla_parse_args
+{
+ declare i
+
+ for i in "$@"; do
+ case ${i} in
+ --bugzilla-base-url=*)
+ BUGZILLA_BASE_URL=${i#*=}
+ ;;
+ esac
+ done
+}
+
+function bugzilla_determine_issue
+{
+ declare input=$1
+ declare patchnamechunk
+ declare maybeissue
+
+ if [[ ! "${input}" =~ ^BZ: ]]; then
+ return 1
+ fi
+
+ if [[ -n "${BUGZILLA_ISSUE}" ]]; then
+ return 0
+ fi
+
+ # shellcheck disable=SC2016
+ BUGZILLA_ISSUE=$(echo "${input}" | cut -f2 -d: )
+
+ # shellcheck disable=SC2034
+ ISSUE=${input}
+ add_footer_table "Bugzilla Issue" "${BUGZILLA_ISSUE}"
+ return 0
+}
+
+## @description Try to guess the branch being tested using a variety of heuristics
+## @audience private
+## @stability evolving
+## @replaceable no
+## @return 0 on success, with PATCH_BRANCH updated appropriately
+## @return 1 on failure
+function bugzilla_determine_branch
+{
+ return 1
+}
+
+function bugzilla_http_fetch
+{
+ declare input=$1
+ declare output=$2
+
+ if [[ -z "${BUGZILLA_BASE_URL}" ]]; then
+ return 1
+ fi
+
+ ${CURL} --silent --fail \
+ --output "${output}" \
+ --location \
+ "${BUGZILLA_BASE_URL}/${input}"
+}
+
+
+function bugzilla_locate_patch
+{
+ declare input=$1
+ declare fileloc=$2
+ declare relativeurl
+
+ if [[ -z "${BUGZILLA_BASE_URL}" ]]; then
+ return 1
+ fi
+
+ bugzilla_determine_issue "${input}"
+ if [[ $? != 0 || -z "${BUGZILLA_ISSUE}" ]]; then
+ return 1
+ fi
+
+ yetus_debug "bugzilla_locate_patch: trying ${BUGZILLA_BASE_URL}/show_bug.cgi?id=${BUGZILLA_ISSUE}"
+
+ if [[ "${OFFLINE}" == true ]]; then
+ yetus_debug "bugzilla_locate_patch: offline, skipping"
+ return 1
+ fi
+
+ bugzilla_http_fetch "show_bug.cgi?id=${BUGZILLA_ISSUE}" "${PATCH_DIR}/bugzilla"
+
+ if [[ $? != 0 ]]; then
+ yetus_debug "bugzilla_locate_patch: not a Bugzilla."
+ return 1
+ fi
+
+ #shellcheck disable=SC2016
+ relativeurl=$(${AWK} '/action=diff/ && match($0,"attachment\.cgi.id=[0-9]*"){print substr($0,RSTART,RLENGTH)}' \
+ "${PATCH_DIR}/bugzilla" | \
+ tail -1)
+ PATCHURL="${BUGZILLA_BASE_URL}${relativeurl}"
+ #relativeurl="${relativeurl}&action=diff&context=patch&collapsed=&headers=1&format=raw"
+ echo "${input} patch is being downloaded at $(date) from"
+ echo "${PATCHURL}"
+ add_footer_table "Bugzilla Patch URL" "${PATCHURL}"
+ bugzilla_http_fetch "${relativeurl}" "${fileloc}"
+ if [[ $? != 0 ]];then
+ yetus_error "ERROR: ${input}/${PATCHURL} could not be downloaded."
+ cleanup_and_exit 1
+ fi
+ return 0
+}
+
+## @description Write the contents of a file to Bugzilla
+## @params filename
+## @stability stable
+## @audience public
+function bugzilla_write_comment
+{
+ return 0
+}
+
+## @description Print out the finished details to Bugzilla
+## @audience private
+## @stability evolving
+## @replaceable no
+## @param runresult
+function bugzilla_finalreport
+{
+ return 0
+}
[3/7] yetus git commit: YETUS-64. file our DOAP
Posted by aw...@apache.org.
YETUS-64. file our DOAP
Signed-off-by: Sean Busbey <bu...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/d45ae412
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/d45ae412
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/d45ae412
Branch: refs/heads/YETUS-83
Commit: d45ae412412ea827ec01542afaafb45cc5029633
Parents: 1bab736
Author: Allen Wittenauer <aw...@apache.org>
Authored: Wed Oct 21 00:31:27 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Oct 29 07:52:22 2015 -0700
----------------------------------------------------------------------
asf-site-src/source/doap_yetus.rdf | 54 +++++++++++++++++++++++++++++++++
1 file changed, 54 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/d45ae412/asf-site-src/source/doap_yetus.rdf
----------------------------------------------------------------------
diff --git a/asf-site-src/source/doap_yetus.rdf b/asf-site-src/source/doap_yetus.rdf
new file mode 100644
index 0000000..b57a982
--- /dev/null
+++ b/asf-site-src/source/doap_yetus.rdf
@@ -0,0 +1,54 @@
+<?xml version="1.0"?>
+<?xml-stylesheet type="text/xsl"?>
+<rdf:RDF xml:lang="en"
+ xmlns="http://usefulinc.com/ns/doap#"
+ xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
+ xmlns:asfext="http://projects.apache.org/ns/asfext#"
+ xmlns:foaf="http://xmlns.com/foaf/0.1/">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+ <Project rdf:about="https://yetus.apache.org">
+ <created>2015-09-16</created>
+ <license rdf:resource="http://spdx.org/licenses/Apache-2.0" />
+ <name>Apache Yetus</name>
+ <homepage rdf:resource="https://yetus.apache.org/" />
+ <asfext:pmc rdf:resource="https://yetus.apache.org/" />
+ <shortdesc>A collection of libraries and tools that enable contribution and release processes for software projects.</shortdesc>
+ <description>A collection of libraries and tools that enable contribution and release processes for software projects.</description>
+ <bug-database rdf:resource="https://issues.apache.org/jira/browse/yetus" />
+ <mailing-list rdf:resource="https://yetus.apache.org/mailinglists/" />
+ <download-page rdf:resource="http://yetus.apache.org/downloads/" />
+ <programming-language>Bash</programming-language>
+ <programming-language>Java</programming-language>
+ <programming-language>Python</programming-language>
+ <category rdf:resource="http://projects.apache.org/category/build-management" />
+ <category rdf:resource="http://projects.apache.org/category/library" />
+ <category rdf:resource="http://projects.apache.org/category/testing" />
+ <repository>
+ <GitRepository>
+ <location rdf:resource="https://git-wip-us.apache.org/repos/asf/yetus.git"/>
+ <browse rdf:resource="https://github.com/apache/yetus"/>
+ </GitRepository>
+ </repository>
+ <maintainer>
+ <foaf:Person>
+ <foaf:name>Allen Wittenauer</foaf:name>
+ <foaf:mbox rdf:resource="mailto:aw@apache.org"/>
+ </foaf:Person>
+ </maintainer>
+ </Project>
+</rdf:RDF>
[7/7] yetus git commit: YETUS-138. Skipping maven poms that don't
package causes testing holes (aw)
Posted by aw...@apache.org.
YETUS-138. Skipping maven poms that don't package causes testing holes (aw)
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/c3a20699
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/c3a20699
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/c3a20699
Branch: refs/heads/YETUS-83
Commit: c3a206998ce007a226e6ee5aea00b95645198751
Parents: 417c356
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Oct 27 14:51:30 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Oct 29 08:02:18 2015 -0700
----------------------------------------------------------------------
.../documentation/latest/precommit-advanced.md | 2 --
precommit/test-patch.sh | 27 +++++---------------
2 files changed, 6 insertions(+), 23 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/c3a20699/asf-site-src/source/documentation/latest/precommit-advanced.md
----------------------------------------------------------------------
diff --git a/asf-site-src/source/documentation/latest/precommit-advanced.md b/asf-site-src/source/documentation/latest/precommit-advanced.md
index f68db7a..9b068b6 100644
--- a/asf-site-src/source/documentation/latest/precommit-advanced.md
+++ b/asf-site-src/source/documentation/latest/precommit-advanced.md
@@ -226,8 +226,6 @@ There are a handful of extremely important system variables that make life easie
* CHANGED\_FILES is a list of all files that appear to be added, deleted, or modified in the patch.
-* CHANGED\_UNFILTERED\_MODULES is a list of all modules that house all of the CHANGED\_FILES. Be aware that the root of the source tree is reported as '.'.
-
* CHANGED\_MODULES reports which modules that appear to have source code in them.
* GITHUB\_REPO is to help test-patch when talking to Github. If test-patch is given just a number on the command line, it will default to using this repo to determine the pull request.
http://git-wip-us.apache.org/repos/asf/yetus/blob/c3a20699/precommit/test-patch.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.sh b/precommit/test-patch.sh
index 93874a2..709c8b7 100755
--- a/precommit/test-patch.sh
+++ b/precommit/test-patch.sh
@@ -70,8 +70,7 @@ function setup_defaults
# shellcheck disable=SC2034
CHANGED_MODULES=""
- # shellcheck disable=SC2034
- CHANGED_UNFILTERED_MODULES=""
+
# shellcheck disable=SC2034
CHANGED_UNION_MODULES=""
USER_MODULE_LIST=""
@@ -320,6 +319,8 @@ function finish_footer_table
{
local maxmem
+ add_footer_table "modules" "C: ${CHANGED_MODULES} U: ${CHANGED_UNION_MODULES}"
+
# `sort | head` can cause a broken pipe error, but we can ignore it just like compute_gitdiff.
# shellcheck disable=SC2016,SC2086
maxmem=$(find "${PATCH_DIR}" -type f -exec ${AWK} 'match($0, /^\[INFO\] Final Memory: [0-9]+/)
@@ -997,7 +998,7 @@ function module_skipdir
## @audience private
## @stability stable
## @replaceable no
-## @return None; sets ${CHANGED_MODULES} and ${CHANGED_UNFILTERED_MODULES}
+## @return None; sets ${CHANGED_MODULES}
function find_changed_modules
{
local i
@@ -1044,24 +1045,8 @@ function find_changed_modules
fi
#shellcheck disable=SC2086,SC2034
- CHANGED_UNFILTERED_MODULES=$(echo ${builddirs} ${USER_MODULE_LIST} | tr ' ' '\n' | sort -u)
- #shellcheck disable=SC2086,SC2116
- CHANGED_UNFILTERED_MODULES=$(echo ${CHANGED_UNFILTERED_MODULES})
-
- if [[ ${BUILDTOOL} = maven ]]; then
- # Filter out modules without code
- for module in ${builddirs}; do
- ${GREP} "<packaging>pom</packaging>" "${module}/pom.xml" > /dev/null
- if [[ "$?" != 0 ]]; then
- buildmods="${buildmods} ${module}"
- fi
- done
- else
- buildmods=${CHANGED_UNFILTERED_MODULES}
- fi
-
- #shellcheck disable=SC2086,SC2034
- CHANGED_MODULES=$(echo ${buildmods} ${USER_MODULE_LIST} | tr ' ' '\n' | sort -u)
+ CHANGED_MODULES=$(echo ${builddirs} ${USER_MODULE_LIST} | tr ' ' '\n' | sort -u)
+ CHANGED_MODULES=$(echo ${CHANGED_MODULES} ${USER_MODULE_LIST} | tr ' ' '\n' | sort -u)
# turn it back into a list so that anyone printing doesn't
# generate multiline output