Skip to content

Commit

Permalink
Merge pull request #3974 from niteria/dedupe-recursive-include-dirs
Browse files Browse the repository at this point in the history
Dedupe include dirs inherited from dependencies
  • Loading branch information
23Skidoo authored Oct 12, 2016
2 parents 366f5a8 + 6e31b30 commit 5345fcf
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions Cabal/Distribution/Simple/Configure.hs
Original file line number Diff line number Diff line change
Expand Up @@ -1603,8 +1603,12 @@ checkForeignDeps pkg lbi verbosity = do
++ collectField PD.cppOptions
++ collectField PD.ccOptions
++ [ "-I" ++ dir
| dep <- deps
, dir <- Installed.includeDirs dep ]
| dir <- ordNub [ dir
| dep <- deps
, dir <- Installed.includeDirs dep ]
-- dedupe include dirs of dependencies
-- to prevent quadratic blow-up
]
++ [ opt
| dep <- deps
, opt <- Installed.ccOptions dep ]
Expand Down

0 comments on commit 5345fcf

Please sign in to comment.