Skip to content

Commit

Permalink
delete System.PosixCompat.User module
Browse files Browse the repository at this point in the history
  • Loading branch information
mitchellwrosen committed Mar 15, 2023
1 parent d4efce8 commit 87ffe05
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 136 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## Unreleased

- Remote `System.PosixCompat.User` module

## Version 0.6 (2022-05-22)

- Better support for symbolic links
2 changes: 0 additions & 2 deletions src/System/PosixCompat.hs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ module System.PosixCompat (
, module System.PosixCompat.Time
, module System.PosixCompat.Types
, module System.PosixCompat.Unistd
, module System.PosixCompat.User
, usingPortableImpl
) where

Expand All @@ -20,7 +19,6 @@ import System.PosixCompat.Temp
import System.PosixCompat.Time
import System.PosixCompat.Types
import System.PosixCompat.Unistd
import System.PosixCompat.User

-- | 'True' if unix-compat is using its portable implementation,
-- or 'False' if the unix package is simply being re-exported.
Expand Down
133 changes: 0 additions & 133 deletions src/System/PosixCompat/User.hsc

This file was deleted.

1 change: 0 additions & 1 deletion unix-compat.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ Library
System.PosixCompat.Time
System.PosixCompat.Types
System.PosixCompat.Unistd
System.PosixCompat.User

if os(windows)
c-sources:
Expand Down

2 comments on commit 87ffe05

@ocharles
Copy link

@ocharles ocharles commented on 87ffe05 Aug 19, 2023

Choose a reason for hiding this comment

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

Out of interest, why was this removed, and what should users do instead? Dhall used this module, and it's unclear what the migration path is.

Edit: nvm, I see #2.

@mitchellwrosen
Copy link
Member Author

Choose a reason for hiding this comment

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

@ocharles Yeah, and see #3 for another long discussion. I've moved on from maintaining this package, but (if you care to read all that's in that thread) you'll see that adding the module back is an option on the table for whoever wants to do it. So far, no one's made the patch.

Please sign in to comment.