Skip to content

Exclude Directory.Build.rsp from .gitignore template #42401

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

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

MattKotsenas
Copy link
Member

@MattKotsenas MattKotsenas commented Jul 26, 2024

Directory.Build.rsp is a documented file that allows setting default arguments to command line builds. However, our .gitignore template ignores all *.rsp files. which causes confusion:

  1. Devs write an .rsp file and if they aren't being attentive forget to commit it
  2. Adding it to git requires git add --force, which some devs mistake for a destructive or not-recommended action

Thus, explicitly allow the Directory.Build.rsp file.

@MattKotsenas MattKotsenas requested a review from a team as a code owner July 26, 2024 17:19
@ghost ghost added Area-Infrastructure untriaged Request triage from a team member labels Jul 26, 2024
@baronfel
Copy link
Member

We normally freeze templates by preview 6 or so, so checking with the rest of the team about applicability here. I still think we should take this, but we have some old scars from the .NET 6 release that we're working with here :)

@MattKotsenas
Copy link
Member Author

Pinging here for latest status. If this is frozen until after .NET 9 that's fine, just want to make sure that:

  1. We definitely don't want this for 9
  2. At what point I should resume nagging 😅

@baronfel baronfel added the needs team triage Requires a full team discussion label Aug 6, 2024
@marcpopMSFT marcpopMSFT added this to the 10.0.1xx milestone Aug 6, 2024
@marcpopMSFT
Copy link
Member

Triage: Ping us again in ~2 weeks after we've forked a 9.0.1xx branch for .NET9 servicing. We'll take this for 10 so as to avoid any late template changes (noting that this one is probably lower risk that source changes that have typically been the focus).

@MiYanni
Copy link
Member

MiYanni commented Aug 21, 2024

@marcpopMSFT

Triage: Ping us again in ~2 weeks after we've forked a 9.0.1xx branch for .NET9 servicing. We'll take this for 10 so as to avoid any late template changes (noting that this one is probably lower risk that source changes that have typically been the focus).

I think this has happened now, right? The build failure looks unrelated.

@marcpopMSFT
Copy link
Member

@MiYanni yes, they should be free to merge this into main and the VMR failures are from the window when the ubuntu vmr leg had issues. Since there's no rush, I'd just rerun.

Directory.Build.rsp is a [documented file][1] that allows setting default arguments to command line builds. However, our .gitignore template ignores _all_ `*.rsp` files. which causes confusion:

1. Devs write an .rsp file, and if they aren't being attentive forget to commit it
2. Adding it to git requires passing a `--force` flag to `git add`, which some devs mistake for a destructive or not-recommended action.

Thus, explicitly allow the `Directory.Build.rsp` file.

[1]: https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-response-files?view=vs-2022#directorybuildrsp
@baronfel
Copy link
Member

Thanks @MattKotsenas!

@baronfel baronfel merged commit b58586e into dotnet:main Aug 22, 2024
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure needs team triage Requires a full team discussion untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants