Merge remote-tracking branch 'namada/james/mainline/fix-abcipp' (#754)…

… into main

* namada/james/mainline/fix-abcipp:
  Fix changelog
  Fix up for namada v0.10.1
  Add changelog
  Add check-abcipp command
  Use ferveo-tpke flag to stop tendermint-rpc being pulled into wasm
  First attempt at fixing shared abcipp

.changelog/unreleased/bug-fixes/754-fix-abcipp.md

@@ -0,0 +1,2 @@
+- Fix building with the `abcipp` feature again
+  ([#754](https://github.com/anoma/namada/pull/754))

Makefile

@@ -41,6 +41,14 @@ check:
 	make -C $(wasms_for_tests) check && \
 	$(foreach wasm,$(wasm_templates),$(check-wasm) && ) true
 
+check-abcipp:
+	$(cargo) check \
+		--workspace \
+		--exclude namada_tests \
+		--all-targets \
+		--no-default-features \
+		--features "abcipp ibc-mocks-abcipp testing"
+
 clippy-wasm = $(cargo) +$(nightly) clippy --manifest-path $(wasm)/Cargo.toml --all-targets -- -D warnings
 
 clippy:
@@ -60,7 +68,7 @@ clippy-abcipp:
 	$(cargo) +$(nightly) clippy --all-targets \
 		--manifest-path ./shared/Cargo.toml \
 		--no-default-features \
-		--features "testing wasm-runtime abcipp ibc-mocks-abcipp" && \
+		--features "testing wasm-runtime abcipp ibc-mocks-abcipp ferveo-tpke" && \
 	$(cargo) +$(nightly) clippy \
 		--all-targets \
 		--manifest-path ./vm_env/Cargo.toml \

apps/src/lib/client/rpc.rs

@@ -131,7 +131,7 @@ pub async fn query_epoch(args: args::Query) -> Epoch {
 /// Query the last committed block
 pub async fn query_block(
     args: args::Query,
-) -> tendermint_rpc::endpoint::block::Response {
+) -> crate::facade::tendermint_rpc::endpoint::block::Response {
     let client = HttpClient::new(args.ledger_address).unwrap();
     let response = client.latest_block().await.unwrap();
     println!(

apps/src/lib/client/types.rs

@@ -8,11 +8,11 @@ use namada::types::masp::{TransferSource, TransferTarget};
 use namada::types::storage::Epoch;
 use namada::types::transaction::GasLimit;
 use namada::types::{key, token};
-use tendermint_config::net::Address as TendermintAddress;
 
 use super::rpc;
 use crate::cli::{args, Context};
 use crate::client::tx::Conversions;
+use crate::facade::tendermint_config::net::Address as TendermintAddress;
 
 #[derive(Clone, Debug)]
 pub struct ParsedTxArgs {

shared/Cargo.toml

@@ -60,12 +60,18 @@ tendermint-rpc = [
   "async-client",
   "dep:tendermint-rpc",
 ]
+tendermint-rpc-abcipp = [
+  "async-client",
+  "dep:tendermint-rpc-abcipp",
+]
 
 abcipp = [
   "ibc-proto-abcipp",
   "ibc-abcipp",
   "tendermint-abcipp",
-  "tendermint-proto-abcipp"
+  "tendermint-proto-abcipp",
+  # it's OK to include the tendermint-rpc feature here, as we aren't currently building wasms with `abcipp`
+  "tendermint-rpc-abcipp",
 ]
 
 abciplus = [

shared/src/ledger/queries/mod.rs

@@ -99,7 +99,7 @@ pub mod tm {
     #[derive(Error, Debug)]
     pub enum Error {
         #[error("{0}")]
-        Tendermint(#[from] tendermint_rpc::Error),
+        Tendermint(#[from] crate::tendermint_rpc::Error),
         #[error("Decoding error: {0}")]
         Decoding(#[from] std::io::Error),
         #[error("Info log: {0}, error code: {1}")]
@@ -109,7 +109,7 @@ pub mod tm {
     }
 
     #[async_trait::async_trait]
-    impl Client for tendermint_rpc::HttpClient {
+    impl Client for crate::tendermint_rpc::HttpClient {
         type Error = Error;
 
         async fn request(
@@ -122,11 +122,11 @@ pub mod tm {
             let data = data.unwrap_or_default();
             let height = height
                 .map(|height| {
-                    tendermint::block::Height::try_from(height.0)
+                    crate::tendermint::block::Height::try_from(height.0)
                         .map_err(|_err| Error::InvalidHeight(height))
                 })
                 .transpose()?;
-            let response = tendermint_rpc::Client::abci_query(
+            let response = crate::tendermint_rpc::Client::abci_query(
                 self,
                 // TODO open the private Path constructor in tendermint-rpc
                 Some(std::str::FromStr::from_str(&path).unwrap()),
@@ -136,12 +136,12 @@ pub mod tm {
             )
             .await?;
             match response.code {
-                tendermint::abci::Code::Ok => Ok(EncodedResponseQuery {
+                crate::tendermint::abci::Code::Ok => Ok(EncodedResponseQuery {
                     data: response.value,
                     info: response.info,
                     proof: response.proof,
                 }),
-                tendermint::abci::Code::Err(code) => {
+                crate::tendermint::abci::Code::Err(code) => {
                     Err(Error::Query(response.info, code))
                 }
             }

shared/src/ledger/queries/shell.rs

@@ -2,7 +2,6 @@ use borsh::{BorshDeserialize, BorshSerialize};
 use masp_primitives::asset_type::AssetType;
 use masp_primitives::merkle_tree::MerklePath;
 use masp_primitives::sapling::Node;
-use tendermint::merkle::proof::Proof;
 
 use crate::ledger::events::log::dumb_queries;
 use crate::ledger::events::Event;
@@ -11,6 +10,7 @@ use crate::ledger::queries::{require_latest_height, EncodedResponseQuery};
 use crate::ledger::storage::traits::StorageHasher;
 use crate::ledger::storage::{DBIter, DB};
 use crate::ledger::storage_api::{self, ResultExt, StorageRead};
+use crate::tendermint::merkle::proof::Proof;
 use crate::types::address::Address;
 use crate::types::hash::Hash;
 use crate::types::storage::{self, BlockResults, Epoch, PrefixValue};

shared/src/ledger/queries/types.rs

@@ -1,8 +1,7 @@
-use tendermint::merkle::proof::Proof;
-
 use crate::ledger::events::log::EventLog;
 use crate::ledger::storage::{DBIter, Storage, StorageHasher, DB};
 use crate::ledger::storage_api;
+use crate::tendermint::merkle::proof::Proof;
 use crate::types::storage::BlockHeight;
 #[cfg(feature = "wasm-runtime")]
 use crate::vm::wasm::{TxCache, VpCache};
@@ -148,12 +147,12 @@ impl RequestQuery {
     /// spec. A negative block height will cause an error.
     pub fn try_from_tm<D, H>(
         storage: &Storage<D, H>,
-        tendermint_proto::abci::RequestQuery {
+        crate::tendermint_proto::abci::RequestQuery {
             data,
             path,
             height,
             prove,
-        }: tendermint_proto::abci::RequestQuery,
+        }: crate::tendermint_proto::abci::RequestQuery,
     ) -> Result<Self, String>
     where
         D: DB + for<'iter> DBIter<'iter>,

shared/src/lib.rs

@@ -6,6 +6,10 @@
 #![deny(rustdoc::broken_intra_doc_links)]
 #![deny(rustdoc::private_intra_doc_links)]
 
+#[cfg(all(not(feature = "abcipp"), feature = "ferveo-tpke"))]
+pub use tendermint_rpc;
+#[cfg(all(feature = "abcipp", feature = "ferveo-tpke"))]
+pub use tendermint_rpc_abcipp as tendermint_rpc;
 #[cfg(not(feature = "abcipp"))]
 pub use {ibc, ibc_proto, tendermint, tendermint_proto};
 #[cfg(feature = "abcipp")]

shared/src/proto/types.rs

@@ -4,25 +4,14 @@ use std::hash::{Hash, Hasher};
 use borsh::{BorshDeserialize, BorshSchema, BorshSerialize};
 use prost::Message;
 use serde::{Deserialize, Serialize};
-#[cfg(not(feature = "ABCI"))]
-#[cfg(feature = "ferveo-tpke")]
-use tendermint_proto::abci::Event;
-#[cfg(not(feature = "ABCI"))]
-#[cfg(feature = "ferveo-tpke")]
-use tendermint_proto::abci::EventAttribute;
-#[cfg(not(feature = "ABCI"))]
-use tendermint_proto::abci::ResponseDeliverTx;
-#[cfg(feature = "ABCI")]
-#[cfg(feature = "ferveo-tpke")]
-use tendermint_proto_abci::abci::Event;
-#[cfg(feature = "ABCI")]
-#[cfg(feature = "ferveo-tpke")]
-use tendermint_proto_abci::abci::EventAttribute;
-#[cfg(feature = "ABCI")]
-use tendermint_proto_abci::abci::ResponseDeliverTx;
 use thiserror::Error;
 
 use super::generated::types;
+#[cfg(feature = "ferveo-tpke")]
+use crate::tendermint_proto::abci::Event;
+#[cfg(feature = "ferveo-tpke")]
+use crate::tendermint_proto::abci::EventAttribute;
+use crate::tendermint_proto::abci::ResponseDeliverTx;
 use crate::types::key::*;
 use crate::types::time::DateTimeUtc;
 #[cfg(feature = "ferveo-tpke")]