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

Makefile: have "install" depend on pgo #95287

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

kmod
Copy link
Contributor

@kmod kmod commented Jul 26, 2022

Previously, make install would ignore pgo. This isn't an
issue if one runs make && make install, since make will
build with pgo (if requested) and then make install won't
rebuild anything.

But if one simply does ./configure --enable-optimizations && make install,
the build system would build a non-pgo build and then install that.

This commit changes the Makefile so that install depends on
all, which includes pgo when requested.

Previously, `make install` would ignore pgo. This isn't an
issue if one runs `make && make install`, since `make` will
build with pgo (if requested) and then `make install` won't
rebuild anything.

But if one simply does `./configure --enable-optimizations && make install`,
the build system would build a non-pgo build and then install that.

This commit changes the Makefile so that `install` depends on
`all`, which includes pgo when requested.
@cpython-cla-bot
Copy link

cpython-cla-bot bot commented Jul 26, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants