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

ignore rcedit-x86.exe. It gets downloaded when windows package is built. #19192

Closed
wants to merge 1 commit into from

Conversation

copyrights
Copy link

rcedit-x86.exe is downloaded when building windows package.

I added it to .gitignore to prevent accidental commits.

Acknowledgement: No entry in AUTHORS is required as it is not code contribution.

@pchote
Copy link
Member

pchote commented Mar 7, 2021

The convention in the packaging scripts is to delete any temporary files they create or download. This one was just forgotten. Would you mind changing this PR to instead rm -rf "${BUILTDIR}" rcedit-x64.exe in the windows buildpackage.sh?

@pchote
Copy link
Member

pchote commented Mar 27, 2021

I filed #19293 implementing my requested approach.

@pchote pchote closed this Mar 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants