Commit 122eae5
committed
configury: remove whitespace from "conftestval"-style tests
Several configure tests write integer result values to files and then
do something like foo=$(cat conftestval). However, if the conftestval
file contains whitespace around the integer value (e.g., if
conftestval was written by a Fortran test code), then $foo will still
contain that whitespace, too -- which then propagates throughout the
configury and elsewhere.
This commit uses xargs to strip off the whitespace. There's really
only one place where this is happening that is a problem, but let's do
it in all places where we foo=$(cat conftestval) with integer results,
just as a matter of consistency and defensive programming.
Thanks to Lisandro Dalcin for raising the issue.
Signed-off-by: Jeff Squyres <jeff@squyres.com>1 parent 77728ef commit 122eae5
File tree
4 files changed
+4
-4
lines changed- config
4 files changed
+4
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
104 | | - | |
| 104 | + | |
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
141 | 141 | | |
142 | 142 | | |
143 | 143 | | |
144 | | - | |
| 144 | + | |
145 | 145 | | |
146 | 146 | | |
147 | 147 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
79 | 79 | | |
80 | 80 | | |
81 | 81 | | |
82 | | - | |
| 82 | + | |
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
| |||
0 commit comments