Upcoming v0.64.3 cherrypicks discussion

[kelset]

Conversations on this thread are limited to 0.64 releases major issues and backport (cherry-pick) requests from commits that are already on master.

An example of a good such request is a bug fix for a serious issue that has been merged into master but did not make the 0.64.2 cut, with a link to the specific commit hash on master with the commit to cherry-pick, like this example link: facebook/react-native@bd2b7d6

In other words, if you cannot point to a particular commit on master, then your request likely belongs as a new issue in http://github.com/facebook/react-native/issues.

[omnikron]

Would it be possible to include facebook/react-native@bdfe2a5 , which fixes an issue with broken iOS builds (facebook/react-native#31193)? The commit is in master since 16th April but we are facing the same issue today after upgrading to 0.64.2. We added patch-package to apply the commit on top of 0.64.0 every time we build our app and can confirm that it works.

[danilobuerger]

@omnikron @kelset wasn't this already requested to be cherry picked in 0.64.2?

[omnikron]

@omnikron @kelset wasn't this already requested to be cherry picked in 0.64.2?

Yes, but it was not included due to merge conflicts. I didn't find the reference before I wrote my previous comment but it's here: #224 (comment)

Not sure whether the list of unmerged cherry-picks carries over from release to release - apologies if so.

[kelset]

We will also cherry pick this: facebook/react-native#31682

re: this commit facebook/react-native@bdfe2a5 as pointed out couldn't get cherry picked cleanly, so it's not bound to happen as is. What would need to happen is to have a dedicated PR that targets 0.64 and replicates the changes needed in a way that goes in cleanly.

[danilobuerger]

I have it as a commit in my fork that cleanly cherry-picks onto 0.64.2: feastr/react-native@d9f7770 if anybody wants to PR it to the 0.64 branch, feel free.

[omnikron]

I have it as a commit in my fork that cleanly cherry-picks onto 0.64.2: feastr/react-native@d9f7770 if anybody wants to PR it to the 0.64 branch, feel free.

I've done just that: facebook/react-native#31688

@kelset is this sufficient for cherry-picking? Happy to help in any other way.

[kelset]

We also want facebook/react-native@4d40b53 IIRC

[andreialecu]

I hope this is not dumb to ask for here, but updating to 0.65 will be a hard task for a lot of people, and this small fix would be incredible to have as part of 0.64.

Font weights have been broken on Android since forever, and this finally fixes them. It's been merged recently:

facebook/react-native#29117 (facebook/react-native@3827ca6)

[birkir]

Maybe we can include accessible colors for DynamicColorIOS in 64.3? facebook/react-native@4b9d9dd

[dskrzypczynski]

I think this commit is very important: facebook/react-native@70da640