search

agocorona / MFlow

(Haskell) Web application server with stateful, type safe user interactions and widget combinators
http://mflowdemo.herokuapp.com
Other
100 stars 12 forks source link

7.8 Fixes and Warp fixes #26

Closed sheganinans closed 10 years ago

sheganinans commented 10 years ago

It seems that the new Typeable class (new in ghc 7.7) does not like having custom instances. The docs say to just derive it.

[ 9 of 13] Compiling MFlow.Forms.Widgets ( src/MFlow/Forms/Widgets.hs, dist/build/MFlow/Forms/Widgets.o )

src/MFlow/Forms/Widgets.hs:111:3: ‘typeOf’ is not a (visible) method of class ‘Typeable’

I went ahead and commented lines 110-121 and just derived it and peppered all the neccessary Typeable constraints, and everything seems to compile well, now test are needed. Also fixed some random stuff to do with Network.Wai.Handler.Warp.Settings.

Although this still breaks:

Demos/Menu.hs:28:8: Could not find module ‘Text.Hamlet’ It is a member of the hidden package ‘shakespeare-2.0.0.1’. Perhaps you need to add ‘shakespeare’ to the build-depends in your .cabal file. Use -v to see a list of the files searched for.

sheganinans commented 10 years ago

I may want to break this out into two pull reqs, since the Warp part is really different from the Typeable constraints. What do you think?

agocorona commented 10 years ago

Try to add shakespeare tot he dependency list of the demos-blaze executable . That must fix the problem.

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file. Use -v to see a list of the files searched for.

2014-04-02 21:53 GMT+02:00, Aistis Raulinaitis notifications@github.com:

It seems that the new Typeable class (new in ghc 7.7) does not like having custom instances. The docs say to just derive it.

[ 9 of 13] Compiling MFlow.Forms.Widgets ( src/MFlow/Forms/Widgets.hs, dist/build/MFlow/Forms/Widgets.o )

src/MFlow/Forms/Widgets.hs:111:3: 'typeOf' is not a (visible) method of class 'Typeable'

I went ahead and commented lines 110-121 and just derived it and peppered all the neccessary Typeable constraints, and everything seems to compile well, now test are needed. Also fixed some random stuff to do with Network.Wai.Handler.Warp.Settings.

Although this still breaks:

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file. Use -v to see a list of the files searched for.

You can merge this Pull Request by running:

git pull https://github.com/sheganinans/MFlow master

Or you can view, comment on it, or merge it online at:

https://github.com/agocorona/MFlow/pull/26

-- Commit Summary --

  • 7.8 Fixes and Warp fixes

-- File Changes --

M src/MFlow/Forms/Widgets.hs (24)
M src/MFlow/Wai/Blaze/Html/All.hs (9)

-- Patch Links --

https://github.com/agocorona/MFlow/pull/26.patch https://github.com/agocorona/MFlow/pull/26.diff

Reply to this email directly or view it on GitHub: https://github.com/agocorona/MFlow/pull/26

Alberto.

sheganinans commented 10 years ago

Oh boy, seems to cascade to a whole bunch more stuff. I'll see what I can do.

On Wed, Apr 2, 2014 at 3:46 PM, agocorona notifications@github.com wrote:

Try to add shakespeare tot he dependency list of the demos-blaze executable . That must fix the problem.

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file. Use -v to see a list of the files searched for.

2014-04-02 21:53 GMT+02:00, Aistis Raulinaitis notifications@github.com:

It seems that the new Typeable class (new in ghc 7.7) does not like having custom instances. The docs say to just derive it.

[ 9 of 13] Compiling MFlow.Forms.Widgets ( src/MFlow/Forms/Widgets.hs, dist/build/MFlow/Forms/Widgets.o )

src/MFlow/Forms/Widgets.hs:111:3: 'typeOf' is not a (visible) method of class 'Typeable'

I went ahead and commented lines 110-121 and just derived it and peppered all the neccessary Typeable constraints, and everything seems to compile well, now test are needed. Also fixed some random stuff to do with Network.Wai.Handler.Warp.Settings.

Although this still breaks:

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file. Use -v to see a list of the files searched for.

You can merge this Pull Request by running:

git pull https://github.com/sheganinans/MFlow master

Or you can view, comment on it, or merge it online at:

https://github.com/agocorona/MFlow/pull/26

-- Commit Summary --

  • 7.8 Fixes and Warp fixes

-- File Changes --

M src/MFlow/Forms/Widgets.hs (24) M src/MFlow/Wai/Blaze/Html/All.hs (9)

-- Patch Links --

https://github.com/agocorona/MFlow/pull/26.patch https://github.com/agocorona/MFlow/pull/26.diff

Reply to this email directly or view it on GitHub: https://github.com/agocorona/MFlow/pull/26

Alberto.

Reply to this email directly or view it on GitHubhttps://github.com/agocorona/MFlow/pull/26#issuecomment-39393676 .

agocorona commented 10 years ago

I had to revert the typeable change in WIdgets.hs to compile in my environment. I indicated the change in the code.

Do you know how we can detect the ghc version?.

2014-04-02 21:53 GMT+02:00 Aistis Raulinaitis notifications@github.com:

It seems that the new Typeable class (new in ghc 7.7) does not like having custom instances. The docs say to just derive it.

[ 9 of 13] Compiling MFlow.Forms.Widgets ( src/MFlow/Forms/Widgets.hs, dist/build/MFlow/Forms/Widgets.o )

src/MFlow/Forms/Widgets.hs:111:3: 'typeOf' is not a (visible) method of class 'Typeable'

I went ahead and commented lines 110-121 and just derived it and peppered all the neccessary Typeable constraints, and everything seems to compile well, now test are needed. Also fixed some random stuff to do with Network.Wai.Handler.Warp.Settings.

Although this still breaks:

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file.

Use -v to see a list of the files searched for.

You can merge this Pull Request by running

git pull https://github.com/sheganinans/MFlow master

Or view, comment on, or merge it at:

https://github.com/agocorona/MFlow/pull/26 Commit Summary

  • 7.8 Fixes and Warp fixes

File Changes

  • M src/MFlow/Forms/Widgets.hshttps://github.com/agocorona/MFlow/pull/26/files#diff-0(24)
  • M src/MFlow/Wai/Blaze/Html/All.hshttps://github.com/agocorona/MFlow/pull/26/files#diff-1(9)

Patch Links:

Reply to this email directly or view it on GitHubhttps://github.com/agocorona/MFlow/pull/26 .

Alberto.

sheganinans commented 10 years ago

I'll take a look at it, it should be possible to fix this with some CPP preprocessing.

On Wed, Apr 9, 2014 at 11:31 PM, agocorona notifications@github.com wrote:

I had to revert the typeable change in WIdgets.hs to compile in my environment. I indicated the change in the code.

Do you know how we can detect the ghc version?.

2014-04-02 21:53 GMT+02:00 Aistis Raulinaitis notifications@github.com:

It seems that the new Typeable class (new in ghc 7.7) does not like having custom instances. The docs say to just derive it.

[ 9 of 13] Compiling MFlow.Forms.Widgets ( src/MFlow/Forms/Widgets.hs, dist/build/MFlow/Forms/Widgets.o )

src/MFlow/Forms/Widgets.hs:111:3: 'typeOf' is not a (visible) method of class 'Typeable'

I went ahead and commented lines 110-121 and just derived it and peppered all the neccessary Typeable constraints, and everything seems to compile well, now test are needed. Also fixed some random stuff to do with Network.Wai.Handler.Warp.Settings.

Although this still breaks:

Demos/Menu.hs:28:8: Could not find module 'Text.Hamlet' It is a member of the hidden package 'shakespeare-2.0.0.1'. Perhaps you need to add 'shakespeare' to the build-depends in your .cabal file.

Use -v to see a list of the files searched for.

You can merge this Pull Request by running

git pull https://github.com/sheganinans/MFlow master

Or view, comment on, or merge it at:

https://github.com/agocorona/MFlow/pull/26 Commit Summary

  • 7.8 Fixes and Warp fixes

File Changes

Patch Links:

Reply to this email directly or view it on GitHub< https://github.com/agocorona/MFlow/pull/26> .

Alberto.

Reply to this email directly or view it on GitHubhttps://github.com/agocorona/MFlow/pull/26#issuecomment-40047078 .