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

AVRO-3859: build.sh clean fails to remove some C# files #2500

Merged
merged 1 commit into from
Sep 18, 2023
Merged

AVRO-3859: build.sh clean fails to remove some C# files #2500

merged 1 commit into from
Sep 18, 2023

Conversation

zcsizmadia
Copy link
Contributor

What is the purpose of the change

  • Make sure that build.sh clean removes all bin and obj folders (AVRO-3859)

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable)

@zcsizmadia
Copy link
Contributor Author

zcsizmadia commented Sep 18, 2023

@RyanSkraba Quick fix for AVRO-3859. I followed the pattern which is already in the clean step. Personally I like using git clean -dfx for such cleanings. It deletes all untracked and unwanted files, and by honoring the .gitignore file(s).

In that case the clean step would be just a simple git clean -dxf and not multiple rm commands.

@zcsizmadia zcsizmadia changed the title AE-3859: build.sh clean fails to remove some C# files AVRO-3859: build.sh clean fails to remove some C# files Sep 18, 2023
Copy link
Contributor

@RyanSkraba RyanSkraba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM -- so fast! Thanks :D

I do the exact thing (git clean -fdx) but it can be a bit dangerous to put in a script during development (to distinguish between a new source file and a new compiled file). Do you know if there's a way to clean that only remove things that would otherwise be .gitignore'd?

@zcsizmadia zcsizmadia merged commit ae05154 into apache:master Sep 18, 2023
6 checks passed
@zcsizmadia zcsizmadia deleted the avro-3859-clean-csharp-bin-obj branch September 18, 2023 15:10
@zcsizmadia
Copy link
Contributor Author

git clean -dXf

Note the capital X

@RyanSkraba
Copy link
Contributor

🤯 Today I learned!! Thanks for this! https://issues.apache.org/jira/browse/AVRO-3864

RanbirK pushed a commit to RanbirK/avro that referenced this pull request May 13, 2024
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