Skip to content

Commit a6b99b8

Browse files
authored
Merge pull request haskell#10033 from bacchanalia/8963
Add comment clarifying disabling check for scripts
2 parents d1a6ced + d03137e commit a6b99b8

File tree

1 file changed

+3
-2
lines changed
  • Cabal/src/Distribution/PackageDescription/Check

1 file changed

+3
-2
lines changed

Cabal/src/Distribution/PackageDescription/Check/Target.hs

+3-2
Original file line numberDiff line numberDiff line change
@@ -825,9 +825,10 @@ checkGHCOptions title t opts = do
825825
checkFlags
826826
["-prof"]
827827
(PackageBuildWarning $ OptProf title)
828-
-- Does not apply to scripts.
829-
-- Why do we need this? See #8963.
830828
pid <- asksCM (pnPackageId . ccNames)
829+
-- Scripts add the -o flag in the fake-package.cabal in order to have the
830+
-- executable name match the script name even when there are characters
831+
-- in the script name which are illegal to have as a target name.
831832
unless (pid == fakePackageId) $
832833
checkFlags
833834
["-o"]

0 commit comments

Comments
 (0)