feat: ibc-app-transfer-types should not lead to dependency on tenderming

[dzmitry-lahoda]

Feature Summary

here is my part of cargo tree:

  ├── ibc-app-transfer-types v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   ├── displaydoc v0.2.4 (proc-macro)
│   │   │   ├── proc-macro2 v1.0.70 (*)
│   │   │   ├── quote v1.0.33 (*)
│   │   │   └── syn v2.0.39 (*)
│   │   ├── ibc-core v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   ├── ibc-core-channel v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   │   ├── ibc-core-channel-types v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   │   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   │   │   │   ├── displaydoc v0.2.4 (proc-macro) (*)
│   │   │   │   │   ├── ibc-core-client-types v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   │   │   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   │   │   │   │   ├── displaydoc v0.2.4 (proc-macro) (*)
│   │   │   │   │   │   ├── ibc-core-commitment-types v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   │   │   │   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   │   │   │   │   │   ├── displaydoc v0.2.4 (proc-macro) (*)
│   │   │   │   │   │   │   ├── ibc-primitives v0.48.1 (https://github.com/dzmitry-lahoda-forks/ibc-rs.git?branch=dz/14#1f7bdc66)
│   │   │   │   │   │   │   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   │   │   │   │   │   │   ├── displaydoc v0.2.4 (proc-macro) (*)
│   │   │   │   │   │   │   │   ├── ibc-proto v0.38.0
│   │   │   │   │   │   │   │   │   ├── base64 v0.21.5
│   │   │   │   │   │   │   │   │   ├── bytes v1.5.0
│   │   │   │   │   │   │   │   │   │   └── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │   ├── flex-error v0.4.4
│   │   │   │   │   │   │   │   │   │   └── paste v1.0.14 (proc-macro)
│   │   │   │   │   │   │   │   │   ├── ics23 v0.11.0
│   │   │   │   │   │   │   │   │   │   ├── anyhow v1.0.75
│   │   │   │   │   │   │   │   │   │   ├── bytes v1.5.0 (*)
│   │   │   │   │   │   │   │   │   │   ├── hex v0.4.3 (*)
│   │   │   │   │   │   │   │   │   │   ├── informalsystems-pbjson v0.6.0
│   │   │   │   │   │   │   │   │   │   │   ├── base64 v0.13.1
│   │   │   │   │   │   │   │   │   │   │   └── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │   │   ├── prost v0.12.3
│   │   │   │   │   │   │   │   │   │   │   ├── bytes v1.5.0 (*)
│   │   │   │   │   │   │   │   │   │   │   └── prost-derive v0.12.3 (proc-macro)
│   │   │   │   │   │   │   │   │   │   │       ├── anyhow v1.0.75
│   │   │   │   │   │   │   │   │   │   │       ├── itertools v0.11.0
│   │   │   │   │   │   │   │   │   │   │       │   └── either v1.9.0
│   │   │   │   │   │   │   │   │   │   │       ├── proc-macro2 v1.0.70 (*)
│   │   │   │   │   │   │   │   │   │   │       ├── quote v1.0.33 (*)
│   │   │   │   │   │   │   │   │   │   │       └── syn v2.0.39 (*)
│   │   │   │   │   │   │   │   │   │   ├── ripemd v0.1.3
│   │   │   │   │   │   │   │   │   │   │   └── digest v0.10.7 (*)
│   │   │   │   │   │   │   │   │   │   ├── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │   │   ├── sha2 v0.10.8 (*)
│   │   │   │   │   │   │   │   │   │   └── sha3 v0.10.8
│   │   │   │   │   │   │   │   │   │       ├── digest v0.10.7 (*)
│   │   │   │   │   │   │   │   │   │       └── keccak v0.1.4
│   │   │   │   │   │   │   │   │   ├── parity-scale-codec v3.6.5
│   │   │   │   │   │   │   │   │   │   ├── arrayvec v0.7.4
│   │   │   │   │   │   │   │   │   │   ├── byte-slice-cast v1.2.2
│   │   │   │   │   │   │   │   │   │   ├── impl-trait-for-tuples v0.2.2 (proc-macro)
│   │   │   │   │   │   │   │   │   │   │   ├── proc-macro2 v1.0.70 (*)
│   │   │   │   │   │   │   │   │   │   │   ├── quote v1.0.33 (*)
│   │   │   │   │   │   │   │   │   │   │   └── syn v1.0.109 (*)
│   │   │   │   │   │   │   │   │   │   └── parity-scale-codec-derive v3.6.5 (proc-macro)
│   │   │   │   │   │   │   │   │   │       ├── proc-macro-crate v1.3.1
│   │   │   │   │   │   │   │   │   │       │   ├── once_cell v1.18.0
│   │   │   │   │   │   │   │   │   │       │   └── toml_edit v0.19.15
│   │   │   │   │   │   │   │   │   │       │       ├── indexmap v2.1.0
│   │   │   │   │   │   │   │   │   │       │       │   ├── equivalent v1.0.1
│   │   │   │   │   │   │   │   │   │       │       │   └── hashbrown v0.14.3
│   │   │   │   │   │   │   │   │   │       │       ├── toml_datetime v0.6.5
│   │   │   │   │   │   │   │   │   │       │       └── winnow v0.5.19
│   │   │   │   │   │   │   │   │   │       ├── proc-macro2 v1.0.70 (*)
│   │   │   │   │   │   │   │   │   │       ├── quote v1.0.33 (*)
│   │   │   │   │   │   │   │   │   │       └── syn v1.0.109 (*)
│   │   │   │   │   │   │   │   │   ├── prost v0.12.3 (*)
│   │   │   │   │   │   │   │   │   ├── scale-info v2.10.0
│   │   │   │   │   │   │   │   │   │   ├── cfg-if v1.0.0
│   │   │   │   │   │   │   │   │   │   ├── derive_more v0.99.17 (proc-macro) (*)
│   │   │   │   │   │   │   │   │   │   ├── parity-scale-codec v3.6.5 (*)
│   │   │   │   │   │   │   │   │   │   └── scale-info-derive v2.10.0 (proc-macro)
│   │   │   │   │   │   │   │   │   │       ├── proc-macro-crate v1.3.1 (*)
│   │   │   │   │   │   │   │   │   │       ├── proc-macro2 v1.0.70 (*)
│   │   │   │   │   │   │   │   │   │       ├── quote v1.0.33 (*)
│   │   │   │   │   │   │   │   │   │       └── syn v1.0.109 (*)
│   │   │   │   │   │   │   │   │   ├── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │   ├── subtle-encoding v0.5.1
│   │   │   │   │   │   │   │   │   │   └── zeroize v1.7.0 (*)
│   │   │   │   │   │   │   │   │   └── tendermint-proto v0.34.0
│   │   │   │   │   │   │   │   │       ├── bytes v1.5.0 (*)
│   │   │   │   │   │   │   │   │       ├── flex-error v0.4.4 (*)
│   │   │   │   │   │   │   │   │       ├── num-derive v0.3.3 (proc-macro)
│   │   │   │   │   │   │   │   │       │   ├── proc-macro2 v1.0.70 (*)
│   │   │   │   │   │   │   │   │       │   ├── quote v1.0.33 (*)
│   │   │   │   │   │   │   │   │       │   └── syn v1.0.109 (*)
│   │   │   │   │   │   │   │   │       ├── num-traits v0.2.17
│   │   │   │   │   │   │   │   │       │   [build-dependencies]
│   │   │   │   │   │   │   │   │       │   └── autocfg v1.1.0
│   │   │   │   │   │   │   │   │       ├── prost v0.12.3 (*)
│   │   │   │   │   │   │   │   │       ├── prost-types v0.12.3
│   │   │   │   │   │   │   │   │       │   └── prost v0.12.3 (*)
│   │   │   │   │   │   │   │   │       ├── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │       ├── serde_bytes v0.11.12
│   │   │   │   │   │   │   │   │       │   └── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   │       ├── subtle-encoding v0.5.1 (*)
│   │   │   │   │   │   │   │   │       └── time v0.3.30
│   │   │   │   │   │   │   │   │           ├── deranged v0.3.9
│   │   │   │   │   │   │   │   │           │   └── powerfmt v0.2.0
│   │   │   │   │   │   │   │   │           ├── powerfmt v0.2.0
│   │   │   │   │   │   │   │   │           ├── time-core v0.1.2
│   │   │   │   │   │   │   │   │           └── time-macros v0.2.15 (proc-macro)
│   │   │   │   │   │   │   │   │               └── time-core v0.1.2
│   │   │   │   │   │   │   │   ├── parity-scale-codec v3.6.5 (*)
│   │   │   │   │   │   │   │   ├── prost v0.12.3 (*)
│   │   │   │   │   │   │   │   ├── scale-info v2.10.0 (*)
│   │   │   │   │   │   │   │   ├── serde v1.0.193 (*)
│   │   │   │   │   │   │   │   ├── tendermint v0.34.0
│   │   │   │   │   │   │   │   │   ├── bytes v1.5.0 (*)
│   │   │   │   │   │   │   │   │   ├── digest v0.10.7 (*)

So it is visible that somehow tendermint is in path via ibc-core and ibc-primitives, while does not seems ibc-app-transfer-types should have lead to this.

Proposal

Check if that dep can be eliminated

[dzmitry-lahoda]

here https://github.com/cosmos/ibc-rs/blob/eafdd300727ebc310b2189406cd0bc50f94d2c67/ibc-primitives/src/types/timestamp.rs#L11C22-L11C22

[dzmitry-lahoda]

it is just wrapper on wrapper https://github.com/informalsystems/tendermint-rs/blob/194d81ec23fae7f2aad341895044e9c970eb2cd5/tendermint/src/time.rs#L39C24-L39C24

[Farhad-Shabani]

That's very true. We are aware of this actually. Relevant issues for more context:

• Investigate removing the dependency on tendermint-rs #459
• Timestamp define: not be strongly coupled with the time in tendermint #180

Just a matter of checking our bandwidth. Let's see where this lands on our to-do list.

[dzmitry-lahoda]

Making time optional is best for my case.