Skip to content
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

[3.31.0] CI builds fails, command not found: bob #2108

Closed
efstathiosntonas opened this issue Apr 23, 2024 · 15 comments · Fixed by #2109
Closed

[3.31.0] CI builds fails, command not found: bob #2108

efstathiosntonas opened this issue Apr 23, 2024 · 15 comments · Fixed by #2109
Assignees
Labels
Missing repro This issue need minimum repro scenario Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS

Comments

@efstathiosntonas
Copy link

Description

After running yarn install on CI on a macos 14 machine the install fails with:

react-native-screens@npm:3.31.0 [e76ba] couldn't be built successfully (exit code 127, logs can be found here: /private/var/folders/cm/2r_39vcd593436b3_ht4nyqw0000gp/T/xfs-1e614b9e/build.log)

the log states:

command not found: bob

Steps to reproduce

  1. I'm using yarn 4.1.1
  2. use latest version of react-native-screens 3.31.0
  3. yarn install fails

Snack or a link to a repository

no-repo-needed

Screens version

3.31.0

React Native version

0.73.7

Platforms

Android, iOS

JavaScript runtime

Hermes

Workflow

React Native (without Expo)

Architecture

Paper (Old Architecture)

Build type

Release mode

Device

Real device

Device model

No response

Acknowledgements

Yes

@github-actions github-actions bot added Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS Missing repro This issue need minimum repro scenario labels Apr 23, 2024
Copy link

Hey! 👋

The issue doesn't seem to contain a minimal reproduction.

Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

@keuvy
Copy link

keuvy commented Apr 23, 2024

just happened to me too

node_modules/react-native-screens: Command failed. Exit code: 127 Command: bob build && husky install

/bin/sh: bob: command not found

@mdenizkaba
Copy link

mdenizkaba commented Apr 23, 2024

Command failed.
Exit code: 127
Command: bob build && husky install
Arguments:
Directory: /Users/******/node_modules/react-native-screens
Output:
/bin/sh: bob: command not found

@copiiinux
Copy link

Experiencing the same behaviour on Windows and RN@0.74

npm ERR! code 1
npm ERR! path C:\Users\XXXX\projects\YYYYYYY\node_modules\react-native-screens
npm ERR! command failed
npm ERR! command C:\WINDOWS\system32\cmd.exe /d /s /c bob build && husky install
npm ERR! 'bob' is not recognized as an internal or external command,
npm ERR! operable program or batch file.
[...]
Process finished with exit code 1

@FantWings
Copy link

same here react native 0.74.0
error /Users/zhijun/Documents/github/furries-client-rn/node_modules/react-native-screens: Command failed.
Exit code: 127
Command: bob build && husky install
Arguments:
Directory: /Users/zhijun/Documents/github/furries-client-rn/node_modules/react-native-screens

@coawazie
Copy link

Command failed. Exit code: 127 Command: bob build && husky install Arguments: Directory: /Users/******/node_modules/react-native-screens Output: /bin/sh: bob: command not found

Same! Happened after I upgraded to rn 0.74.0

@tarunsmartdata
Copy link

error /node_modules/react-native-screens: Command failed.
Exit code: 127
Command: bob build && husky install
Arguments:
Directory: /node_modules/react-native-screens
Output:

@efstathiosntonas
Copy link
Author

efstathiosntonas commented Apr 23, 2024

can someone install react-native-builder-bob globally and try to install rn-screens afterwards?

I’m afk and can’t test it at the moment 🥴

@Acanguven
Copy link

@efstathiosntonas

npm ERR! command sh -c bob build && husky install
npm ERR! ℹ Building target commonjs
npm ERR! ℹ Cleaning up previous build at lib/commonjs
npm ERR! ℹ Compiling 61 files in src with babel
npm ERR! ✔ Wrote files to lib/commonjs
npm ERR! ℹ Building target module
npm ERR! ℹ Cleaning up previous build at lib/module
npm ERR! ℹ Compiling 61 files in src with babel
npm ERR! ✔ Wrote files to lib/module
npm ERR! ℹ Building target typescript
npm ERR! ℹ Cleaning up previous build at lib/typescript
npm ERR! ℹ Generating type definitions with tsc
npm ERR! ✖ Couldn't find a tsconfig.json in the project root.
npm ERR! bob build

@tboba tboba self-assigned this Apr 23, 2024
@mehmetdemiray
Copy link

mehmetdemiray commented Apr 23, 2024

I solved problem with changing version for now. I think this version is working when build

3.31.0-rc.1

npm i react-native-screens@3.31.0-rc.1
or
yarn add react-native-screens@3.31.0-rc.1

I haven't encountered any problems during runtime yet.

@zerobertelprivat
Copy link

Hello,

same problem here with version 3.31.0 (using RN 0.74.0 / yarn 1.22.22).
Cannot do "yarn add react-native-screens" ....
The problem doesnt exit with version 3.31.0-rc.1

Thank you,
Robert

@tboba
Copy link
Member

tboba commented Apr 23, 2024

Hello everyone! We're currently aware of this issue with @kkafar. It was mistake on our side, since we've bumped Yarn to v4 version and didn't realize that it triggers postinstall step 😄
Sorry, we'll be releasing a patch version in upcoming hour.

@Henus321
Copy link

Henus321 commented Apr 23, 2024

npm ERR! code 127
npm ERR! path node_modules/react-native-screens
npm ERR! command failed
npm ERR! command sh -c bob build && husky install
npm ERR! sh: bob: command not found

tboba added a commit that referenced this issue Apr 23, 2024
… package.json (#2109)

## Description

This PR intents to fix error, during the install step on other React
Native applications. That's because of the name of the step - it
shouldn't be named as `postinstall`, as method named this way is
triggered by yarn v4. This was not an issue before, as we used yarn v1.

Fixes #2108.

## Changes

- Changed `postinstall` name to `prepare` in package.json
- Added `yarn prepare` command during the `yarn release`.

## Checklist

- [ ] Ensured that CI passes
@kkafar
Copy link
Member

kkafar commented Apr 23, 2024

@tboba Released 3.31.1. Let us know, whether the issue disappeared

@gremech
Copy link

gremech commented Apr 23, 2024

The issue is solved in 3.31.1

ja1ns pushed a commit to WiseOwlTech/react-native-screens that referenced this issue Oct 9, 2024
… package.json (software-mansion#2109)

## Description

This PR intents to fix error, during the install step on other React
Native applications. That's because of the name of the step - it
shouldn't be named as `postinstall`, as method named this way is
triggered by yarn v4. This was not an issue before, as we used yarn v1.

Fixes software-mansion#2108.

## Changes

- Changed `postinstall` name to `prepare` in package.json
- Added `yarn prepare` command during the `yarn release`.

## Checklist

- [ ] Ensured that CI passes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing repro This issue need minimum repro scenario Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS
Projects
None yet