Skip to content

Commit

Permalink
Fix some warnings and formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
jgotoh committed Apr 2, 2024
1 parent 409b6ac commit f449135
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 13 deletions.
2 changes: 1 addition & 1 deletion cabal-install/src/Distribution/Client/ProjectConfig.hs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ import Distribution.Client.VCS
, validateSourceRepos
)

import GHC.Stack (HasCallStack, callStack)
import Distribution.Client.BuildReports.Types
( ReportLevel (..)
)
Expand All @@ -104,6 +103,7 @@ import Distribution.Client.HttpUtils
)
import Distribution.Client.Types
import Distribution.Client.Utils.Parsec (renderParseError)
import GHC.Stack (HasCallStack, callStack)

import Distribution.Simple.Errors
import Distribution.Simple.PackageDescription (flattenDups)
Expand Down
7 changes: 4 additions & 3 deletions cabal-install/src/Distribution/Client/ProjectConfig/Parsec.hs
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,12 @@ import Distribution.Fields.ConfVar (parseConditionConfVar)
import Distribution.Fields.ParseResult

-- AST type
import Distribution.Fields (Field (..), FieldLine (..), FieldName, Name (..), PWarning, SectionArg (..), readFields', showPWarning)
import Distribution.Fields (Field (..), FieldLine (..), FieldName, Name (..), SectionArg (..), readFields', showPWarning)
import Distribution.Fields.LexerMonad (toPWarnings)
import Distribution.Parsec (CabalParsing, PError (..), ParsecParser, parsec, parsecFilePath, parsecToken, runParsecParser)
import Distribution.Parsec.Position (Position (..), zeroPos)
import Distribution.Parsec.Warning (PWarnType (..))
import Distribution.Simple.Program.Db (ProgramDb, defaultProgramDb, lookupKnownProgram, knownPrograms)
import Distribution.Simple.Program.Db (ProgramDb, defaultProgramDb, knownPrograms, lookupKnownProgram)
import Distribution.Simple.Program.Types (programName)
import Distribution.Simple.Setup (Flag (..))
import Distribution.Types.CondTree (CondBranch (..), CondTree (..))
Expand Down Expand Up @@ -95,7 +95,7 @@ parseProjectSkeleton cacheDir httpTransport verbosity seenImports source bs = (s
let res' = case result of
Right cfg -> pure cfg
Left (_, errors) -> do
traverse_ (\(PError pos str) -> parseFailure pos str) errors
traverse_ (\(PError errPos str) -> parseFailure errPos str) errors
parseFatalFailure pos $ "Failed to parse import " ++ importLoc

rest <- go [] xs
Expand Down Expand Up @@ -170,6 +170,7 @@ parseProjectSkeleton cacheDir httpTransport verbosity seenImports source bs = (s

programDb = defaultProgramDb

knownProgramNames :: ProgramDb -> [String]
knownProgramNames programDb = (programName . fst) <$> knownPrograms programDb

-- | Monad in which sections are parsed
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ instance Parsec WriteGhcEnvironmentFilesPolicy where
"never" -> return NeverWriteGhcEnvironmentFiles
"ghc8.4.4+" -> return WriteGhcEnvironmentFilesOnlyForGhc844AndNewer
policy ->
P.unexpected
$ "Cannot parse the GHC environment file write policy '"
<> policy
<> "'"
P.unexpected $
"Cannot parse the GHC environment file write policy '"
<> policy
<> "'"
10 changes: 5 additions & 5 deletions cabal-install/src/Distribution/Client/Utils/Parsec.hs
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,11 @@ renderParseError
-> [PWarning]
-> String
renderParseError filepath contents errors warnings =
unlines
$ [ "Errors encountered when parsing cabal file " <> filepath <> ":"
]
++ renderedErrors
++ renderedWarnings
unlines $
[ "Errors encountered when parsing cabal file " <> filepath <> ":"
]
++ renderedErrors
++ renderedWarnings
where
filepath' = normalise filepath

Expand Down

0 comments on commit f449135

Please sign in to comment.