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

The cypress release process has the possibility to generate a package with a bin file with windows line endings. #19771

Closed
mjhenkes opened this issue Jan 19, 2022 · 4 comments · Fixed by #20023
Assignees

Comments

@mjhenkes
Copy link
Member

Current behavior

The windows build of the cypress package will produce a binary with windows line endings. If this package is released it will cause issues like #19747.

Desired behavior

Produce a binary without windows line endings.

Test code to reproduce

In CI cause the windows build to run last and check out the generated package.

Cypress Version

n/a

Other

We have been lucky previously to not have seen this with either the darwin or linux build finishing last.

I believe issue was introduced with #8483, so it has been a possibility for some time.

Options for correction include:
Prevent windows from setting windows line endings when building.
Prevent windows from pushing the generated package to s3
Use linux to build the package and push it to s3 prior to generating the binary build jobs for each os.

@tgriesser
Copy link
Member

We have been lucky previously to not have seen this with either the darwin or linux build finishing last.

We didn't build Windows with Circle until recently, so that might be why we haven't seen it?

@jennifer-shehane
Copy link
Member

I don't think that would have affected the timing to make Windows be a slower build. The instance we saw, it seems the Darwin build finished earlier than usual. I think it has literally been luck for 16 months since #8483 was merged.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Feb 4, 2022

The code for this is done in cypress-io/cypress#20023, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot cypress-bot bot added stage: pending release and removed stage: needs review The PR code is done & tested, needs review labels Feb 4, 2022
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Feb 15, 2022

Released in 9.5.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v9.5.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants