Skip to content

Commit

Permalink
Include.hs needs conduit 1.1+ to build
Browse files Browse the repository at this point in the history
Data/Yaml/Include.hs:35:21:
    Couldn't match type ‘()’ with ‘Event’
    Expected type: ConduitM Event Event m1 ()
      Actual type: Source m1 Event
    In a stmt of a 'do' block:
      go (cfp : seen) includeFile
      $= CL.filter (`notElem` irrelevantEvents)
    In the expression:
      do { let includeFile = takeDirectory cfp </> unpack (decodeUtf8 f);
           go (cfp : seen) includeFile
           $= CL.filter (`notElem` irrelevantEvents) }
    In a case alternative:
        EventScalar f (UriTag "!include") _ _
          -> do { let includeFile = ...;
                  go (cfp : seen) includeFile
                  $= CL.filter (`notElem` irrelevantEvents) }
  • Loading branch information
juhp committed Dec 3, 2014
1 parent 8274360 commit fe4ef27
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion yaml.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ library
build-depends: base >= 4 && < 5
, transformers >= 0.1
, bytestring >= 0.9.1.4
, conduit >= 1.0.11 && < 1.3
, conduit >= 1.1.0 && < 1.3
, resourcet >= 0.3 && < 1.2
, aeson >= 0.5
, containers
Expand Down

3 comments on commit fe4ef27

@hsenag
Copy link

@hsenag hsenag commented on fe4ef27 Mar 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible to edit the bounds for 0.8.10 on hackage to incorporate this fix?

@snoyberg
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@hsenag
Copy link

@hsenag hsenag commented on fe4ef27 Mar 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brilliant, thanks.

Please sign in to comment.