-
Notifications
You must be signed in to change notification settings - Fork 24.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
react-native-git-upgrade fails on git diff --binary HEAD~1 HEAD #12336
Comments
Does this happen on a fresh 0.38.0 install? |
From https://nodejs.org/api/child_process.html#child_process_event_exit:
It means that the @macnube Do you see a reason to have this process killed by a signal on your machine (Ctrl-C, watchdog or whatever)? Does it happen every time ? Anyway, it could be a good idea to improve the debug information in the exec function:
|
@lacker You know, I think this may have been an install that I downgraded back to 0.38.0 at some point because I was running into some other issues. I'll see what happens if I start from a fresh install with the project. @ncuillery Nothing comes to mind and it was happening every time, though admittedly I didn't more than 5 times. With the extra info you've provided, I'll keep an eye out to see if I can find the culprit and will report back soon. |
I did a clean install and now I'm running on the latest. |
Summary: When performing an upgrade using react-native-git-upgrade git can fail with a signal, however, only exit codes are accounted for. Related issue: #12336 In my case, Git fails with a `SIGPIPE` signal - have not figured out why yet, but that's a separate issue. Before, since exit code was not zero, the console would default to the error case, but since no exit code was supplied, the error message was meaningless - `exited with code null`. Now, it errors out with `terminated with signal 'SIGPIPE'`, while still taking account of exit codes. Quoting [nodejs docs](https://nodejs.org/api/child_process.html#child_process_event_exit): > The 'exit' event is emitted after the child process ends. If the process exited, code is the final exit code of the process, otherwise null. If the process terminated due to receipt of a signal, signal is the string name of the signal, otherwise null. One of the two will always be non-null. I would be happy to write a test case for this but I haven't seen any react-native-git-upgrade ones? [CLI] [ENHANCEMENT] [react-native/react-native-git-upgrade] - Git terminated by signal error message Closes #16822 Differential Revision: D6387451 Pulled By: shergin fbshipit-source-id: 25bf9dabdb5fda70d220bcd5f14c3c92bba8c3d4
Summary: When performing an upgrade using react-native-git-upgrade git can fail with a signal, however, only exit codes are accounted for. Related issue: facebook#12336 In my case, Git fails with a `SIGPIPE` signal - have not figured out why yet, but that's a separate issue. Before, since exit code was not zero, the console would default to the error case, but since no exit code was supplied, the error message was meaningless - `exited with code null`. Now, it errors out with `terminated with signal 'SIGPIPE'`, while still taking account of exit codes. Quoting [nodejs docs](https://nodejs.org/api/child_process.html#child_process_event_exit): > The 'exit' event is emitted after the child process ends. If the process exited, code is the final exit code of the process, otherwise null. If the process terminated due to receipt of a signal, signal is the string name of the signal, otherwise null. One of the two will always be non-null. I would be happy to write a test case for this but I haven't seen any react-native-git-upgrade ones? [CLI] [ENHANCEMENT] [react-native/react-native-git-upgrade] - Git terminated by signal error message Closes facebook#16822 Differential Revision: D6387451 Pulled By: shergin fbshipit-source-id: 25bf9dabdb5fda70d220bcd5f14c3c92bba8c3d4
Summary: When performing an upgrade using react-native-git-upgrade git can fail with a signal, however, only exit codes are accounted for. Related issue: facebook#12336 In my case, Git fails with a `SIGPIPE` signal - have not figured out why yet, but that's a separate issue. Before, since exit code was not zero, the console would default to the error case, but since no exit code was supplied, the error message was meaningless - `exited with code null`. Now, it errors out with `terminated with signal 'SIGPIPE'`, while still taking account of exit codes. Quoting [nodejs docs](https://nodejs.org/api/child_process.html#child_process_event_exit): > The 'exit' event is emitted after the child process ends. If the process exited, code is the final exit code of the process, otherwise null. If the process terminated due to receipt of a signal, signal is the string name of the signal, otherwise null. One of the two will always be non-null. I would be happy to write a test case for this but I haven't seen any react-native-git-upgrade ones? [CLI] [ENHANCEMENT] [react-native/react-native-git-upgrade] - Git terminated by signal error message Closes facebook#16822 Differential Revision: D6387451 Pulled By: shergin fbshipit-source-id: 25bf9dabdb5fda70d220bcd5f14c3c92bba8c3d4
Description
Attempting to upgrade react native from 0.38.0 to 0.39.2 using react-native-git-upgrade 0.39.2
Reproduction
Solution
No idea!
Additional Information
The text was updated successfully, but these errors were encountered: