remove "abcipp" feature

Author: tzemanovic

Makefile

@@ -83,7 +83,7 @@ check-mainnet:
 check-crates:
 	$(foreach p,$(crates), echo "Checking $(p)" && cargo +$(nightly) check -Z unstable-options --tests -p $(p) && ) \
 		make -C $(wasms_for_tests) check && \
-		cargo check --package namada --target wasm32-unknown-unknown --no-default-features --features "abciplus,namada-sdk"
+		cargo check --package namada --target wasm32-unknown-unknown --no-default-features --features "namada-sdk"
 
 clippy-wasm = $(cargo) +$(nightly) clippy --manifest-path $(wasm)/Cargo.toml --all-targets -- -D warnings
 

apps/Cargo.toml

@@ -50,7 +50,7 @@ name = "namadar"
 path = "src/bin/namada-relayer/main.rs"
 
 [features]
-default = ["std", "abciplus"]
+default = ["std"]
 mainnet = [
   "namada/mainnet",
 ]
@@ -59,14 +59,10 @@ std = ["ed25519-consensus/std", "rand/std", "rand_core/std", "namada/std", "nama
 testing = ["namada_test_utils"]
 benches = ["testing", "namada_test_utils"]
 integration = []
-abciplus = [
-    "namada/abciplus",
-    "namada/tendermint-rpc",
-]
 
 
 [dependencies]
-namada = {path = "../shared", features = ["ferveo-tpke", "masp-tx-gen", "multicore", "http-client"]}
+namada = {path = "../shared", features = ["ferveo-tpke", "masp-tx-gen", "multicore", "http-client", "tendermint-rpc"]}
 namada_sdk = {path = "../sdk", default-features = false, features = ["wasm-runtime", "masp-tx-gen"]}
 namada_test_utils = {path = "../test_utils", optional = true}
 ark-serialize.workspace = true

apps/src/lib/mod.rs

@@ -21,19 +21,6 @@ pub mod wasm_loader;
 pub use std;
 
 pub mod facade {
-    //! Facade module to reason about `abcipp` feature flag logic.
-
-    #[cfg(not(feature = "abcipp"))]
-    pub use {
-        namada::tendermint, namada::tendermint_proto, namada::tendermint_rpc,
-        tendermint_config, tower_abci,
-    };
-    #[cfg(feature = "abcipp")]
-    pub use {
-        tendermint_abcipp as tendermint,
-        tendermint_config_abcipp as tendermint_config,
-        tendermint_proto_abcipp as tendermint_proto,
-        tendermint_rpc_abcipp as tendermint_rpc,
-        tower_abci_abcipp as tower_abci,
-    };
+    pub use namada::{tendermint, tendermint_proto, tendermint_rpc};
+    pub use {tendermint_config, tower_abci};
 }

apps/src/lib/node/ledger/shell/finalize_block.rs

@@ -223,56 +223,6 @@ where
                     self.allow_tx_replay(wrapper_tx);
                 }
 
-                #[cfg(not(any(feature = "abciplus", feature = "abcipp")))]
-                if let TxType::Wrapper(wrapper) = &tx_header.tx_type {
-                    // Charge fee if wrapper transaction went out of gas or
-                    // failed because of fees
-                    let error_code =
-                        ErrorCodes::from_u32(processed_tx.result.code).unwrap();
-                    if (error_code == ErrorCodes::TxGasLimit)
-                        | (error_code == ErrorCodes::FeeError)
-                    {
-                        let masp_transaction = wrapper
-                            .unshield_section_hash
-                            .map(|ref hash| {
-                                tx.get_section(hash)
-                                    .map(|section| {
-                                        if let Section::MaspTx(transaction) =
-                                            section
-                                        {
-                                            Some(transaction.to_owned())
-                                        } else {
-                                            None
-                                        }
-                                    })
-                                    .flatten()
-                            })
-                            .flatten();
-                        if let Err(msg) = protocol::charge_fee(
-                            wrapper,
-                            masp_transaction,
-                            ShellParams::new(
-                                TxGasMeter::new_from_sub_limit(u64::MAX),
-                                &mut self.wl_storage,
-                                &mut self.vp_wasm_cache,
-                                &mut self.tx_wasm_cache,
-                            ),
-                            Some(&native_block_proposer_address),
-                            &mut BTreeSet::default(),
-                        ) {
-                            self.wl_storage.write_log.drop_tx();
-                            tracing::error!(
-                                "Rejected wrapper tx {} could not pay fee: {}",
-                                Hash::sha256(
-                                    tx::try_from(processed_tx.as_ref())
-                                        .unwrap()
-                                ),
-                                msg
-                            )
-                        }
-                    }
-                }
-
                 continue;
             }
 

apps/src/lib/node/ledger/shell/mod.rs

@@ -339,7 +339,6 @@ impl ShellMode {
 
     /// If this node is a validator, broadcast a tx
     /// to the mempool using the broadcaster subprocess
-    #[cfg_attr(feature = "abcipp", allow(dead_code))]
     pub fn broadcast(&self, data: Vec<u8>) {
         if let Self::Validator {
             broadcast_sender, ..
@@ -1059,7 +1058,6 @@ where
         tx_bytes: &[u8],
         r#_type: MempoolTxType,
     ) -> response::CheckTx {
-        #[cfg(not(feature = "abcipp"))]
         use namada::types::transaction::protocol::{
             ethereum_tx_data_variants, ProtocolTxType,
         };
@@ -1134,7 +1132,6 @@ where
 
         match tx_type.tx_type {
             TxType::Protocol(protocol_tx) => match protocol_tx.tx {
-                #[cfg(not(feature = "abcipp"))]
                 ProtocolTxType::EthEventsVext => {
                     let ext = try_vote_extension!(
                         "Ethereum events",
@@ -1156,7 +1153,6 @@ where
                         response.log = String::from(VALID_MSG);
                     }
                 }
-                #[cfg(not(feature = "abcipp"))]
                 ProtocolTxType::BridgePoolVext => {
                     let ext = try_vote_extension!(
                         "Bridge pool roots",
@@ -1180,7 +1176,6 @@ where
                         response.log = String::from(VALID_MSG);
                     }
                 }
-                #[cfg(not(feature = "abcipp"))]
                 ProtocolTxType::ValSetUpdateVext => {
                     let ext = try_vote_extension!(
                         "validator set update",
@@ -1839,14 +1834,6 @@ mod test_utils {
         }
     }
 
-    /// Get the only validator's voting power.
-    #[inline]
-    #[cfg(not(feature = "abcipp"))]
-    #[allow(dead_code)]
-    pub fn get_validator_bonded_stake() -> namada::types::token::Amount {
-        200_000_000_000.into()
-    }
-
     /// Config parameters to set up a test shell.
     pub struct SetupCfg<H> {
         /// The last comitted block height.