diff options
| author | John Ericson <John.Ericson@Obsidian.Systems> | 2022-06-02 01:48:04 +0000 |
|---|---|---|
| committer | John Ericson <John.Ericson@Obsidian.Systems> | 2022-06-07 05:04:22 +0000 |
| commit | a7fece19cb56b6aa3ae7f81d1a34276ad55fdf2a (patch) | |
| tree | 1b750e50b3a511c55a034d4914356ffbf92c4363 | |
| parent | 04209f2a6a49f6cdc116b5cb73ccd1749c90f88b (diff) | |
| download | haskell-wip/enumerate-deps-only-no-count.tar.gz | |
Don't print the number of deps in count-deps testswip/enumerate-deps-only-no-count
It is redundant information and a source of needless version control
conflicts when multiple MRs are changing the deps list.
Just printing the list and not also its length is fine.
| -rw-r--r-- | testsuite/tests/count-deps/CountDepsAst.stdout | 2 | ||||
| -rw-r--r-- | testsuite/tests/count-deps/CountDepsParser.stdout | 2 | ||||
| -rw-r--r-- | utils/count-deps/Main.hs | 3 |
3 files changed, 3 insertions, 4 deletions
diff --git a/testsuite/tests/count-deps/CountDepsAst.stdout b/testsuite/tests/count-deps/CountDepsAst.stdout index 491fccd11b..562b0f2e9d 100644 --- a/testsuite/tests/count-deps/CountDepsAst.stdout +++ b/testsuite/tests/count-deps/CountDepsAst.stdout @@ -1,4 +1,4 @@ -Found 287 Language.Haskell.Syntax module dependencies +Found Language.Haskell.Syntax module dependencies GHC.Builtin.Names GHC.Builtin.PrimOps GHC.Builtin.PrimOps.Ids diff --git a/testsuite/tests/count-deps/CountDepsParser.stdout b/testsuite/tests/count-deps/CountDepsParser.stdout index e338d85507..e448875858 100644 --- a/testsuite/tests/count-deps/CountDepsParser.stdout +++ b/testsuite/tests/count-deps/CountDepsParser.stdout @@ -1,4 +1,4 @@ -Found 294 GHC.Parser module dependencies +Found GHC.Parser module dependencies GHC.Builtin.Names GHC.Builtin.PrimOps GHC.Builtin.PrimOps.Ids diff --git a/utils/count-deps/Main.hs b/utils/count-deps/Main.hs index 9dcc75619d..d431f00dda 100644 --- a/utils/count-deps/Main.hs +++ b/utils/count-deps/Main.hs @@ -40,8 +40,7 @@ printDeps libdir modName dot = do if not dot then do let modules = Map.keys modGraph - num = length modules - putStrLn $ "Found " ++ show num ++ " " ++ modName ++ " module dependencies" + putStrLn $ "Found " ++ modName ++ " module dependencies" forM_ modules putStrLn else -- * Copy the digraph output to a file ('deps.dot' say) |