Return-Path: X-Original-To: apmail-incubator-celix-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-celix-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B8F60D384 for ; Tue, 11 Dec 2012 18:07:31 +0000 (UTC) Received: (qmail 38010 invoked by uid 500); 11 Dec 2012 18:07:31 -0000 Delivered-To: apmail-incubator-celix-dev-archive@incubator.apache.org Received: (qmail 37985 invoked by uid 500); 11 Dec 2012 18:07:31 -0000 Mailing-List: contact celix-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: celix-dev@incubator.apache.org Delivered-To: mailing list celix-dev@incubator.apache.org Received: (qmail 37976 invoked by uid 99); 11 Dec 2012 18:07:31 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Dec 2012 18:07:31 +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 (nike.apache.org: domain of pepijnnoltes@gmail.com designates 74.125.82.175 as permitted sender) Received: from [74.125.82.175] (HELO mail-we0-f175.google.com) (74.125.82.175) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Dec 2012 18:07:23 +0000 Received: by mail-we0-f175.google.com with SMTP id z53so1727488wey.6 for ; Tue, 11 Dec 2012 10:07:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=W4bP422wbdC+dHsFhLMvA6axrt0hufxYjVsXhOGs9Cs=; b=YDUdgU05itAAqtwzG7tto8OpMpKO5fiCXdPclfE4quWzobzjZ6s7kaDofNVP+fGf+l KWxqCrR/3oPBlp6CoPqS9zi07OmEfDxmWi3zJ4PY0fcYD9hptSYG5K4BnlmgL2N0zutf 4TubnZxtKmDIgZSj3OHm7QVRlqutGTb3rhkudhKuACTAmoSA4ZOset04FNgpsC+SfsP/ cDauR26jkm8Kqg0SLdOgEwKaIkC+mr1farvmPudp2JAMG1KVBvnA51xc4BIDUdxu+mFv VfRzwILo0boaVi6uwhq/KKQWTdqjjy2X0LuUZPQFZfCTqXccbHmk73vqzH+2YV7rb0KM JYyA== MIME-Version: 1.0 Received: by 10.194.93.40 with SMTP id cr8mr2015548wjb.16.1355249223043; Tue, 11 Dec 2012 10:07:03 -0800 (PST) Received: by 10.217.51.69 with HTTP; Tue, 11 Dec 2012 10:07:02 -0800 (PST) In-Reply-To: References: Date: Tue, 11 Dec 2012 19:07:02 +0100 Message-ID: Subject: Re: [jira] [Resolved] (CELIX-44) Remote Shell From: Pepijn Noltes To: celix-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=047d7bae49ec8588fb04d0978cad X-Virus-Checked: Checked by ClamAV on apache.org --047d7bae49ec8588fb04d0978cad Content-Type: text/plain; charset=ISO-8859-1 On Tue, Dec 11, 2012 at 6:04 PM, Alexander Broekhuis wrote: > > > > > > For OSX I am using is 10.8, which comes APR 1.4.5. And this seems to work > > with remote shell. > > When building against the APR from MacPorts (version 1.4.6) I get a > > segfault when connecting with telnet. > > I checked at work and there I'm using CentOS 5.8 with APR version 1.4.6 > and > > in this case I don't get segfaults when connecting. > > > > When i have some time, I will look try to fix this for OSX. > > > > I upgraded my installation and now it works. 10.6 has version > 1.3.something. > > When I was still on 10.6 I tried MacPorts version as well, but that one > seems to miss support for the wakeable pollset, even though the code > compiles fine with it. > > In the remote_shell.c the remoteShell_stopConnection doesn't check the > error code of the apr_pollset_wakeup, but this one returns an error with > the MacPorts version (APR_EINIT). > I corrected this and adjusted the creating of a parentless memory pool for newly created connections. Apparently this resulted - in my case - in a segfault. Could you verify if this works for you? > > > > > > > I think it would be wise to specify a minimum version of APR we support. > > The current building and running page ([1]) mentioned a dependency to > APR, > > but not which version. I also think that we can expect from users to > > compile/install a suitable APR version and not depend on the APR versions > > installed in for example OSX. So IMO specifying a minimal APR version of > > 1.4.x is no problem, or am I missing something ... > > > > Normally I would agree, but upgrading the installed version on MacOSX is a > problem (it probably breaks the httpd installation etc). So the only option > is to use the MacPorts version, but even though it compiles, at runtime > there are still problems.. So either the MacPorts version is missing > something, or we are missing something ;). > > The point is, I'd like to be able to support OSX 10.6, and trying to fix > this is needed for that one :). Understandable, but the APR website only has a link to the documentation for version 0.9 and 1.4 (see [1]). If you enter a url manual you can find the 1.3 documentation (see [2]), but still I prefer a versions which is supported from the website. [1] http://apr.apache.org/ [2] http://apr.apache.org/docs/apr/1.3/ Greetings, Pepijn --047d7bae49ec8588fb04d0978cad--