Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

Upcoming v0.63.5 cherrypicks discussion #218

Closed
5 tasks
kelset opened this issue Nov 30, 2020 · 23 comments
Closed
5 tasks

Upcoming v0.63.5 cherrypicks discussion #218

kelset opened this issue Nov 30, 2020 · 23 comments
Labels
backport request Cherry picking a change into an existing release stable Stable version

Comments

@kelset
Copy link
Member

kelset commented Nov 30, 2020

Conversations on this thread are limited to 0.63 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.63.4 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 https://github.com/facebook/react-native/issues.


Note: we don't know yet if we will work on a 0.63.5 release, it's basically based on how things shape up with 0.64.


Update: it's unlikely we'll do a 0.63 patch, but in case here's the list of commits requested/wanted/needed:

Stretch goals:

@kelset kelset added stable Stable version backport request Cherry picking a change into an existing release labels Nov 30, 2020
@grabbou
Copy link
Member

grabbou commented Dec 14, 2020

When facebook/react-native#30543 lands, we will need to cherry-pick it to this branch too. It is affecting users on older React Native versions as well that upgraded to latest Xcode.

Edit: there it is: facebook/react-native@fdcacd7

@grabbou
Copy link
Member

grabbou commented Dec 15, 2020

I am going to cherry-pick the aforementioned commit and test whether it fixes the issue on this branch as well, as a part of my testing of the same commit for the 0.64 branch!

@grabbou
Copy link
Member

grabbou commented Dec 15, 2020

This should also be cherry-picked facebook/react-native#30592 to avoid adding a whole new post install hook to a Podfile.

@retyui
Copy link

retyui commented Dec 27, 2020

Really wait for these changes facebook/react-native@b7b0e23

To have an opportunity to get tap location on the <TextInput/> component

@kelset
Copy link
Member Author

kelset commented Jan 4, 2021

@retyui that commit adds new functionality so it's not a candidate for a patch release.

@sandfox
Copy link

sandfox commented Jan 5, 2021

Could we backport in facebook/react-native@d85d5d2 from facebook/react-native#29882
It's a fix for Modal onDismiss behaviour. It's already in 0.64

@kelset
Copy link
Member Author

kelset commented Jan 5, 2021

@sandfox I've already attempted cherry-picking it during the last round, but it's really complicated because the files involved changed a lot - so it's not achievable. The only option would be for someone to do a PR against the 0.63 branch with the fix "repurposed" to work on it.

@sandfox
Copy link

sandfox commented Jan 5, 2021

@kelset I'll see if I can do that then: facebook/react-native#30688
So far it's working properly on our internal test case

@priteshrnandgaonkar
Copy link

priteshrnandgaonkar commented Feb 15, 2021

Just a reminder, please use the latest Flipper pod, as we have solved the issues related to Apple M1 machines and arm compatibilities. Latest Flipper pod is 0.75.1

@mrousavy
Copy link
Member

@priteshrnandgaonkar a bit off-topic, but does the latest Flipper pod also fix facebook/react-native#30836?

@kelset
Copy link
Member Author

kelset commented Feb 16, 2021

@mrousavy from the comments in the issue seems like it did. In my experience, you can just change your podfile to

use_flipper!('Flipper' => '0.75.1')

To change the Flipper version

@kelset
Copy link
Member Author

kelset commented Feb 25, 2021

potential cherry pick: facebook/react-native@3399896

@retyui
Copy link

retyui commented Mar 9, 2021

p.s. @kelset seem you recommend a legacy ruby syntax

-use_flipper!('Flipper' => '0.75.1')
+use_flipper!({ 'Flipper' => '0.75.1' })

@retyui
Copy link

retyui commented Mar 9, 2021

Still waiting for a fix for Modal::onDismiss on iOS facebook/react-native@d85d5d2

@kelset
Copy link
Member Author

kelset commented Mar 10, 2021

We are getting closer to 0.64.0 so atm we are fully focused on that - no ETA for a new patch for 0.63.x.

@sandfox
Copy link

sandfox commented Mar 10, 2021

@retyui - there's an open (mine) PR for it facebook/react-native#30688 that can be used if 0.63.5 gets the go ahead (assuming poeple are ok with merging it of course).

@kelset
Copy link
Member Author

kelset commented Mar 16, 2021

potentially useful to consider: facebook/react-native@0e9296b

@anchi20
Copy link

anchi20 commented Mar 19, 2021

here, facebook/react-native#29070

@tido64
Copy link
Collaborator

tido64 commented Mar 19, 2021

@anchi20: Please read the first post:

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

Please let us know when your PR has landed on master.

@kelset
Copy link
Member Author

kelset commented Apr 29, 2021

update: it's not likely we'll do a 0.63.5 patch, but since the Xcode 12.5 issue is fairly big it could be the case. We are still waiting for a commit to land in master, but I've started to update the top post with the list of commits we should cherry pick.

@todorone
Copy link

todorone commented May 5, 2021

@kelset Not everybody can migrate to 0.64 yet, so releasing a fix to XCode 12.5 will be a lifesaver... 🥺

@kelset
Copy link
Member Author

kelset commented May 5, 2021

@todorone if you are on 0.63 you only need to tweak your podfile to address the Xcode 12.5 issue, see details here: facebook/react-native#31480

@kelset
Copy link
Member Author

kelset commented May 26, 2021

Given how the update for Xcode 12.5 didn't require an update for 0.63, I'm going to close this off. We are currently focused on 0.64 and 0.65 so there won't be new 0.63 releases for the time being.

@kelset kelset closed this as completed May 26, 2021
@react-native-community react-native-community locked as resolved and limited conversation to collaborators May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport request Cherry picking a change into an existing release stable Stable version
Projects
None yet
Development

No branches or pull requests

9 participants