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

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

Tags

None

Referenced Files

	F2201298: D9635.id32515.diff
	Sat, Jul 6, 3:16 AM

	F2200679: D9635.id.diff
	Sat, Jul 6, 12:42 AM

	Unknown Object (File)
	Wed, Jul 3, 8:33 AM

	Unknown Object (File)
	Tue, Jun 11, 9:43 PM

	Unknown Object (File)
	Mon, Jun 10, 7:27 PM

	Unknown Object (File)
	Apr 19 2024, 6:27 PM

	Unknown Object (File)
	Apr 4 2024, 5:33 PM

	Unknown Object (File)
	Apr 1 2024, 5:42 AM

View All 21 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 32515

View Options

scripts/ensure_rustup_setup.sh

[scripts] remove cargo-lipo from rustup setupClosedPublicActions