Ping for maintainer... 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
Regards. 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 2014-06-04 18:21 GMT-03:00 Martin Boretto < martin.bore...@tallertechnologies.com>: > Hi everyone, > > This patch fixes a possible 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 >
_______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel