Skip to content

Commit

Permalink
Fix dist install checks - add missing set -e.
Browse files Browse the repository at this point in the history
  • Loading branch information
Kami committed Aug 1, 2023
1 parent 24a455f commit ae935f1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion scripts/dist_install_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
set -e

# Verify library installs without any dependencies when using python setup.py
# install
Expand All @@ -26,7 +27,7 @@ python --version
pip show requests && exit 1
pip show typing && exit 1
pip show enum34 && exit 1
pip show apache-libcloud
pip show apache-libcloud && exit 1

# Install the library
python setup.py install
Expand Down
3 changes: 2 additions & 1 deletion scripts/dist_wheel_install_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
# under the License.

# Verify library installs without any dependencies when using built wheel
set -e

echo "Running dist wheel install checks"
python --version
Expand All @@ -26,7 +27,7 @@ python --version
pip show requests && exit 1
pip show typing && exit 1
pip show enum34 && exit 1
pip show apache-libcloud
pip show apache-libcloud && exit 1
rm -rf dist/apache_libcloud-*.whl

pip install wheel
Expand Down

0 comments on commit ae935f1

Please sign in to comment.