Return-Path: X-Original-To: apmail-qpid-proton-archive@minotaur.apache.org Delivered-To: apmail-qpid-proton-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 36D55105F9 for ; Wed, 4 Dec 2013 15:53:55 +0000 (UTC) Received: (qmail 62527 invoked by uid 500); 4 Dec 2013 15:53:47 -0000 Delivered-To: apmail-qpid-proton-archive@qpid.apache.org Received: (qmail 62315 invoked by uid 500); 4 Dec 2013 15:53:44 -0000 Mailing-List: contact proton-help@qpid.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: proton@qpid.apache.org Delivered-To: mailing list proton@qpid.apache.org Received: (qmail 62135 invoked by uid 99); 4 Dec 2013 15:53:40 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 04 Dec 2013 15:53:40 +0000 Date: Wed, 4 Dec 2013 15:53:40 +0000 (UTC) From: "ASF subversion and git services (JIRA)" To: proton@qpid.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (PROTON-445) Binding installation ignores prefix MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/PROTON-445?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13839015#comment-13839015 ] ASF subversion and git services commented on PROTON-445: -------------------------------------------------------- Commit 1547833 from [~mcpierce] in branch 'proton/trunk' [ https://svn.apache.org/r1547833 ] Revert "PROTON-445: Perl install now honors CMAKE_INSTALL_PREFIX" This reverts commit 17a0b3b5cb46301e40b2082f6c5316a0876422c1. > Binding installation ignores prefix > ----------------------------------- > > Key: PROTON-445 > URL: https://issues.apache.org/jira/browse/PROTON-445 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c > Affects Versions: 0.5 > Reporter: Justin Ross > Assignee: Darryl L. Pierce > Fix For: 0.6 > > Attachments: what-a-mess.txt > > > It allows you *prepend* to the install prefix, but it gives you no way afaict to actually change the prefix. > This is the opposite of nice. If you set a prefix for your build *and* you try to get your bindings slotted in with them, via DESTDIR, you get this: > # cmake -DCMAKE_INSTALL_PREFIX:PATH=/opt/myplace /var/tmp/jross/baker/proton/source > # make install DESTDIR=/opt/myplace > /opt/myplace/usr/lib/python/*python files* > /opt/myplace/opt/myplace/lib/*c files* > ^^ Note "/opt/myplace/opt/myplace", the first from DESTDIR, the second from CMAKE_INSTALL_PREFIX > What it is doing now is simply abuse of DESTDIR. DESTDIR is intended to be a mechanism for staged installs (packaging systems use this), and it cannot function correctly as an override for prefix. > http://www.gnu.org/prep/standards/html_node/DESTDIR.html > My proposed solution to this is to stop this madness: make the binding install honor CMAKE_INSTALL_PREFIX. Let the developer be responsible for choosing the right location for his or her distribution. -- This message was sent by Atlassian JIRA (v6.1#6144)