Skip to content

[FIX] - Update build profiles #575

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nhussein11
Copy link
Collaborator

This PR aims to take over #557 and fix #534

PieWol and others added 2 commits April 22, 2025 11:16
* Update set-up-a-template.md

improve wording

* update compiler flag tip
@nhussein11 nhussein11 added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) labels Apr 22, 2025
@nhussein11 nhussein11 requested a review from 0xLucca April 22, 2025 14:43
@nhussein11 nhussein11 self-assigned this Apr 22, 2025
@nhussein11 nhussein11 requested a review from a team as a code owner April 22, 2025 14:43
@nhussein11 nhussein11 mentioned this pull request Apr 22, 2025
Copy link
Collaborator

@eshaben eshaben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! LGTM

Initial compilation may take several minutes, depending on your machine specifications. Always use the `--release` flag to build optimized, production-ready artifacts.
Initial compilation may take several minutes, depending on your machine specifications. Use the `--release` flag for improved runtime performance compared to the default `--debug` build. If you need to troubleshoot issues, the `--debug` build provides better diagnostics.

For production deployments, consider using a dedicated `--production` profile (which can be copied from the [Polkadot SDK](https://github.com/paritytech/polkadot-sdk/blob/5d47972fad149da688f096d25ba643bd783f3604/Cargo.toml#L1436-L1439){target=\_blank}) - this can provide an additional 15-30% performance improvement over the standard `--release` profile.
Copy link
Contributor

@ggwpez ggwpez Apr 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is --profile production.

I will ask to include this into the templates to make it available here without additional effort.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! @ggwpez, should we hold this PR until paritytech/polkadot-sdk-parachain-template#35 is resolved?

github-merge-queue bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Apr 30, 2025
…8378)

Addressing
paritytech/polkadot-sdk-parachain-template#35

Adding the production build profile for 15-30% speedup when people build
with --profile production, see
polkadot-developers/polkadot-docs#534 and
polkadot-developers/polkadot-docs#575.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-merge-queue bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Apr 30, 2025
…8378)

Addressing
paritytech/polkadot-sdk-parachain-template#35

Adding the production build profile for 15-30% speedup when people build
with --profile production, see
polkadot-developers/polkadot-docs#534 and
polkadot-developers/polkadot-docs#575.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
paritytech-release-backport-bot bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Apr 30, 2025
…8378)

Addressing
paritytech/polkadot-sdk-parachain-template#35

Adding the production build profile for 15-30% speedup when people build
with --profile production, see
polkadot-developers/polkadot-docs#534 and
polkadot-developers/polkadot-docs#575.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
(cherry picked from commit ce69f0d)
@0xLucca
Copy link
Collaborator

0xLucca commented May 5, 2025

Hey @ggwpez ! I see that this was added to the polkadot-sdk repository. Please let us know when this is added to the polkadot-sdk-parachain-template one, and create a tag or release for it so we can add references

@0xLucca 0xLucca added the B2 - Blocked Pull request is blocked and should not be merged label May 5, 2025
castillax pushed a commit to paritytech/polkadot-sdk that referenced this pull request May 12, 2025
…8378)

Addressing
paritytech/polkadot-sdk-parachain-template#35

Adding the production build profile for 15-30% speedup when people build
with --profile production, see
polkadot-developers/polkadot-docs#534 and
polkadot-developers/polkadot-docs#575.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nhussein11 nhussein11 removed the B0 - Needs Review Pull request is ready for review label May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B2 - Blocked Pull request is blocked and should not be merged C1 - Medium Medium priority task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[template tutorial] Explain build profiles
5 participants