Remove camlp4 and update

[mor1]

Finally updating mirage-decks, including moving away from camlp4.
Not ready for merge yet.

[yomimono]

I see you have "not done" in the commit messages still -- what's left to do here? I haven't looked closely at the changes but I see that your tests are passing, which seems like a good sign. Thanks for your work on this!

[mor1]

When I last tested it, I think I'd botched something in the conversion so pages weren't rendering quite right. Should finish it off in the next week.

Cheers,

R.

On 16 Oct 2016, at 18:04, Mindy Preston notifications@github.com wrote:

I see you have "not done" in the commit messages still -- what's left to do here? I haven't looked closely at the changes but I see that your tests are passing, which seems like a good sign. Thanks for your work on this!

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[mor1]

Ok, I think that works now. Only light testing but the two topmost decks appear to be ok and that tests the index and both Reveal 2.4.0 and Reveal 2.6.2 which is basically everything.

If Travis passes I think this is ready for merge -- look ok to you @yomimono @avsm?

[yomimono]

This is great! Thanks very much, @mor1 . :)

[yomimono]

What do we need to do to rebuild http://decks.openmirage.org/ ?

[mor1]

The site ought to rebuild as part of Travis completing testing the merge result successfully.
It should then reboot due to the post-merge hook in the deployment repo.
I'll check shortly...

[mor1]

Hm, Travis has failed the DEPLOY=1 build -- looks like the name of the output .xen file changed. Will look into it.