-
Notifications
You must be signed in to change notification settings - Fork 32
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
Haddock "source" links are broken since the OsPath
transition
#198
Comments
There's supposedly a way without CPP: https://discourse.haskell.org/t/pre-ghc-proposal-instantiate-backpack-signatures-when-importing/6739/17 |
Backpack would not be my first choice. |
This does not require backpack. See the comment:
|
That sounds much better. But at least for |
If you want to do the same for filepath, feel free... but it's going to be a little more work. |
Before: https://hackage.haskell.org/package/filepath-1.4.2.2/docs/src/System.FilePath.Posix.html#takeFileName
After: https://hackage.haskell.org/package/filepath-1.4.100.3/docs/src/System.FilePath.Posix.html#takeFileName
@hasufell I think I understand the underlying motivation, and I am generally sympathetic to DRYing things up, this is why so far I refrained from reporting this as a bug.
However, I think that this puts us into an unfortunate situation:
Most importantly, code is far more often read than written. This is why personally I would not be willing to sacrifice readability for DRYness.
But also, CPP is a real challenge for tooling. This does not only affect you if you work on a package that makes heavy use of CPP. It also makes it difficult to e.g. implement a "go to definition" that works across packages. By my book less CPP is always a good thing.
For me, the final straw is that it's not only
filepath
, but that I see a spreading pattern here (directory
,file-io
, ...).The text was updated successfully, but these errors were encountered: