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

Fix discovery of working directory for worktrees #734

Merged
merged 1 commit into from
Aug 12, 2021
Merged

Conversation

tmat
Copy link
Member

@tmat tmat commented Aug 12, 2021

SourceLink used gitdir to determine the directory (and incorrectly interpreting the content) but turns out git doesn't use it at all for this purpose.

Fixes dotnet/roslyn#55149
Fixes #726

SourceLink used gitdir to determine the directory (and incorrectly interpreting the content) but turns out git doesn't use it at all for this purpose.
@tmat tmat merged commit 48e002e into dotnet:main Aug 12, 2021
@tmat tmat deleted the WorkTree branch August 12, 2021 22:29
@tmat tmat added this to the 1.1.0 milestone Aug 12, 2021
dariuszkuc referenced this pull request in apollographql/federation-hotchocolate Oct 13, 2023
…11)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [Microsoft.SourceLink.GitHub](https://togithub.com/dotnet/sourcelink)
| `1.1.0` -> `1.1.1` |
[![age](https://developer.mend.io/api/mc/badges/age/nuget/Microsoft.SourceLink.GitHub/1.1.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/Microsoft.SourceLink.GitHub/1.1.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/Microsoft.SourceLink.GitHub/1.1.0/1.1.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Microsoft.SourceLink.GitHub/1.1.0/1.1.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>dotnet/sourcelink (Microsoft.SourceLink.GitHub)</summary>

###
[`v1.1.1`](https://togithub.com/dotnet/sourcelink/releases/tag/1.1.1)

[Compare
Source](https://togithub.com/dotnet/sourcelink/compare/1.1.0...1.1.1)

#### Notable Changes

- Feature: GitWeb Source Link provider by
[@&#8203;Glen-Nicol-Garmin](https://togithub.com/Glen-Nicol-Garmin) in
[https://github.com/dotnet/sourcelink/pull/505](https://togithub.com/dotnet/sourcelink/pull/505)
- Use submodule.name.url to determine the URL of a submodule by
[@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/587](https://togithub.com/dotnet/sourcelink/pull/587)
- Add Microsoft.SourceLink.Tools source package implementing
SourceLinkMap by [@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/663](https://togithub.com/dotnet/sourcelink/pull/663)
- Added support for gitea by
[@&#8203;Mik4sa](https://togithub.com/Mik4sa) in
[https://github.com/dotnet/sourcelink/pull/674](https://togithub.com/dotnet/sourcelink/pull/674)
- Enable source-build with Arcade SDK fixes by
[@&#8203;dagood](https://togithub.com/dagood) in
[https://github.com/dotnet/sourcelink/pull/692](https://togithub.com/dotnet/sourcelink/pull/692)
- Add netstandard2.0 target to tools package by
[@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/702](https://togithub.com/dotnet/sourcelink/pull/702)
- Update license to MIT by [@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/730](https://togithub.com/dotnet/sourcelink/pull/730)
- Allow using .git directory instead of gitdir redirect in submodules.
by [@&#8203;crummel](https://togithub.com/crummel) in
[https://github.com/dotnet/sourcelink/pull/653](https://togithub.com/dotnet/sourcelink/pull/653)
- Fix discovery of working directory for worktrees by
[@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/734](https://togithub.com/dotnet/sourcelink/pull/734)
- Add support for the new GitLab raw url by
[@&#8203;rgl](https://togithub.com/rgl) in
[https://github.com/dotnet/sourcelink/pull/713](https://togithub.com/dotnet/sourcelink/pull/713)
- Target netcoreapp3.1 by [@&#8203;tmat](https://togithub.com/tmat) in
[https://github.com/dotnet/sourcelink/pull/767](https://togithub.com/dotnet/sourcelink/pull/767)

#### New Contributors

- [@&#8203;Glen-Nicol-Garmin](https://togithub.com/Glen-Nicol-Garmin)
made their first contribution in
[https://github.com/dotnet/sourcelink/pull/505](https://togithub.com/dotnet/sourcelink/pull/505)
-
[@&#8203;IgorKustov-ChathamFinancial](https://togithub.com/IgorKustov-ChathamFinancial)
made their first contribution in
[https://github.com/dotnet/sourcelink/pull/592](https://togithub.com/dotnet/sourcelink/pull/592)
- [@&#8203;v-chmart](https://togithub.com/v-chmart) made their first
contribution in
[https://github.com/dotnet/sourcelink/pull/604](https://togithub.com/dotnet/sourcelink/pull/604)
- [@&#8203;0xced](https://togithub.com/0xced) made their first
contribution in
[https://github.com/dotnet/sourcelink/pull/668](https://togithub.com/dotnet/sourcelink/pull/668)
- [@&#8203;nycdotnet](https://togithub.com/nycdotnet) made their first
contribution in
[https://github.com/dotnet/sourcelink/pull/672](https://togithub.com/dotnet/sourcelink/pull/672)
- [@&#8203;Mik4sa](https://togithub.com/Mik4sa) made their first
contribution in
[https://github.com/dotnet/sourcelink/pull/674](https://togithub.com/dotnet/sourcelink/pull/674)
- [@&#8203;rgl](https://togithub.com/rgl) made their first contribution
in
[https://github.com/dotnet/sourcelink/pull/713](https://togithub.com/dotnet/sourcelink/pull/713)
- [@&#8203;adiaaida](https://togithub.com/adiaaida) made their first
contribution in
[https://github.com/dotnet/sourcelink/pull/737](https://togithub.com/dotnet/sourcelink/pull/737)

**Full Changelog**:
dotnet/sourcelink@1.0.0...1.1.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/apollographql/federation-hotchocolate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy44LjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

%(SourceRoot.Identity) ends with \.git\ in worktree #line directives fail to build in worktrees
1 participant