Skip to content

Makefile: have "install" depend on pgo - #95287

Open
kmod wants to merge 2 commits into
python:mainfrom
kmod:install_pgo
Open

kmod wants to merge 2 commits into
python:mainfrom
kmod:install_pgo

Conversation

@kmod

@kmod kmod commented Jul 26, 2022

Copy link
Copy Markdown
Contributor

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.
@ghost

ghost commented Jul 26, 2022 •

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot

Copy link
Copy Markdown

Most changes to Python require a NEWS entry.

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

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 11, 2026
@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label Oct 23, 2026
@emmatyping

Copy link
Copy Markdown
Member

Hi @kmod, can you please add a news entry?

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.

4 participants