Incompatibility with fsnotify >= 0.3

sjakobi commented 6 years ago

    /tmp/stack19783/steeloverseer-2.0.2.0/app/Main.hs:258:5: error:
        • The constructor ‘FSNotify.Added’ should have 3 arguments, but has been given 2
        • In the pattern: FSNotify.Added path _
          In a case alternative:
              FSNotify.Added path _ -> S.yield (FileAdded (go cwd path))
          In the second argument of ‘S.for’, namely
            ‘(\case
                FSNotify.Added path _ -> S.yield (FileAdded (go cwd path))
                FSNotify.Modified path _ -> S.yield (FileModified (go cwd path))
                FSNotify.Removed _ _ -> pure ())’
        |
    258 |     FSNotify.Added    path _ -> S.yield (FileAdded    (go cwd path))
        |     ^^^^^^^^^^^^^^^^^^^^^^^^

dlidstrom commented 5 years ago

Is there a workaround?

mitchellwrosen commented 5 years ago

@sjakobi @dlidstrom Are you able to build with stack? The stack.yaml should resolve to a compatible version of fsnotify (however I'm sure 0.3 support would not be hard to add).

mitchellwrosen commented 5 years ago

Ah! I understand what's going on now. The version on Hackage is out of date. HEAD is compatible with fsnotify 0.3.

sjakobi commented 5 years ago

The version on Hackage is out of date. HEAD is compatible with fsnotify 0.3.

Oh, I hadn't noticed!

@schell Would you mind making a release for this fix?

schell commented 5 years ago

2.1 is now live :)

schell / steeloverseer

Incompatibility with fsnotify >= 0.3 #37