summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Gamari <ben@smart-cactus.org>2020-01-23 14:46:21 -0500
committerBen Gamari <ben@smart-cactus.org>2020-01-23 20:47:48 -0500
commitde1de32539859acdd7cf5ba3d60d42af6c1631fd (patch)
tree3af7d6c3a215c09691c8408525049fe9f6b5b235
parent3dae006fc424e768bb43fc73851a08fefcb732a5 (diff)
downloadhaskell-wip/T17156.tar.gz
hadrian: Throw error on duplicate-named flavourswip/T17156
Throw an error if the user requests a flavour for which there is more than one match. Fixes #17156.
-rwxr-xr-xhadrian/src/Settings.hs10
1 files changed, 6 insertions, 4 deletions
diff --git a/hadrian/src/Settings.hs b/hadrian/src/Settings.hs
index 2263f68107..068e7ba914 100755
--- a/hadrian/src/Settings.hs
+++ b/hadrian/src/Settings.hs
@@ -58,12 +58,14 @@ flavour :: Action Flavour
flavour = do
flavourName <- fromMaybe userDefaultFlavour <$> cmdFlavour
kvs <- userSetting ([] :: [KeyVal])
- let unknownFlavour = error $ "Unknown build flavour: " ++ flavourName
- flavours = hadrianFlavours ++ userFlavours
+ let flavours = hadrianFlavours ++ userFlavours
(_settingErrs, tweak) = applySettings kvs
- return $ maybe unknownFlavour tweak $
- find ((== flavourName) . name) flavours
+ return $
+ case filter (\fl -> name fl == flavourName) flavours of
+ [] -> error $ "Unknown build flavour: " ++ flavourName
+ [f] -> tweak f
+ _ -> error $ "Multiple build flavours named " ++ flavourName
getIntegerPackage :: Expr Package
getIntegerPackage = expr (integerLibrary =<< flavour)