Skip to content

Commit

Permalink
Remove readline installation from xjalienfs recipe (#5655)
Browse files Browse the repository at this point in the history
Since #5534 we depend on
gnureadline globally anyway, no point in reinstalling it in the recipe
  • Loading branch information
singiamtel authored Oct 18, 2024
1 parent d56c511 commit 2fe6122
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions xjalienfs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,6 @@ prepend_path:
# works inside and outside a virtualenv, but unset VIRTUAL_ENV to make sure we
# only depend on stuff we installed using our Python and Python-modules.

# on macos try to install gnureadline and just skip if fails (alienpy can work without it)
# macos python readline implementation is build on libedit which does not work
[[ "$ARCHITECTURE" == osx_* ]] && { env -u VIRTUAL_ENV ALIBUILD=1 \
python3 -m pip install --force-reinstall \
--target="$INSTALLROOT/lib/python/site-packages" \
gnureadline || : ; }

env -u VIRTUAL_ENV ALIBUILD=1 \
python3 -m pip install --force-reinstall \
--target="$INSTALLROOT/lib/python/site-packages" \
Expand Down

0 comments on commit 2fe6122

Please sign in to comment.