You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Chia-Ping Tsai (JIRA)" <ji...@apache.org> on 2017/10/27 22:50:00 UTC
[jira] [Created] (YETUS-571) The checkstyle always passes if the
base dir is located at "/"
Chia-Ping Tsai created YETUS-571:
------------------------------------
Summary: The checkstyle always passes if the base dir is located at "/"
Key: YETUS-571
URL: https://issues.apache.org/jira/browse/YETUS-571
Project: Yetus
Issue Type: Bug
Reporter: Chia-Ping Tsai
Priority: Minor
We can't fetch any info from checkstyle report for changed files because the {{SED}} performs a global substitution.
{code}
echo_and_redirect "${logfile}" ${cmd}
${SED} -e "s,^\[ERROR\] ,,g" -e "s,^\[WARN\] ,,g" "${logfile}" \
| ${GREP} ^/ \
| ${SED} -e "s,${BASEDIR},.,g" \
> "${tmp}"
...
for j in "${CHANGED_FILES[@]}"; do
${GREP} "${j}" "${tmp}" >> "${tmp}.1"
done
{code}
The incorrect path will look like this:
{quote}
..-server/src/main/java/org/apache/hadoop./regionserver/HRegion.java:7953:
{quote}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)