This repository has been archived by the owner. It is now read-only.

CouchDB: update to 1.2.0 #11440

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
10 participants
Contributor

dch commented Apr 4, 2012

  • bump to latest release
  • add --devel option for next anticipated release branch
  • document how to run CouchDB as a system or user daemon
  • document how to resolve bus errors & segfaults in Erlang/OTP
  • add missing dependencies
Contributor

dch commented Apr 4, 2012

Replaces the pull req for 1.1.1 & incorporates all the useful feedback provided - thanks!

1.2.0 source is heading out to the mirrors at the moment, official announcement will follow when that is completed.

Contributor

janl commented Apr 6, 2012

The release is a go. +1

@ghost ghost assigned adamv Apr 6, 2012

Contributor

adamv commented Apr 6, 2012

Rebase on master (though the merge is simple)

Contributor

adamv commented Apr 7, 2012

@jacknagel @Sharpie @mistydemeo @MikeMcQuaid - opinion on the OpenSSL dep?

Owner

MikeMcQuaid commented Apr 7, 2012

Not acceptable. Why can you not use the system OpenSSL? What specifically are the problems with doing so and is there no way to workaround that?

Contributor

dch commented Apr 7, 2012

On 7 April 2012 05:41, Mike McQuaid
reply@reply.github.com
wrote:

Not acceptable. Why can you not use the system OpenSSL? What specifically are the problems with doing so and is there no way to workaround that?


Reply to this email directly or view it on GitHub:
mxcl#11440 (comment)

Hi Mike,

ref https://gist.github.com/1199903

Using the system openssl generates so many segfaults that it is
impossible even to complete the CouchDB dev test suite, let alone be
stable in production usage. Using 1.0.1 addresses this.

Thanks
Dave

Contributor

wonk commented Apr 8, 2012

Please add --with-geocouch to the Formula. Now would be a great time... Thx.
#10777

Contributor

dch commented Apr 8, 2012

@wonk perhaps geocouch should be a separate recipe, dependent on couchdb?

Contributor

wonk commented Apr 9, 2012

@dch Hmm, yes... separates could be better. I was getting my feet wet with brew, so I just merged the geocouch into your couchdb 1.2.0 recipe. It did not occur to me to build a new recipe (because I was thinking "small steps"). However, now that you mention it, I "had to" add a few (ugly) conditionals for warnings, so perhaps the separate geocouch recipe would clean those up.

Then (ultimately) the geocouch recipe could be dependent on either couchdb or couchbase (by command line flag/installer choice), since they're now separate projects but geocouch will (for the foreseeable future) support both? For now, we could just support couchdb, but perhaps someone will add couchbase down the road.

So a brand new recipe... that defaults to couchdb (but could select couchone down the road)... and builds on that? Is that the general consensus/recommendation?

Contributor

dch commented Apr 9, 2012

On 8 April 2012 21:00, wonk
reply@reply.github.com
wrote:

@dch  Hmm, yes... separates could be better.  I was getting my feet wet with brew, so I just merged the geocouch into your couchdb 1.2.0 recipe.  It did not occur to me to build a new recipe (because I was thinking "small steps").  However, now that you mention it, I "had to" add a few (ugly) conditionals for warnings, so perhaps the separate geocouch recipe would clean those up.

Then (ultimately) the geocouch recipe could be dependent on either couchdb or couchone (by command line flag/installer choice), since they're now separate projects but geocouch will (for the foreseeable future) support both?  For now, we could just support couchdb, but someone could add couchone down the road.

So a brand new recipe... that selects either couchdb or couchone... and builds on that?  Is that the general consensus?

I'd suggest new recipe using only couchdb. If couchbase provides a dev
version I am sure it will have geocouch built in directly.

Contributor

imrefazekas commented Apr 11, 2012

+1. Needed very much!
When will be this formula available?

Contributor

dch commented Apr 12, 2012

  • rebased on master
  • cleaned dependencies to avoid requirement to brew link icu/openssl

NB set of 3 related patches, requires committing:

  • #11247 for 64-bit openssl
  • #11609 to add openssl as erlang dependency,
  • #11440 for couchdb 1.2.0
Library/Formula/couchdb.rb
couchdb
+
+ If you are getting bus errors or segmentation faults, ensure
+ that you have installed keg-only OpenSSL prior to brewing Erlang.
@mistydemeo

mistydemeo Apr 12, 2012

Contributor

This part can be removed - if Erlang depends on openssl then it will automatically be used, and if Erlang doesn't depend on openssl than the keg-only openssl won't be linked against even if it's manually installed.

Owner

MikeMcQuaid commented Apr 13, 2012

Erlang needs to depend on OpenSSL now too?

Contributor

jacknagel commented Apr 13, 2012

Note what @Sharpie asked here: mxcl#11247

There are questions that need to be answered more thoroughly before we can pull any of this.

Owner

MikeMcQuaid commented Apr 14, 2012

Ok, we can pull CouchDB as is I think. I can't as I don't have the bandwidth but the caveats seem fine.

Contributor

jacknagel commented Apr 14, 2012

I don't like the caveats; we shouldn't be telling users that they need to run otool and resolve these dependencies themselves.

CouchDB: update to 1.2.0
- bump to latest release
- add --devel option for next anticipated release branch
- document how to run CouchDB as a system or user daemon
Contributor

dch commented Apr 17, 2012

hopefully final update, with otool notes removed.

@Sharpie thanks for headsup on geocouch formula, I've commented there.

Contributor

dch commented Apr 19, 2012

Homebrewers can we get this committed please soon? I think there's a concensus that this is ready to go!

Contributor

Sharpie commented Apr 19, 2012

I think there's a concensus that this is ready to go!

Do we still need to resolve #11440 #11609 before committing this? There are still unanswered questions holding that pull request up.

Contributor

Sharpie commented Apr 19, 2012

Do we still need to resolve #11440 before committing this?

Erp, that would be #11609, not 11440.

Contributor

dch commented Apr 19, 2012

I'm suggesting to commit the couchdb 1.2.0 update as is; the
underlying erlang issue has been present in previous versions as well
(clearly, as its not a couchdb bug).

I've not had a response on erlang-bugs nor have I tracked down the
specific openssl issue yet, this is going to take a few weeks at this
point to get it nailed down.

Can we run with what we've got here?

On 19 April 2012 15:31, Charlie Sharpsteen
reply@reply.github.com
wrote:

Do we still need to resolve #11440 before committing this?

Erp, that would be #11609, not 11440.


Reply to this email directly or view it on GitHub:
mxcl#11440 (comment)

Contributor

wonk commented Apr 19, 2012

+1, & #11659 (geocouch 1.2.0) would be nice too.

Owner

MikeMcQuaid commented Apr 19, 2012

I think we can should just commit CouchDB 1.2.0 as-is and work on the other issues later.

Contributor

imrefazekas commented Apr 20, 2012

+1 Agree, commit as soon as possible please ...

@Sharpie Sharpie closed this in 3b39db9 Apr 20, 2012

Contributor

Sharpie commented Apr 20, 2012

Tested and pulled. Thanks to everyone for their patience.

Contributor

dch commented Apr 20, 2012

On 20 April 2012 19:33, Charlie Sharpsteen
reply@reply.github.com
wrote:

Tested and pulled. Thanks to everyone for their patience.


Reply to this email directly or view it on GitHub:
mxcl#11440 (comment)

@Sharpie & team thanks also for your comments & contributions. It's a
better set of recipes for your efforts. Much appreciated!

rohansingh added a commit to rohansingh/homebrew that referenced this pull request May 7, 2012

CouchDB: update to 1.2.0
- bump to latest release
- add --devel option for next anticipated release branch
- document how to run CouchDB as a system or user daemon

Closes #11440.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>

Sharpie added a commit to Sharpie/homebrew that referenced this pull request Sep 12, 2012

CouchDB: update to 1.2.0
- bump to latest release
- add --devel option for next anticipated release branch
- document how to run CouchDB as a system or user daemon

Closes #11440.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>

snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request Dec 17, 2012

CouchDB: update to 1.2.0
- bump to latest release
- add --devel option for next anticipated release branch
- document how to run CouchDB as a system or user daemon

Closes #11440.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>

@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.