Hi everyone,
This patch fixes an issue.
When a needed function is not found it didn't show the warning but it still
put the faults variable to 1.
With this patch I propose to always see the warning and set the faults to
1. Also set an ending message saying when not everything went fine.
Martin Boretto
Software Engineer
Taller Technologies - Argentina
San Lorenzo 47, 3rd Floor, Office 5
Córdoba, Argentina
*Mobile:* [+54 358 4206995 (ARG)]
*Skype:* tinacho_b
diff --git a/merge-helpers/check_submission b/merge-helpers/check_submission
index a934646..5444b1b 100755
--- a/merge-helpers/check_submission
+++ b/merge-helpers/check_submission
@@ -198,10 +198,8 @@ function check_methods(){
echo $bsp : ${i%(*}"()" present in file $(grep -rlE
"[a-z|A-Z|0-9|_]+[ ]*$i[ ]*\(" *)
fi
else
- if [[ $warnings -eq "1" || $important -eq "1" || $verbose -eq "1"
]]; then
echo "$bsp : $(basename $filename) : ${i%(*}() function does not
exist in $filename "
flag="1"
- fi
faults="1"
fi
fi
@@ -687,4 +685,6 @@ start "$@"
# End of Checks
if [[ $faults -eq "0" ]] ; then
echo -e "\nNo faults found in BSP organization"
+else
+ echo -e "\nFound faults in BSP organization. Please, check"
fi
_______________________________________________
rtems-devel mailing list
rtems-devel@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel
