Return-Path: X-Original-To: apmail-celix-dev-archive@www.apache.org Delivered-To: apmail-celix-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 511A017E73 for ; Thu, 2 Oct 2014 19:50:01 +0000 (UTC) Received: (qmail 41700 invoked by uid 500); 2 Oct 2014 19:50:01 -0000 Delivered-To: apmail-celix-dev-archive@celix.apache.org Received: (qmail 41669 invoked by uid 500); 2 Oct 2014 19:50:01 -0000 Mailing-List: contact dev-help@celix.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@celix.apache.org Delivered-To: mailing list dev@celix.apache.org Received: (qmail 41652 invoked by uid 99); 2 Oct 2014 19:50:00 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Oct 2014 19:50:00 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=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.44 as permitted sender) Received: from [74.125.82.44] (HELO mail-wg0-f44.google.com) (74.125.82.44) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Oct 2014 19:49:29 +0000 Received: by mail-wg0-f44.google.com with SMTP id y10so4046581wgg.15 for ; Thu, 02 Oct 2014 12:49:28 -0700 (PDT) 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=OWVK7FMlu8aYx4XPA1c7npFSsgiT4guSTMthiAdtiAc=; b=n6yDYhS0LKRuKv2AkfjLkhFbvzsdJnKh5ZzXngG5iOYK3CwY9aYcGHdpiAL0wdKw1S /XuP7MYS2yEMy7jqhirZhxEAAiZDoLremS8N2wPH1WHD+gll2Pb4KMD7SRykjnjSWCnE sevluBl8SP7vBMCms+qHGb3OnFSsZ5jj05XldluR008MjEuH8RK1aF+hlPk7RO0M8K00 YwfxWsdkcXeyjIkmTC8sY0tjJu6cq78DyBqgXcutvvBVx1M/1by/M1SykhgLNFXdUNX+ i6t9JmGPWbf/rtzIuUp3zjfODOLfJllhjK9Myi0K1l6wuxXPqMuX1poYjJYhGQo4KXo6 0aBg== MIME-Version: 1.0 X-Received: by 10.180.72.239 with SMTP id g15mr6785601wiv.47.1412279368664; Thu, 02 Oct 2014 12:49:28 -0700 (PDT) Received: by 10.194.77.203 with HTTP; Thu, 2 Oct 2014 12:49:28 -0700 (PDT) In-Reply-To: References: Date: Thu, 2 Oct 2014 21:49:28 +0200 Message-ID: Subject: Re: Symbol visibility From: Pepijn Noltes To: dev@celix.apache.org Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org Hi, On Thu, Oct 2, 2014 at 9:25 AM, Alexander Broekhuis wrote: > Hi all, > > In the past I have done some work for symbol visibility, this has never > been enabled properly (or has been disabled for some reason). I'd like to > enable this now in the build. > > I did some local testing, and I don't see any problems with it. Besides > setting visibility in the framework and the utils, something has to be > setup for use in private/exported/imported libraries as well. I can add > something for this. > > What is a problem however, is the fw_log function which is now used in some > bundles as well. The fw_log function is specifically for the framework > itself, and not for others, so it should not be exported. > In places where logging is needed, the log_service should be used. Bjoern > already proposed to write some helper code to easily get a logger and use > it. I'd like to see this as well. > > Besides the fw_log I assume several other (framework private) functions are > now exported as well, this will need to be addressed later on as well. > > What do you all think? +1 Sounds like a good idea, I assume that if a compiler/linker does not support this you can easily turn if off ? Greetings, Pepijn