Merge pull request #12622 from alex--m/topic/fix_dist_buildrpm

jsquyres · web-flow · commit 691e780b068e · 2024-06-18T10:25:34.000-04:00
dist/buildrpm: fix incorrect test, which passed even with an empty path
diff --git a/contrib/dist/linux/buildrpm.sh b/contrib/dist/linux/buildrpm.sh
@@ -253,7 +253,7 @@ echo "--> Found specfile: $specfile"
 #
 # try to find Libfabric lib subir
 #
-if test -n $libfabric_path; then
+if test -n "$libfabric_path"; then
     # does lib64 exist?
     if test -d $libfabric_path/lib64; then
         # yes, so I will use lib64 as include dir

Original file line number	Diff line number	Diff line change
`@@ -253,7 +253,7 @@ echo "--> Found specfile: $specfile"`
`253`	`253`	`#`
`254`	`254`	`# try to find Libfabric lib subir`
`255`	`255`	`#`
`256`		`-if test -n $libfabric_path; then`
	`256`	`+if test -n "$libfabric_path"; then`
`257`	`257`	`# does lib64 exist?`
`258`	`258`	`if test -d $libfabric_path/lib64; then`
`259`	`259`	`# yes, so I will use lib64 as include dir`