Skip to content

Commit d0edc44

Browse files
committed
Remove a bashism in configure
1 parent 0796d2e commit d0edc44

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

configure

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,21 +39,21 @@ else
3939
echo "Check https://www.bloomberg.com/professional/support/api-library/ for possible support first."
4040
echo "Contributions welcome, see https://github.com/Rblp/blp for integration with Rblapi."
4141
echo "The build will proceed but not be functional for lack of a library."
42-
sed -e"s/@have_blp@/-DNoBlpHere/" src/Makevars.in > src/Makevars
42+
sed -e"s/@have_blp@/no/" src/Makevars.in > src/Makevars
4343
exit 0
4444
fi
4545

4646
## Populate Makevars
4747
arch=$(uname -m)
48-
if [ "${arch}" = "x86_64" -a "${platform}" = "linux" ]; then
48+
if [ "${arch}" = "x86_64" ] && [ "${platform}" = "linux" ]; then
4949
echo "Setting up compilation for a ${platform} ${arch} system"
5050
sed -e"s/@have_blp@/yes/" src/Makevars.in > src/Makevars
5151
flavour="64"
5252
# this used to matter when Blp supported x86 on macos, they no longer do
5353
#if [ "${platform}" = "osx" ]; then
5454
# cpu="x86"
5555
#fi
56-
elif [ "${arch}" = "arm64" -a "${platform}" = "osx" ]; then
56+
elif [ "${arch}" = "arm64" ] && [ "${platform}" = "osx" ]; then
5757
echo "Setting up compilation for a ${platform} ${arch} system"
5858
sed -e"s/@have_blp@/yes/" src/Makevars.in > src/Makevars
5959
flavour="64"

0 commit comments

Comments
 (0)