Return-Path: X-Original-To: apmail-couchdb-dev-archive@www.apache.org Delivered-To: apmail-couchdb-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 307737203 for ; Thu, 6 Oct 2011 10:59:34 +0000 (UTC) Received: (qmail 75490 invoked by uid 500); 6 Oct 2011 10:59:33 -0000 Delivered-To: apmail-couchdb-dev-archive@couchdb.apache.org Received: (qmail 75458 invoked by uid 500); 6 Oct 2011 10:59:33 -0000 Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@couchdb.apache.org Delivered-To: mailing list dev@couchdb.apache.org Received: (qmail 75450 invoked by uid 99); 6 Oct 2011 10:59:33 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 06 Oct 2011 10:59:33 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS,T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of bchesneau@gmail.com designates 209.85.215.180 as permitted sender) Received: from [209.85.215.180] (HELO mail-ey0-f180.google.com) (209.85.215.180) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 06 Oct 2011 10:59:27 +0000 Received: by eyb6 with SMTP id 6so3489669eyb.11 for ; Thu, 06 Oct 2011 03:59:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; bh=abTCsWz3lenOoAZg/XCwNwPksMY7uBmX2mlsepIn7XM=; b=qKOAVj/lQyFjvxOlG1g0fyu8p7LkWyU6W/6vNSLU83/Q80nYnawNGkLwl89sK+/3/3 WcrubMmnqGEWmbaTbG7s+CeNvzaLrMth/3VcO2k/cl7tmlsuqGRdrae8JX5iK5iUI3Qy AP6igGrFa0/P2cHeWZgHoZByf7TkKz+bhRtM4= MIME-Version: 1.0 Received: by 10.213.17.17 with SMTP id q17mr2227256eba.115.1317898746834; Thu, 06 Oct 2011 03:59:06 -0700 (PDT) Received: by 10.213.4.141 with HTTP; Thu, 6 Oct 2011 03:59:06 -0700 (PDT) In-Reply-To: References: Date: Thu, 6 Oct 2011 12:59:06 +0200 Message-ID: Subject: Re: CouchDB 1.1.1 From: Benoit Chesneau To: dev@couchdb.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org On Thu, Oct 6, 2011 at 11:06 AM, Paul Davis w= rote: > On Thu, Oct 6, 2011 at 3:51 AM, Benoit Chesneau wro= te: >> On Thu, Oct 6, 2011 at 10:25 AM, Paul Davis wrote: >>> On Thu, Oct 6, 2011 at 3:23 AM, Robert Newson wrot= e: >>>> All, >>>> >>>> Paul Davis has researched the issue and it seems intractable. >>>> >>>> I would like to remove 1.8.5 support from 1.1.1. It was not present in >>>> 1.1.0 so will not be (officially) missed. >>>> >>>> The place for a breaking change of this magnitude is 1.2, not a minor >>>> bug fix release. >>>> >>>> Thoughts? >>>> B. >>>> >>> >>> +1 on removing the paren hack for sure. >>> >>> Not sure about removing 1.8.5 support completely. On the one hand, it >>> would prevent breakage because people couldn't link against the >>> breaking SM. On the other hand, it prevents people from linking >>> against 1.8.5 which means it won't build on Ubuntu 11.x. >> >> >> Is this really a problem with 1.8.5 ? How do you explain then that I >> have to remove this changes to have refuge working with it then? >> >> https://github.com/refuge/refuge/commit/481bd6623ccfc8895eacf8a1528bffa5= efa4ad47#apps/couch/share/server/util.js >> >> Refuge is only working with 1.8.5. >> >> >> - beno=EEt >> > > Things I know for certain: > > This fails in the js shell from the 1.8.5 tarball: > > =A0 =A0eval("function(){}") > > It also fails on the package in Debian that is supposed to be > SpiderMonkey 1.8.5. It also fails on a checkout of the mercurial > repository with hash 959c1e6bdb11. > > Fact of the matter, SpiderMonkey distribution is notorious in its > avoidance of anything resembling a versioning scheme that we can use > to refer to API/ABI compatibility across platforms. Not that it > matters as it's apparently a fix, so at the end of the day "new > SpiderMonkey breaks code that has worked for years." > > By enabling the linkage (for which, if code were proper, things would > work) we introduce a non-obvious new behavior error. > > If we *intentionally disable* 1.8.5 linking, then we avoid introducing > errors subtly into code that has always worked if someone tries to > upgrade SpiderMonkey to a version that breaks old code. That said, we > also limit the ability of CouchDB to run easily on OS's that include > newer SpiderMonkeys. > The odd thing is that refuge is using the archive downloadable on mozilla website. Anyway that doesn't change anything to the fact that we should correct our js I agree. I would be for a 2.0 then, like noah. - benoit