Skip to content

Commit

Permalink
Merge pull request #1105 from JeanChristopheMorinPerso/remove_rez_pip…
Browse files Browse the repository at this point in the history
…_search

Remove the -s flag to rez-pip because "pip search" is decommissioned
  • Loading branch information
nerdvegas authored Jul 13, 2021
2 parents f4930af + 24d4fa9 commit 6b780dd
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions src/rez/cli/pip.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ def setup_parser(parser, completions=False):
parser.add_argument(
"-i", "--install", action="store_true",
help="install the package")
parser.add_argument(
"-s", "--search", action="store_true",
help="search for the package on PyPi")
parser.add_argument(
"-r", "--release", action="store_true",
help="install as released package; if not set, package is installed "
Expand All @@ -47,17 +44,12 @@ def command(opts, parser, extra_arg_groups=None):
# Prevent other rez.* loggers from printing debugs
logging.getLogger('rez').setLevel(logging.INFO)

from rez.pip import pip_install_package, run_pip_command
from rez.pip import pip_install_package
import warnings

if not (opts.search or opts.install):
parser.error("Expected one of: --install, --search")

if opts.search:
with run_pip_command(["search", opts.PACKAGE]) as p:
p.wait()
return

if opts.pip_ver:
with warnings.catch_warnings():
# Cause all warnings to always be triggered.
Expand Down

0 comments on commit 6b780dd

Please sign in to comment.