Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-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 73C65105B7 for ; Tue, 30 Apr 2013 02:53:54 +0000 (UTC) Received: (qmail 85051 invoked by uid 500); 30 Apr 2013 02:53:54 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 85025 invoked by uid 500); 30 Apr 2013 02:53:54 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 85013 invoked by uid 99); 30 Apr 2013 02:53:54 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 30 Apr 2013 02:53:54 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of iclelland@google.com designates 209.85.219.54 as permitted sender) Received: from [209.85.219.54] (HELO mail-oa0-f54.google.com) (209.85.219.54) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 30 Apr 2013 02:53:49 +0000 Received: by mail-oa0-f54.google.com with SMTP id l20so50353oag.41 for ; Mon, 29 Apr 2013 19:53:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:content-type; bh=WyrhfAAHrBMUW12fGgiSrVIVb+nxbIV2FZ+5Fd8DXG0=; b=nRueNOnM0fod7yGmqVtXHRdn5vTFk2OgA8IZ25lkWDUL3k0U0GhoZxdKRelFGY9s2c xIzLnjRpsZsssCSuQHnGE1EIKek/rH1rFPn7Sm/2pNCyS72TTBL+Zlb5jevVygLeSTR8 QQ3yEBvyxUduY52isIM1VJKyHI4QGK0krzHyoXGnOwcl0nn8gPdOov6A3wO+09P62ZMi qN1fXgtQqpBMrGkO3SUz8D2dA9js44BU6ax9qjc/ASKrfRdu+QFGdEjM75HflhB7m2Hn ZJv72CBdp5GMNI8pXU+i4IRD+v8p7M71f6KYGLYkIYlnLuxmS9Ab5KZLqhUpgXn4OHin C5Ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:content-type:x-gm-message-state; bh=WyrhfAAHrBMUW12fGgiSrVIVb+nxbIV2FZ+5Fd8DXG0=; b=AIfu7AsVvpTVcbug8j8udreclMQ7AvCZeGT+GwKEGeWSf+t8knT24harfZV7ldPYxA 8ZkclkrfiGTKrXDocSXb+J2enL6nGA5tl21sRkxfgtYBYw+VIqDcp+ElDaZHis3M5miG UZ/ez7Zlx1CxWcJMQwl253pjBfTM6IdP9bfe6y6wGLNvko1laLff80Fu9u1ie0U68pRn U83OOqNGY0g9SmWxvjN/Q6ZD5uXFPIeULbKsNKRbVJJ5DiGBvd+R6hcTdgNHdY03gzTf 2892o4v7GosuKe1Q3SdjIA2EQ6B9oh8NfNf+0kSCqC03vhzIXaF6Ip61fKBytDvUNiTS TOtw== MIME-Version: 1.0 X-Received: by 10.182.22.133 with SMTP id d5mr15593318obf.43.1367290408345; Mon, 29 Apr 2013 19:53:28 -0700 (PDT) Received: by 10.182.156.19 with HTTP; Mon, 29 Apr 2013 19:53:28 -0700 (PDT) In-Reply-To: References: Date: Mon, 29 Apr 2013 22:53:28 -0400 Message-ID: Subject: Re: [Android] Including okHttp as source might not be such a good idea From: Ian Clelland To: dev@cordova.apache.org Content-Type: multipart/alternative; boundary=001a11c2ef86183d2304db8b1bdb X-Gm-Message-State: ALoCoQlLGaDuhZ4H3UpsanensaFoSJwguxELHYcWWcf9g9XQhfqO5RkOd5CI5/YIYAGjNwzqJZ7iOcxMhup1beG+Z7xwvhSupTW5UolZIHJfAoo4oLXawWm7+LAc5Ad0AxAMLAnFQN8iGB6miAqkj7cTHcoQLtpUnyqxWSC4GMwiL1fHf/dm9SvQk/i/sbzdwC7BmJhe4DV2 X-Virus-Checked: Checked by ClamAV on apache.org --001a11c2ef86183d2304db8b1bdb Content-Type: text/plain; charset=UTF-8 Lets not try to force all developers to change their settings; that sounds like a world of pain. It should work as a jar -- I can't see any compelling reason to have the source available in eclipse, except for debugging. If there are bugs in that code, I would definitely prefer to be fixing them upstream, though, rather than having our own fork. Can we distribute the jar with cordova-android, rather than fetching it at install time like with commons-codec? I'm fairly certain that the Apache license allows that. Also, I don't think that OkHttp has tagged releases like codec, and I think I would rather ship with tested versions rather than pointing new users at whatever HEAD happens to be at at any given day. If that's okay, I'll package it up, and replace the source with the jar on master. On Mon, Apr 29, 2013 at 6:17 PM, Joe Bowser wrote: > Hey > > I'm currently working with Master and I'm noticing that Eclipse really > hates all these @Override Annotations put in the code. Since > everyone's Eclipse will probably work slightly different in this > regard, and I have no idea how fix this other than by deleting all the > annotations, I propose that we use a jar for okHttp instead of the > source, similar to how we use commons-codec. > > Alternatively, if we can figure out how to all agree on the same > Eclipse settings for Java dev, perhaps that may help as well. As of > right now, I can't build master due to these stupid annotation errors. > > Joe > --001a11c2ef86183d2304db8b1bdb--