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)

Reply via email to