Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation suggests removed --install-scripts flag #10265

Closed
ndevenish opened this issue Aug 3, 2021 · 3 comments · Fixed by #11560
Closed

Documentation suggests removed --install-scripts flag #10265

ndevenish opened this issue Aug 3, 2021 · 3 comments · Fixed by #11560
Labels
good first issue A good item for first time contributors to work on state: awaiting PR Feature discussed, PR is needed type: docs Documentation related

Comments

@ndevenish
Copy link

The documentation for --install-options (online at https://pip.pypa.io/en/latest/cli/pip_install/#cmdoption-install-option) gives the example of passing --install-scripts. Since #7309 this is now an invalid option. This example should be updated (and perhaps the current way to achieve this added).

Text comes from:

install_options: Callable[..., Option] = partial(
Option,
"--install-option",
dest="install_options",
action="append",
metavar="options",
help="Extra arguments to be supplied to the setup.py install "
'command (use like --install-option="--install-scripts=/usr/local/'
'bin"). Use multiple --install-option options to pass multiple '
"options to setup.py install. If you are using an option with a "
"directory path, be sure to use absolute path.",
)

A comment on the deprecation issue asked for this to be opened at: #7309 (comment)

@xavfernandez xavfernandez added type: docs Documentation related state: awaiting PR Feature discussed, PR is needed labels Aug 15, 2021
@pradyunsg pradyunsg added the good first issue A good item for first time contributors to work on label Feb 18, 2022
@hsadia538
Copy link

@ndevenish I want to work on this.

@Patelrahul4884
Copy link

@ndevenish I can work on this issue

@Patelrahul4884
Copy link

@ndevenish do we have to completely remove install_options variable?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue A good item for first time contributors to work on state: awaiting PR Feature discussed, PR is needed type: docs Documentation related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants