Jdesk / memcached

Automatically exported from code.google.com/p/memcached
0 stars 0 forks source link

update doc/threads.txt #116

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.read doc/threads.txt to enable thread support
2../configure --enable-threads

What is the expected output? What do you see instead?
configure: WARNING: unrecognized options: --enable-threads

What version of the product are you using? On what operating system?
memcached-1.4.4 and git repo

Please provide any additional information below.

I looked at configure.ac, it seems multi-thread support is compiled in by
default, no need to run ./configure --enable-threads, so doc/threads.txt
should be updated. 

Original issue reported on code.google.com by mchun...@gmail.com on 21 Dec 2009 at 11:27

GoogleCodeExporter commented 9 years ago
we should just get rid of that doc file, it's very outdated, since threads are 
now required as of 1.4.

Original comment by lind...@inuus.com on 29 Dec 2009 at 6:03

GoogleCodeExporter commented 9 years ago
fix here..

http://github.com/lindner/memcached/commit/54dfc54f990b43b840c53077525e93921de4a
3f4

Original comment by lind...@inuus.com on 29 Dec 2009 at 6:07

GoogleCodeExporter commented 9 years ago
I'm not sure if we should nuke the entire file.. It does contain a good 
description on how it works, but we should drop the obsolete information 
instead.. ???

Original comment by trond.no...@gmail.com on 14 Sep 2010 at 6:56

GoogleCodeExporter commented 9 years ago
I think Paul's fix was pretty good.  If we want a writeup of how threading 
works in memcached, I think we should do that outside of the source tree.

Original comment by dsalli...@gmail.com on 13 Jul 2011 at 2:05