Page MenuHomePhabricator
Differential D8657

[native] add CommRustModule implementation
ClosedPublic
Actions

Authored by kamil on Jul 28 2023, 9:54 AM.
Tags
None
Referenced Files
F2170733: D8657.id29185.diff
Tue, Jul 2, 3:34 PM
Unknown Object (File)
Mon, Jul 1, 1:02 AM
Unknown Object (File)
Sun, Jun 30, 4:59 PM
Unknown Object (File)
Sun, Jun 30, 2:24 PM
Unknown Object (File)
Wed, Jun 26, 8:56 AM
Unknown Object (File)
Tue, Jun 25, 7:54 PM
Unknown Object (File)
Tue, Jun 25, 12:36 PM
Unknown Object (File)
Tue, Jun 25, 12:36 PM
View All 80 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
Lint Not Applicable
Unit
Tests Not Applicable

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/
10 lines

Diff 29412

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...