Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 136406

Summary: rox.eclass missing quotes
Product: Gentoo Linux Reporter: Peter Hyman <pete4abw>
Component: EclassesAssignee: Daniel Black (RETIRED) <dragonheart>
Status: RESOLVED FIXED    
Severity: enhancement CC: dragonheart
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 99711    
Attachments: rox.eclass
rox.eclass.diff

Description Peter Hyman 2006-06-11 07:14:25 UTC
The following snippet results in incorrect evaluation of $KEEP_SRC. The variable should have been quoted.

		if [ -n ${KEEP_SRC} ]; then
			( cd src
			make clean
			)
		else
			rm -rf src
		fi
Comment 1 Peter Hyman 2006-06-11 07:16:06 UTC
Created attachment 88908 [details]
rox.eclass

corrects unquoted variable in -n test. 

if [ -n "${KEEP_SRC}" ] instead of
if [ -n ${KEEP_SRC} ]
Comment 2 Daniel Black (RETIRED) gentoo-dev 2006-06-11 16:18:29 UTC
Created attachment 88944 [details, diff]
rox.eclass.diff

May as well go all the way. Tell me if I've "corrected" too far.
Comment 3 Daniel Black (RETIRED) gentoo-dev 2006-06-11 16:18:57 UTC
commited