On Thu, Oct 02, 2014 at 11:54:33AM +0400, Kirill Bychkov wrote:
> We're living in an age of clouds, so I want to introduce port of a client for
> "Next-generation Open Source Cloud Storage".
> http://seafile.com/en/home/
> 
> Currently only client is ported because server side is... well... not easy in
> configuration and it's port is incomplete.
> 
> Tested against Linux server and library with very big and tons of small files.
> 
> Comments? OK?
> 
> P.S. You can find all the stuff in openbsd-wip.

That mostly looks good to me, minor nits:

- can you add comments to 'no-so-obvious-patches' ? like the removal of
  evutil.h inclusion..
- add a comment explaining why a test is disabled in libzdb..
- seafile-admin references/hardcodes /var/seafile, while i dont see that
  dir registered anywhere in a PLIST, nor with a corresponding user - is
that stuff supposed to be running as root ?
- i see a desktop client and a daemon - do i understand that 'both' are
  needed for client functionality ? How does the daemon start ?

Landry

Reply via email to