Page MenuHomePhabricator
Differential D6785

[native] Fix Buildkite iOS build to work from inside Nix
ClosedPublic
Actions

Authored by ashoat on Feb 20 2023, 10:47 AM.
Tags
None
Referenced Files
F2094839: D6785.id22737.diff
Mon, Jun 24, 5:59 AM
F2088852: D6785.id22733.diff
Sun, Jun 23, 5:48 PM
Unknown Object (File)
Sat, Jun 22, 10:28 AM
Unknown Object (File)
Wed, Jun 19, 1:33 PM
Unknown Object (File)
Mon, Jun 17, 12:21 PM
Unknown Object (File)
Mon, Jun 17, 11:23 AM
Unknown Object (File)
Sun, Jun 16, 3:42 AM
Unknown Object (File)
Wed, Jun 12, 5:02 PM
View All Files
Subscribers
tomek

Details

Reviewers
atul
• jon
Commits
rCOMM4adee5e8f27e: [native] Fix Buildkite iOS build to work from inside Nix
Summary

We need the same patch we use in native/package.json for yarn react-native and yarn expo.

Test Plan

Let's see if this diff's Buildkite build succeeds :)

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/fix_ios_buildkite
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Feb 20 2023, 10:47 AM
Herald added a subscriber: tomek. · View Herald TranscriptFeb 20 2023, 10:47 AM
atul published this revision for review.Feb 20 2023, 10:50 AM
Harbormaster failed remote builds in B16625: Diff 22733!Feb 20 2023, 10:50 AM
ashoat updated this revision to Diff 22735.Feb 20 2023, 11:19 AM

Move $PATH hack to package.json. I suspect Buildkite is doing something wonky

Harbormaster failed remote builds in B16627: Diff 22735!Feb 20 2023, 11:31 AM
ashoat updated this revision to Diff 22736.Feb 20 2023, 11:35 AM

Seeing this issue because macOS's cp command is in /bin, and so we end up using the GNU one instead

This revision updates us to prefix $PATH with not just /usr/bin, but also /bin, so that we have access to BSD cp

ashoat updated this revision to Diff 22737.Feb 20 2023, 11:45 AM

It worked! Adding /bin to all of the places we prefix /usr/bin for future-proofing

Harbormaster completed remote builds in B16628: Diff 22736.Feb 20 2023, 11:47 AM
ashoat mentioned this in D6783: Update Prettier.Feb 20 2023, 11:47 AM
ashoat mentioned this in D6784: [keyserver] Fix rust-node-addon after deprecation of --experimental-specifier-resolution=node.
ashoat added a child revision: D6783: Update Prettier.Feb 20 2023, 11:47 AM
• jon accepted this revision.Feb 20 2023, 11:53 AM
This revision is now accepted and ready to land.Feb 20 2023, 11:53 AM
Harbormaster completed remote builds in B16629: Diff 22737.Feb 20 2023, 12:00 PM
atul accepted this revision.Feb 20 2023, 12:05 PM
Closed by commit rCOMM4adee5e8f27e: [native] Fix Buildkite iOS build to work from inside Nix (authored by ashoat). · Explain WhyFeb 20 2023, 12:25 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM4adee5e8f27e: [native] Fix Buildkite iOS build to work from inside Nix.

Revision Contents
Changeset List

PathSize
.buildkite/
2 lines

Diff 22733

View Options

.buildkite/ios.yml

Loading...