Tweak Control.Lens import for better compatibility
authorGreg Hurrell <greg@hurrell.net>
Thu, 16 Jun 2016 14:32:31 +0000 (07:32 -0700)
committerGreg Hurrell <greg@hurrell.net>
Thu, 16 Jun 2016 14:32:31 +0000 (07:32 -0700)
One of the CI runs blowing up:

    /home/travis/build/wincent/docvim/lib/Text/Docvim/Optimize.hs:3:1: Warning:
        Module ‘Control.Lens’ does not export ‘Empty’
    <no location info>:
    Failing due to -Werror.

Source: https://travis-ci.org/wincent/docvim/jobs/138094062

I'm going to turn off `-Werror` in CI and keep it locally, but in the meantime I
think I can get compatibility with a wider range of Lens versions if I switch
the import to not mention `Empty`.

See also: https://github.com/wincent/docvim/issues/24

lib/Text/Docvim/Optimize.hs

index 68057f4938905aaabf47925ab97ef4e2fcf00925..45d6337a67d671972f4d92dceb9caa3b18a58bda 100644 (file)
@@ -1,6 +1,6 @@
 module Text.Docvim.Optimize (optimize) where
 
-import Control.Lens hiding (Empty)
+import Control.Lens (children, transform)
 import Text.Docvim.AST
 
 -- | "Optimize" a Project's AST by eliminating empty paths.