Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/ndmitchell/shake
Browse files Browse the repository at this point in the history
  • Loading branch information
ndmitchell committed Nov 15, 2020
2 parents 64d46f1 + b212e25 commit 1fe1439
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGES.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
Changelog for Shake (* = breaking change)

#780, Autodeps should consider a rename as a write to the destination
#778, AutoDeps shouldn't trigger for files read and written
#779, merge dependencies in O(n and a bit)
#779, merge local traces in O(n)
Expand Down
4 changes: 2 additions & 2 deletions src/Development/Shake/Command.hs
Original file line number Diff line number Diff line change
Expand Up @@ -312,8 +312,8 @@ commandExplicitAction oparams = do
| otherwise = act params

autodeps act = do
ResultFSATrace pxs : res <- act params{opts = addFSAOptions "rw" opts, results = ResultFSATrace [] : results}
let written = Set.fromList [x | FSAWrite x <- pxs]
ResultFSATrace pxs : res <- act params{opts = addFSAOptions "rwm" opts, results = ResultFSATrace [] : results}
let written = Set.fromList $ [x | FSAMove x _ <- pxs] ++ [x | FSAWrite x <- pxs]
-- If something both reads and writes to a file, it isn't eligible to be an autodeps
xs <- liftIO $ filterM doesFileExist [x | FSARead x <- pxs, not $ x `Set.member` written]
cwd <- liftIO getCurrentDirectory
Expand Down

0 comments on commit 1fe1439

Please sign in to comment.