Thanks Rahul ,
Will make these changes .

rahul wrote:
> | > As part of the CR :6634053 Integrate MySQL 64-bit release ,I would 
> | > like the changes to be reviewed at
> | > http://cr.opensolaris.org/~sunandam/6634053/
>
> http://cr.opensolaris.org/~sunandam/6634053/usr/src/Targetdirs.udiff.html
>
>
>
>  SYM.ROOT.BIN64= \
>          /usr/lib/64 \
>          /usr/postgres/8.2/bin/64 \
>          /usr/postgres/8.2/lib/64 <---- I think you missed a slash here.
> +        /usr/mysql/5.0/bin/64 \
> +        /usr/mysql/5.0/lib/64
>
> http://cr.opensolaris.org/~sunandam/6634053/usr/src/cmd/mysql-5-0/Makefile.sfw.udiff.html
>  clean:
>          -rm -rf $(MYSQL_DIR)
> +        -rm -rf $(MYSQL_DIR_64)
>  
> +clean32:
> +        -rm -rf $(MYSQL_DIR)
>  
> +clean64:
> +         -rm -rf $(MYSQL_DIR_64)
>
> it may be better to just use
> clean: clean32 clean64
>
>
>  ==========
> +.SUFFIXES:
> +
>
> suffixes are probably not needed any more?
>
> http://cr.opensolaris.org/~sunandam/6634053/usr/src/cmd/mysql-5-0/Solaris/mysql.udiff.html
> some problems with indentation ? :)
>
>
>
> _______________________________________________
>
>
> webstack-discuss mailing list
> webstack-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss
>   


Reply via email to