Bug#405772: [Pkg-ltsp-devel] Bug#405772: ltsp-client: also fix SWAP
	related booleans
    Petter Reinholdtsen 
    pere at hungry.com
       
    Sun Jan  7 16:45:12 UTC 2007
    
    
  
[Vagrant Cascadian]
> +get_boolean(){
> +    case "$(echo $1 | tr 'A-Z' 'a-z')" in
> +       true|y|yes) return 0 ;; 
> +       *) return 1 ;; 
> +    esac
> +}
I'm not sure if this is a good function name for that operation.  What
about boolean_is_true() or something like that.  get_boolean()
indicate to me that it should return the value, and I expect to be
able to use it like VAL=`get_boolean $VAL`, and that is not the case
with this function.
Besides this minor point, the patch look good to me.
Friendly,
-- 
Petter Reinholdtsen
    
    
More information about the Pkg-ltsp-devel
mailing list