From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1E1C6C48BE6 for ; Wed, 16 Jun 2021 15:11:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id EDE70610A3 for ; Wed, 16 Jun 2021 15:11:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234488AbhFPPNu (ORCPT ); Wed, 16 Jun 2021 11:13:50 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:59128 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234390AbhFPPNt (ORCPT ); Wed, 16 Jun 2021 11:13:49 -0400 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 456C13E5; Wed, 16 Jun 2021 17:11:41 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1623856301; bh=sOiicZqrQxYBph3AEKu1KEJU/5pPmKS6fmccgMTllYo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=iRe3emVR+VTqNxcYMHzAAtVQXMmaNrDduFoz25WOa7DNaLI95WM2IIRK3CbYYzD4b NLSpZKyv/rppKv0lE6q8X0s+luSw0FoBctE7RMTTy395YEArvLjWhQNpFQ3XNzmwxD VKbvheV7vYydpvVcCmy27srPVZAq+ElPkfnq0T5c= Date: Wed, 16 Jun 2021 18:11:20 +0300 From: Laurent Pinchart To: Mauro Carvalho Chehab Cc: Ariel D'Alessandro , kieran.bingham@ideasonboard.com, linux-media@vger.kernel.org, hverkuil@xs4all.nl, sean@mess.org, p.zabel@pengutronix.de, ezequiel@collabora.com, nicolas@ndufresne.ca, gjasny@googlemail.com, xavier.claessens@collabora.com, nicolas.dufresne@collabora.com, user.vdr@gmail.com, sakari.ailus@iki.fi, rosenp@gmail.com Subject: Re: [v4l-utils v5 0/5] Add support for meson building Message-ID: References: <20210512184946.102863-1-ariel.dalessandro@collabora.com> <008aec9c-c1e6-7f8b-dde5-8fa53c1e9ee6@collabora.com> <92b5c1bf-f4cf-5d5e-7c16-57d4a3446f67@ideasonboard.com> <443286a1-b955-1ac1-742d-42b9182a435f@collabora.com> <20210616165947.70f73cec@coco.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210616165947.70f73cec@coco.lan> Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org On Wed, Jun 16, 2021 at 04:59:47PM +0200, Mauro Carvalho Chehab wrote: > Em Wed, 16 Jun 2021 11:26:10 -0300 Ariel D'Alessandro escreveu: > > > Kieran, > > > > > > - Meson decided to default libdir to > > > /usr/local/lib/x86_64-linux-gnu > > > while autoconf used > > > /usr/local/lib > > > (it's likely handled by the package managers anyway) > > > > $ meson configure ../build-meson/ | grep libdir > > libdir lib/x86_64-linux-gnu Library directory > > The default isn't nice, and will likely cause troubles for the users, as > /usr/local/lib/x86_64-linux-gnu would hardly be at the ld.so.conf list of > paths for most people. So, when one would try to run a program, it will > fail. This depends on the distribution. On Debian-like distributions, to enable multi-arch support, an arch-specific directory is used. Debian adds it to ld.so.conf (for both /usr/lib and /usr/local/lib). On other distributions, /usr/lib or /usr/lib64 will be used by meson. > > As mentioned in another response. Autotools is building with `-g -O2` by > > default, that'd the equivalent meson configuration option > > --buildtype=debugoptimized. > > I can't understand the Meson's default... no optimization and no debug! > Basically something that it is useless for both developers and for > production. I wonder why they chose a crap default like that... The default is debug, so it includes debug (and no optimization). This can be overridden at the project level with default_options : ['buildtype=debugoptimized'] in the project() function. > > With this configuration the installtion > > sizes are not that different: > > > > $ du -s installation_m* > > 37068 installation_make > > 37848 installation_meson > > It sounds a lot more coherent. > > > In this case the difference is related to libtool `.la` files not being > > generated by meson and gconv/ only installed by meson. The latest is > > probably a feature being only detected by meson in this case, will check > > that out. > > It was opted to not enable gconv by default, as similar patchset - needed > to support ARIB STD B-24 charset used by Japanese digital TV - was sent to > gconv upstream (not sure if it was merged or not). On other words, the > contrib/gconv stuff is independent of v4l-utils itself: it basically > adds an extra charset to be used by gconv upstream. > > The only reason why it is present at v4l2-utils is that this charset > is present on ISDB tables on TV broadcasts in Japan. The gconv library > needs to support it, in order to convert characters from MPEG tables > to the system's charset. > > Btw, that's basically one of the reasons why we print a summary of the > options when ./configure runs. -- Regards, Laurent Pinchart