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)