On Thu, 2014-07-03 at 10:24 +0530, Amit Kucheria wrote:
> On Thu, Jul 3, 2014 at 7:37 AM, Lisa Nguyen <lisa.ngu...@linaro.org> wrote:
[...]
> > diff --git a/include/functions.sh b/include/functions.sh
> > index 6d75e34..d44706f 100644
> > --- a/include/functions.sh
> > +++ b/include/functions.sh
> > @@ -285,10 +285,10 @@ check_cpuhotplug_files() {
> >      shift 1
> >
> >      for i in $@; do
> > -        # skip check for cpu0
> > -        if [ `echo $dirpath | grep -c "cpu0"` -eq 1 ]; then
> > -            continue
> > -        fi
> > +        if [[ $dirpath =~ cpu0$ && $hotplug_allow_cpu0 -eq 0 ]]; then
> 
> I just checked 'man bash' for the meaning of =~ and found this:
> 
> "The return value is 0 if the string matches the pattern, and 1 otherwise."
> 
> So if you find cpu0 in $dirpath, it'll return a zero, right? So your
> above expression becomes zero and you don't end up skipping the test
> for cpu0.

But in shells, isn't zero true and non-zero false, and && behaves
accordingly?

I have another comment though, will the scripts only be used on systems
with bash? If we can't guarantee this perhaps bash specific features
should be avoided. (Is the android shell bash compatible?) 

-- 
Tixy


_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to