[scripts] remove cargo-lipo from rustup setup
ClosedPublic
Actions

Authored by varun on Oct 30 2023, 9:55 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 26, 4:34 AM

	Unknown Object (File)
	Tue, Nov 26, 1:21 AM

	Unknown Object (File)
	Fri, Nov 22, 9:50 PM

	Unknown Object (File)
	Fri, Nov 8, 10:04 PM

	Unknown Object (File)
	Fri, Nov 8, 10:04 PM

	Unknown Object (File)
	Fri, Nov 8, 10:01 PM

	Unknown Object (File)
	Tue, Nov 5, 2:13 AM

	Unknown Object (File)
	Oct 17 2024, 2:21 AM

View All 48 Files

Subscribers

tomek

will

Details

Reviewers

ashoat

Commits

rCOMM303a00eedd94: [scripts] remove cargo-lipo from rustup setup

Summary

we don't need this anymore since we're building a target-specific native_rust_library now

Test Plan

uninstalled cargo-lipo and successfully built the iOS app. confirmed that cargo-lipo still wasn't installed on my machine after build

Diff Detail

Repository

rCOMM Comm

Branch

new-master-2 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Oct 30 2023, 9:55 AM

Herald added subscribers: will, tomek. · View Herald TranscriptOct 30 2023, 9:55 AM

Harbormaster completed remote builds in B23574: Diff 32504.Oct 30 2023, 10:36 AM

varun requested review of this revision.Oct 30 2023, 10:36 AM

ashoat accepted this revision.Oct 30 2023, 11:30 AM

This revision is now accepted and ready to land.Oct 30 2023, 11:30 AM

Closed by commit rCOMM303a00eedd94: [scripts] remove cargo-lipo from rustup setup (authored by varun). · Explain WhyOct 30 2023, 2:32 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM303a00eedd94: [scripts] remove cargo-lipo from rustup setup.

Revision Contents
Changeset List

Path

Size

scripts/

ensure_rustup_setup.sh

4 lines

Diff 32504

View Options

scripts/ensure_rustup_setup.sh

[scripts] remove cargo-lipo from rustup setupClosedPublicActions