nikita-volkov / stm-containers

Containers for STM
http://hackage.haskell.org/package/stm-containers
MIT License
67 stars 13 forks source link

Test failure from Stackage #2

Closed snoyberg closed 9 years ago

snoyberg commented 10 years ago

I'm guessing this is just a case of missing modules:

Resolving dependencies...
Configuring stm-containers-0.2.0...
Running Haddock for stm-containers-0.2.0...
Running hscolour for stm-containers-0.2.0...
Preprocessing library stm-containers-0.2.0...
Preprocessing test suite 'word-array-tests' for stm-containers-0.2.0...
Preprocessing test suite 'api-tests' for stm-containers-0.2.0...
Preprocessing library stm-containers-0.2.0...
Warning: The documentation for the following packages are not installed. No
links will be generated to these packages: base-prelude-0.1.6, focus-0.1.3,
hashable-1.2.2.0, list-t-0.2.6, loch-th-0.2.1, mmorph-1.0.4,
monad-control-0.3.3.0, placeholders-0.1, primitive-0.5.4.0, stm-2.4.3,
text-1.1.1.3, transformers-base-0.4.3

dist/build/tmp-16188/library/STMContainers/WordArray/Indices.hs:47:11: Warning:
    In the use of ‘bitSize’
    (imported from STMContainers.Prelude, but defined in base:Data.Bits):
    Deprecated: "Use 'bitSizeMaybe' or 'finiteBitSize' instead"

dist/build/tmp-16188/library/STMContainers/HAMT/Level.hs:25:9: Warning:
    In the use of ‘bitSize’
    (imported from STMContainers.Prelude, but defined in base:Data.Bits):
    Deprecated: "Use 'bitSizeMaybe' or 'finiteBitSize' instead"
Haddock coverage:
Warning: stm-containers-0.2.0:STMContainers.Prelude: Could not find documentation for exported module: Exports
  25% (  1 /  4) in 'STMContainers.Prelude'
  12% (  2 / 17) in 'STMContainers.WordArray.Indices'
  55% ( 12 / 22) in 'STMContainers.WordArray'
  42% (  5 / 12) in 'STMContainers.SizedArray'
  14% (  1 /  7) in 'STMContainers.HAMT.Level'
   0% (  0 / 13) in 'STMContainers.HAMT.Nodes'
   0% (  0 / 10) in 'STMContainers.HAMT'
  91% ( 10 / 11) in 'STMContainers.Map'
  91% ( 10 / 11) in 'STMContainers.Set'
  92% ( 11 / 12) in 'STMContainers.Multimap'
  93% ( 14 / 15) in 'STMContainers.Bimap'
Documentation created: dist/doc/html/stm-containers/index.html,
dist/doc/html/stm-containers/stm-containers.txt
Preprocessing test suite 'word-array-tests' for stm-containers-0.2.0...
Preprocessing test suite 'api-tests' for stm-containers-0.2.0...
Building stm-containers-0.2.0...
Preprocessing library stm-containers-0.2.0...
[ 1 of 11] Compiling STMContainers.Prelude ( library/STMContainers/Prelude.hs, dist/build/STMContainers/Prelude.o )
[ 2 of 11] Compiling STMContainers.WordArray.Indices ( library/STMContainers/WordArray/Indices.hs, dist/build/STMContainers/WordArray/Indices.o )

library/STMContainers/WordArray/Indices.hs:47:11: Warning:
    In the use of ‘bitSize’
    (imported from STMContainers.Prelude, but defined in base:Data.Bits):
    Deprecated: "Use 'bitSizeMaybe' or 'finiteBitSize' instead"
[ 3 of 11] Compiling STMContainers.WordArray ( library/STMContainers/WordArray.hs, dist/build/STMContainers/WordArray.o )
[ 4 of 11] Compiling STMContainers.SizedArray ( library/STMContainers/SizedArray.hs, dist/build/STMContainers/SizedArray.o )
[ 5 of 11] Compiling STMContainers.HAMT.Level ( library/STMContainers/HAMT/Level.hs, dist/build/STMContainers/HAMT/Level.o )

library/STMContainers/HAMT/Level.hs:25:9: Warning:
    In the use of ‘bitSize’
    (imported from STMContainers.Prelude, but defined in base:Data.Bits):
    Deprecated: "Use 'bitSizeMaybe' or 'finiteBitSize' instead"
[ 6 of 11] Compiling STMContainers.HAMT.Nodes ( library/STMContainers/HAMT/Nodes.hs, dist/build/STMContainers/HAMT/Nodes.o )
[ 7 of 11] Compiling STMContainers.HAMT ( library/STMContainers/HAMT.hs, dist/build/STMContainers/HAMT.o )
[ 8 of 11] Compiling STMContainers.Map ( library/STMContainers/Map.hs, dist/build/STMContainers/Map.o )
[ 9 of 11] Compiling STMContainers.Set ( library/STMContainers/Set.hs, dist/build/STMContainers/Set.o )
[10 of 11] Compiling STMContainers.Multimap ( library/STMContainers/Multimap.hs, dist/build/STMContainers/Multimap.o )
[11 of 11] Compiling STMContainers.Bimap ( library/STMContainers/Bimap.hs, dist/build/STMContainers/Bimap.o )
In-place registering stm-containers-0.2.0...
Preprocessing test suite 'word-array-tests' for stm-containers-0.2.0...

executables/WordArrayTests.hs:5:8:
    Could not find module ‘STMContainers.Transformers’
    Use -v to see a list of the files searched for.

executables/WordArrayTests.hs:8:18:
    Could not find module ‘WordArrayTests.Update’
    Use -v to see a list of the files searched for.

library/STMContainers/Prelude.hs:24:8:
    Could not find module ‘Control.Monad.Trans.Class’
    It is a member of the hidden package ‘transformers-0.3.0.0’.
    Perhaps you need to add ‘transformers’ to the build-depends in your .cabal file.
    Use -v to see a list of the files searched for.
nikita-volkov commented 10 years ago

Must be something on your end, because it passes a Travis build fine.

Note that the failing build target aggregates modules from two source directories: https://github.com/nikita-volkov/stm-containers/blob/master/stm-containers.cabal#L85-L86

snoyberg commented 10 years ago

I don't think so. Run cabal unpack stm-containers and try to find the referenced modules, they don't appear to be there. Running cabal install --run-tests stm-containers likely reproduces this also, but it hasn't completed on my system yet.

nikita-volkov commented 9 years ago

Turns out I accidentally uploaded a corrupt distribution to Hackage, which was missing several modules. The Git repository didn't miss them, so this explains why the Travis build passed.

I've just uploaded a correct distro to Hackage, so things should be fine now.

Again, sorry for all the fuss that accompanied my packages.

snoyberg commented 9 years ago

I still don't see those files in version 0.2.1.

nikita-volkov commented 9 years ago

The reason is clear now. The Cabal file was lacking the other-modules fields in testing executables. I actually never realised they were required prior to this. I've uploaded the version 0.2.2 and double checked it to contain all the files required.

snoyberg commented 9 years ago

Yup, confirmed fixed. Thanks!