Page MenuHomePhabricator
Differential D8657

[native] add CommRustModule implementation
ClosedPublic
Actions

Authored by kamil on Jul 28 2023, 9:54 AM.
Tags
None
Referenced Files
F3525929: D8657.id29408.diff
Mon, Dec 23, 6:50 PM
F3525855: D8657.id29185.diff
Mon, Dec 23, 6:34 PM
F3524395: D8657.diff
Mon, Dec 23, 12:34 PM
Unknown Object (File)
Thu, Dec 12, 9:42 PM
Unknown Object (File)
Mon, Nov 25, 3:00 PM
Unknown Object (File)
Mon, Nov 25, 3:00 PM
Unknown Object (File)
Nov 23 2024, 12:54 PM
Unknown Object (File)
Nov 23 2024, 12:54 PM
View All Files
Subscribers
ashoat

Details

Reviewers
bartek
michal
tomek
varun
Commits
rCOMM1cf20e92d07a: [native] add CommRustModule implementation
Summary

ENG-4232

Creating a separate module for Rust API. This is a part of CommCoreModule refactor.

Methods implementation is a copy from CommCoreModule (but without changes to CommCoreModule yet).

Depends on D8656

Test Plan

Build Android nad iOS

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Jul 28 2023, 9:54 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJul 28 2023, 9:54 AM
kamil added a child revision: D8658: [native] install CommRustModule.Jul 28 2023, 9:55 AM
kamil published this revision for review.Jul 28 2023, 10:21 AM
Harbormaster failed remote builds in B21309: Diff 29185!Jul 28 2023, 10:41 AM
bartek accepted this revision.Jul 31 2023, 2:46 AM
This revision is now accepted and ready to land.Jul 31 2023, 2:46 AM
kamil updated this revision to Diff 29289.Jul 31 2023, 8:52 AM

rebase

Harbormaster completed remote builds in B21377: Diff 29289.Jul 31 2023, 10:14 AM
kamil updated this revision to Diff 29408.Aug 1 2023, 1:25 AM

rebase once more

Harbormaster completed remote builds in B21446: Diff 29408.Aug 1 2023, 1:45 AM
Closed by commit rCOMM1cf20e92d07a: [native] add CommRustModule implementation (authored by kamil). · Explain WhyAug 1 2023, 2:17 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM1cf20e92d07a: [native] add CommRustModule implementation.
kamil mentioned this in D8692: [native] fix Xcode dependencies order.Aug 1 2023, 2:38 AM
kamil mentioned this in rCOMM6ec64752932e: [native] fix Xcode dependencies order.Aug 1 2023, 3:04 AM
tomek mentioned this in D10449: [native] Expose get keyserver keys Identity RPC.Dec 22 2023, 7:18 AM

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
2 lines
57 lines
176 lines
ios/
Comm.xcodeproj/
6 lines

Diff 29185

View Options

native/cpp/CommonCpp/NativeModules/CMakeLists.txt

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.h

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.cpp

Loading...
View Options

native/ios/Comm.xcodeproj/project.pbxproj

Loading...