starlingx/integ
Code Issues Proposed changes

Merge "speclint: only show the diff files"

Browse Source
This commit is contained in:
Zuul 2019-07-05 17:11:30 +00:00 committed by Gerrit Code Review
commit 6e2106dafd
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tools/spec-tools/speclint.sh
View File

@ -14,7 +14,7 @@ usage() {
exit 1 exit 1
} }
if [ -z $@ ]; then if [ ${#@} -eq 0 ]; then
usage usage
fi fi
@ -65,7 +65,7 @@ for spec in $@; do
(cd $(dirname $spec); HOME=$tmpdir rpmspec -q --qf "%{VERSION}\n" $specname) >/dev/null || { (cd $(dirname $spec); HOME=$tmpdir rpmspec -q --qf "%{VERSION}\n" $specname) >/dev/null || {
echo "$spec does not parse properly. Please check your syntax." echo "$spec does not parse properly. Please check your syntax."
failed=1 failed=1
} }
echo "spec-cleaner checking $specname" echo "spec-cleaner checking $specname"
# Make a copy to do some fix-ups required by spec-cleaner # Make a copy to do some fix-ups required by spec-cleaner
@ -84,10 +84,9 @@ for spec in $@; do
done done
# check if some diffs are available # check if some diffs are available
failed=0 for specdiff in $tmpdir/*.cleaner.diff; do
for specdiff in $tmpdir/*; do
if [ -s "$specdiff" ]; then if [ -s "$specdiff" ]; then
echo "##### `basename ${specdiff}` ##### " echo "##### `basename ${specdiff} .cleaner.diff` ##### "
cat $specdiff cat $specdiff
failed=1 failed=1
fi fi