Merge pull request #12631 from alex--m/topic/fix_dist_buildrpm_v4.1.x

jsquyres · web-flow · commit c89eaac6d725 · 2024-06-18T15:26:56.000-04:00
v4.1.x: 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
@@ -254,7 +254,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
`@@ -254,7 +254,7 @@ echo "--> Found specfile: $specfile"`
`254`	`254`	`#`
`255`	`255`	`# try to find Libfabric lib subir`
`256`	`256`	`#`
`257`		`-if test -n $libfabric_path; then`
	`257`	`+if test -n "$libfabric_path"; then`
`258`	`258`	`# does lib64 exist?`
`259`	`259`	`if test -d $libfabric_path/lib64; then`
`260`	`260`	`# yes, so I will use lib64 as include dir`