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

Add Blog roll #156 #208

Merged
merged 5 commits into from
Aug 17, 2014
Merged

Add Blog roll #156 #208

merged 5 commits into from
Aug 17, 2014

Conversation

voila
Copy link
Contributor

@voila voila commented Aug 11, 2014

A new section on the Community page to list Mirage's related blogs

@amirmc
Copy link
Contributor

amirmc commented Aug 11, 2014

Ref #156

@avsm
Copy link
Member

avsm commented Aug 17, 2014

Thanks! I'd like to have the posts linked somewhere individually (e.g. like a Planet setup), but this requires Syndic support, so merging this until that's done.

avsm added a commit that referenced this pull request Aug 17, 2014
@avsm avsm merged commit 2bccf19 into mirage:master Aug 17, 2014
@voila
Copy link
Contributor Author

voila commented Aug 17, 2014

Has syndication support started ? I cannot find any related repository...

@avsm
Copy link
Member

avsm commented Aug 17, 2014

https://github.com/Cumulus/Syndic -- but needs to use cohttp

@amirmc
Copy link
Contributor

amirmc commented Aug 18, 2014

And a planet-like implementation nearly done too.

https://github.com/ocaml/ocaml.org/tree/syndic

sent via mobile

On 18 Aug 2014, at 00:14, Anil Madhavapeddy [email protected] wrote:

https://github.com/Cumulus/Syndic -- but needs to use cohttp


Reply to this email directly or view it on GitHub.

@voila
Copy link
Contributor Author

voila commented Sep 18, 2014

Syndic depends on Xmlm and Calendar. Are these dependencies OK for mirage-www ?

@samoht
Copy link
Member

samoht commented Sep 18, 2014

mirage-www already depends on xmlmx (via cow). Is calendar in pure ocaml?

@voila
Copy link
Contributor Author

voila commented Sep 19, 2014

After cursory look, I would say yes, pure OCaml.

@samoht
Copy link
Member

samoht commented Sep 19, 2014

That's a good start. Then, if it's pure ocaml need to check that:

  • it does not use out_channel and in_channel
  • it does not use the Unix module
  • it does not use Threads

After a quick look, it seems that calendar depends on unix but the dependency is easily removable (ie. just using Unix.gettimeofday and Unix.gmtime which are available in mirage (in the OS.Clock module). So yes, I think that syndic is the good way to go (module few patches to calendar).

@voila
Copy link
Contributor Author

voila commented Sep 19, 2014

Cool, thanks.

@voila
Copy link
Contributor Author

voila commented Sep 21, 2014

They also use Unix.localtime. Should this be added to mirage's Clock module ?

@avsm
Copy link
Member

avsm commented Sep 21, 2014

No, as it requires a timezone db. It's easier just to store the utc time and to render other timezones from within the application. The Clock module is very core to Mirage and needs to stay small.

On 21 Sep 2014, at 22:05, voila [email protected] wrote:

They also use Unix.localtime. Should this be added to mirage's Clock module ?


Reply to this email directly or view it on GitHub.

@voila
Copy link
Contributor Author

voila commented Sep 22, 2014

Ok, well, Syndic does not use that part of Calendar anyway.
By the way, is this the right place to ask questions about how to add syndication to mirage-www ?

@avsm
Copy link
Member

avsm commented Sep 22, 2014

Absolutely, or feel free to mail [email protected] if you want a wider audience (not everyone watches this repo).

On 22 Sep 2014, at 01:28, voila [email protected] wrote:

By the way, is this the right place to ask questions about how to add syndication to mirage-www ?

@voila
Copy link
Contributor Author

voila commented Sep 23, 2014

Well, I was wrong Syndic makes use of code which relies on Unix.localtime()... kinda stumped now.

@Chris00
Copy link
Contributor

Chris00 commented Jan 7, 2015

I have removed the use of Unix in Syndic.

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.

5 participants