Skip to content

Add mainnet, testnet, betanet; remove exit-precompiles compilation features#147

Merged
artob merged 3 commits intodevelopfrom
add-net-related-features
Jun 17, 2021
Merged

Add mainnet, testnet, betanet; remove exit-precompiles compilation features#147
artob merged 3 commits intodevelopfrom
add-net-related-features

Conversation

@sept-en
Copy link
Copy Markdown
Contributor

@sept-en sept-en commented Jun 17, 2021

The purpose of PR is to be able to always build for a specific network (Mainnet/Testnet/Betanet) with explicitly defined set of features.

Please also note that --always-make or -B parameter to make (specified in README.md) is mandatory, otherwise Makefile won't rebuild with just only a different feature.

The expected usage is:

make --always-make mainnet # Build for  Mainnet
make --always-make testnet # Build for Testnet
make --always-make betanet # Build for Betanet

@sept-en sept-en added the C-enhancement Category: New feature or request label Jun 17, 2021
@sept-en sept-en requested review from artob, birchmd and joshuajbouw June 17, 2021 13:22
@sept-en sept-en changed the base branch from master to develop June 17, 2021 13:22
Copy link
Copy Markdown
Contributor

@joshuajbouw joshuajbouw left a comment

Choose a reason for hiding this comment

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

Also, straight forward.

@artob
Copy link
Copy Markdown
Contributor

artob commented Jun 17, 2021

@sept-en I think you can go ahead and remove the exit-precompiles flag while we're at it, I don't see why we wouldn't build with it always going forward now.

Comment thread README.md Outdated
@artob artob changed the title Add mainnet, testnet, betanet compilation features. Add mainnet, testnet, betanet compilation features Jun 17, 2021
@sept-en sept-en requested a review from joshuajbouw June 17, 2021 14:02
@sept-en sept-en changed the title Add mainnet, testnet, betanet compilation features Add mainnet, testnet, betanet; remove exit-precompiles compilation features Jun 17, 2021
Copy link
Copy Markdown
Member

@birchmd birchmd left a comment

Choose a reason for hiding this comment

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

Nice!

@artob artob self-assigned this Jun 17, 2021
@artob artob merged commit 37961b2 into develop Jun 17, 2021
@artob artob deleted the add-net-related-features branch June 17, 2021 14:20
artob added a commit that referenced this pull request Jun 18, 2021
* Implement EIP-2565 support. (#138)
* Fix CI debug pipeline. (#145)
* Enable `meta_call` only with feature flag. (#146)
* Add `mainnet`, `testnet`, and `betanet` feature flags. (#147)
* Prohibit static calls in exit precompiles. (#148)
* Fix `clippy::unnecessary_unwrap`. (#149)
* Fix all Clippy warnings. (#150)
* Create the `ExitToNear` receiver account if it does not exist. (#151)

Co-authored-by: Joshua J. Bouw <joshua@aurora.dev>
Co-authored-by: Kirill <kirill@aurora.dev>
Co-authored-by: Marcelo Fornet <marcelo@aurora.dev>
Co-authored-by: Michael Birch <michael@aurora.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-enhancement Category: New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants