Skip to content

Fix #3047#3092

Merged
mergify[bot] merged 9 commits into
haskell:masterfrom
July541:fix#3047
Aug 14, 2022
Merged

Fix #3047#3092
mergify[bot] merged 9 commits into
haskell:masterfrom
July541:fix#3047

Conversation

@July541

@July541 July541 commented Aug 11, 2022

Copy link
Copy Markdown
Collaborator

It is because of the difference between hs-source-dirs: ./ and hs-source-dirs: ., take /user/project/Lib/A.hs as example,

hs-source-dirs: ./ hs-source-dirs: .
project path /user/project/./ /user/project/.
absolute project path /user/project/ /user/project
canonicalized project path /user/project /user/project

The canonicalized project path will be used as the prefix for /user/project/Lib/A.hs

Canonicalized it to prevent dropping one more character.

@July541
July541 requested a review from jneira as a code owner August 11, 2022 14:16
@July541

July541 commented Aug 11, 2022

Copy link
Copy Markdown
Collaborator Author

/cc @andrewufrank

Comment thread plugins/hls-module-name-plugin/src/Ide/Plugin/ModuleName.hs Outdated
Comment thread plugins/hls-module-name-plugin/src/Ide/Plugin/ModuleName.hs Outdated
@fendor

fendor commented Aug 11, 2022

Copy link
Copy Markdown
Collaborator

Excuse me for this drive-by review, I was just looking at it and found the usage of canonicalizePath might be tricky.

@July541

July541 commented Aug 11, 2022

Copy link
Copy Markdown
Collaborator Author

Excuse me for this drive-by review, I was just looking at it and found the usage of canonicalizePath might be tricky.

I'll try to evaluate the usage of canonicalizePath, at least find an alternative one.

@fendor fendor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, awesome!

Comment thread plugins/hls-module-name-plugin/src/Ide/Plugin/ModuleName.hs Outdated
@July541 July541 added the merge me Label to trigger pull request merge label Aug 14, 2022
@mergify
mergify Bot merged commit 49373fd into haskell:master Aug 14, 2022
sloorush pushed a commit to sloorush/haskell-language-server that referenced this pull request Sep 12, 2022
* Make path canonicalized

* Update extra-source-files

* Replace with normalise

* Change to a more detailed log

* Comment patch detail

* 2 spaces indent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge me Label to trigger pull request merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants