Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

thyme-0.3.5.5 can't compile its test suite #36

Closed
peti opened this issue Aug 7, 2015 · 1 comment
Closed

thyme-0.3.5.5 can't compile its test suite #36

peti opened this issue Aug 7, 2015 · 1 comment

Comments

@peti
Copy link

peti commented Aug 7, 2015

The build with GHC 7.10.2 fails as follows:

tests/Common.hs:5:1: Warning:
    The import of ‘Control.Applicative’ is redundant
      except perhaps to import instances from ‘Control.Applicative’
    To import instances alone, use: import Control.Applicative()
[3 of 3] Compiling Main             ( tests/sanity.hs, dist/build/sanity/sanity-tmp/Main.dyn_o )

tests/sanity.hs:59:23:
    Couldn't match expected type ‘T.TimeLocale’
                with actual type ‘TimeLocale’
    NB: ‘T.TimeLocale’
          is defined in ‘Data.Time.Format.Locale’ in package ‘time-1.5.0.1’
        ‘TimeLocale’
          is defined in ‘System.Locale’ in package ‘old-locale-1.0.0.7’
    In the first argument of ‘T.formatTime’, namely ‘defaultTimeLocale’
    In the expression: T.formatTime defaultTimeLocale spec t'

tests/sanity.hs:69:22:
    Couldn't match expected type ‘T.TimeLocale’
                with actual type ‘TimeLocale’
    NB: ‘T.TimeLocale’
          is defined in ‘Data.Time.Format.Locale’ in package ‘time-1.5.0.1’
        ‘TimeLocale’
          is defined in ‘System.Locale’ in package ‘old-locale-1.0.0.7’
    In the first argument of ‘T.formatTime’, namely ‘defaultTimeLocale’
    In the expression:
      T.formatTime defaultTimeLocale spec (thyme # orig)

tests/sanity.hs:71:22:
    Couldn't match expected type ‘T.TimeLocale’
                with actual type ‘TimeLocale’
    NB: ‘T.TimeLocale’
          is defined in ‘Data.Time.Format.Locale’ in package ‘time-1.5.0.1’
        ‘TimeLocale’
          is defined in ‘System.Locale’ in package ‘old-locale-1.0.0.7’
    In the first argument of ‘T.parseTime’, namely ‘defaultTimeLocale’
    In the expression: T.parseTime defaultTimeLocale spec s

tests/sanity.hs:74:27:
    No instance for (Show t0) arising from a use of ‘show’
    The type variable ‘t0’ is ambiguous
    Note: there are several potential instances:
      instance (MemoTrie-0.6.2:Data.MemoTrie.HasTrie a, Show a,
                Show b) =>
               Show (a MemoTrie-0.6.2:Data.MemoTrie.:->: b)
        -- Defined in ‘MemoTrie-0.6.2:Data.MemoTrie’
      instance Show Test.QuickCheck.Random.QCGen
        -- Defined in ‘Test.QuickCheck.Random’
      instance [safe] Show Args -- Defined in ‘Test.QuickCheck.Test’
      ...plus 153 others
    In the first argument of ‘(++)’, namely ‘show t'’
    In the second argument of ‘(++)’, namely
      ‘show t'
       ++
         "\n\
         \state: "
         ++ show (tp s)’
    In the second argument of ‘(++)’, namely
      ‘"\n\
       \time:  "
       ++
         show t'
         ++
           "\n\
           \state: "
           ++ show (tp s)’
@liyang
Copy link
Member

liyang commented Apr 1, 2016

This is due to time-1.5 abandoning System.Locale.defaultTimeLocale from old-locale and providing its own instead. It's fixed in master.

@liyang liyang closed this as completed Apr 1, 2016
sternenseemann added a commit to NixOS/nixpkgs that referenced this issue May 30, 2023
0z13 pushed a commit to 0z13/nixpkgs that referenced this issue Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants