[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-06-03 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/55#issuecomment-108581857 Should be fine - `git diff HEAD~1 --check` now outputs nothing --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-06-01 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/55#issuecomment-107667388 // rebased --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-28 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/55#issuecomment-106576604 Changed what @kxepal pointed out. Sorry for the second update to the pull request today, my editor does tabs instead of spaces by default and I missed

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-27 Thread arturmazurek
Github user arturmazurek commented on a diff in the pull request: https://github.com/apache/couchdb-couch/pull/55#discussion_r31165173 --- Diff: src/couch_httpd.erl --- @@ -30,6 +30,7 @@ -export([send_json/2,send_json/3,send_json/4,last_chunk/1,parse_multipart_request/3

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-26 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/55#issuecomment-105557076 This is a follow-up to https://github.com/apache/couchdb-couch/pull/53 Added tests for the new functions. Also regarding a comment in that

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-26 Thread arturmazurek
GitHub user arturmazurek opened a pull request: https://github.com/apache/couchdb-couch/pull/55 Fix duplicated Content-Type for show/update functions When a show/update function returned data it added a "Content-Type: application/json" header by itself. Then c

[GitHub] couchdb-couch pull request: Fixing a build for Mac OS 10.10

2015-05-26 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/52#issuecomment-105499041 Oh, that's great, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pr

[GitHub] couchdb-couch pull request: Fixing a build for Mac OS 10.10

2015-05-26 Thread arturmazurek
Github user arturmazurek closed the pull request at: https://github.com/apache/couchdb-couch/pull/52 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] couchdb-couch pull request: Fixing a build for Mac OS 10.10

2015-05-26 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/52#issuecomment-105424822 I've had brew for some time already and it works just fine. I didn't source my bash_profile, however have rebooted multiple times since instal

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-25 Thread arturmazurek
Github user arturmazurek closed the pull request at: https://github.com/apache/couchdb-couch/pull/53 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-25 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/53#issuecomment-105274678 Ok, I'll add the tests and also check the cluster interface. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] couchdb-couch pull request: Fixing a build for Mac OS 10.10

2015-05-25 Thread arturmazurek
Github user arturmazurek commented on the pull request: https://github.com/apache/couchdb-couch/pull/52#issuecomment-105274193 Odd, definitely happens for me. Yosemite 10.10.3 and CC "Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)" Calling `cc -

[GitHub] couchdb-couch pull request: Fix duplicated Content-Type for show/u...

2015-05-25 Thread arturmazurek
GitHub user arturmazurek opened a pull request: https://github.com/apache/couchdb-couch/pull/53 Fix duplicated Content-Type for show/update functions When a show/update function returned data it added a "Content-Type: application/json" header by itself. Then c

[GitHub] couchdb-couch pull request: Fixing a build for Mac OS 10.10

2015-05-23 Thread arturmazurek
GitHub user arturmazurek opened a pull request: https://github.com/apache/couchdb-couch/pull/52 Fixing a build for Mac OS 10.10 Missing libmozjs185. Added /usr/local/lib to linker search path because couchdb suggests using homebrew to install dependancies and that&#