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

7.8 Fixes and Warp fixes #26

Merged
merged 1 commit into from
Apr 2, 2014
Merged

Conversation

sheganinans
Copy link
Contributor

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
Copy link
Contributor Author

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 added a commit that referenced this pull request Apr 2, 2014
@agocorona agocorona merged commit 4616639 into agocorona:master Apr 2, 2014
@agocorona
Copy link
Owner

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:

#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:
#26

Alberto.

@sheganinans
Copy link
Contributor Author

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:

#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:
#26

Alberto.

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

@agocorona
Copy link
Owner

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:

#26
Commit Summary

  • 7.8 Fixes and Warp fixes

File Changes

Patch Links:

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

Alberto.

@sheganinans
Copy link
Contributor Author

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:

#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//pull/26#issuecomment-40047078
.

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

Successfully merging this pull request may close these issues.

2 participants