From reviews-return-86662-apmail-mesos-reviews-archive=mesos.apache.org@mesos.apache.org Wed Mar 27 18:25:55 2019 Return-Path: X-Original-To: apmail-mesos-reviews-archive@minotaur.apache.org Delivered-To: apmail-mesos-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id E400919AC0 for ; Wed, 27 Mar 2019 18:25:55 +0000 (UTC) Received: (qmail 59391 invoked by uid 500); 27 Mar 2019 18:25:55 -0000 Delivered-To: apmail-mesos-reviews-archive@mesos.apache.org Received: (qmail 59358 invoked by uid 500); 27 Mar 2019 18:25:55 -0000 Mailing-List: contact reviews-help@mesos.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@mesos.apache.org Delivered-To: mailing list reviews@mesos.apache.org Received: (qmail 59343 invoked by uid 99); 27 Mar 2019 18:25:55 -0000 Received: from mailrelay1-us-west.apache.org (HELO mailrelay1-us-west.apache.org) (209.188.14.139) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 27 Mar 2019 18:25:55 +0000 Received: from reviews.apache.org (unknown [10.41.0.12]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id C0950E12D4; Wed, 27 Mar 2019 18:25:54 +0000 (UTC) Received: from reviews-vm2.apache.org (localhost [IPv6:::1]) by reviews.apache.org (ASF Mail Server at reviews-vm2.apache.org) with ESMTP id AD848C4027D; Wed, 27 Mar 2019 18:25:54 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============2360707270698721026==" MIME-Version: 1.0 Subject: Re: Review Request 70295: Enabled launcher sealing for RPM packages. From: Gilbert Song To: Joseph Wu , Till Toenshoff Cc: Benjamin Bannier , Mesos Reviewbot Windows , mesos , Mesos Reviewbot , James DeFelice Date: Wed, 27 Mar 2019 18:25:54 -0000 Message-ID: <20190327182554.24116.41817@reviews-vm2.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Gilbert Song X-ReviewGroup: mesos X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/70295/ X-Sender: Gilbert Song References: <20190326012118.42553.22076@reviews-vm2.apache.org> In-Reply-To: <20190326012118.42553.22076@reviews-vm2.apache.org> Reply-To: Gilbert Song X-ReviewRequest-Repository: mesos --===============2360707270698721026== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit > On March 25, 2019, 6:21 p.m., Joseph Wu wrote: > > support/packaging/centos/mesos.spec > > Lines 94 (patched) > > > > > > I'd expect the `--disable-libtool-wrappers` flag too, but do rpm builds omit libtool wrappers anyway? > > > > It's not clear to me when exactly the wrappers are generated and when they aren't > > Benjamin Bannier wrote: > The libtool wrappers (which set up the env to consume dynamic libraries built as part of the build) are generated so executables can be run from the build tree, without installing. They are never installed. OTOH, executables built in autotools projects without libtool wrappers cannot be installed as they are not correctly linked (they'd depend on files from the build tree, not from the install prefix). > > Dropping. @Joseph, in this case, we don't need to disable libtool wrappers since --enable-optimize would make sure there is no .sh init helper generated. - Gilbert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70295/#review214019 ----------------------------------------------------------- On March 25, 2019, 8:09 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70295/ > ----------------------------------------------------------- > > (Updated March 25, 2019, 8:09 a.m.) > > > Review request for mesos, Gilbert Song, Joseph Wu, and Till Toenshoff. > > > Bugs: MESOS-9677 > https://issues.apache.org/jira/browse/MESOS-9677 > > > Repository: mesos > > > Description > ------- > > We enable this flag since with it disabled certain public functions > are not available making it hard to e.g., write modules against this > version of Mesos. > > While launcher sealing depends on a recent kernel, the platform we > build RPMs for already satisfies the requirements. > > > Diffs > ----- > > support/packaging/centos/mesos.spec de905fda3366be5904a0aec8249abcc53b0af9a0 > > > Diff: https://reviews.apache.org/r/70295/diff/1/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > > --===============2360707270698721026==--