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=-7.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 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 70587C48BE5 for ; Wed, 16 Jun 2021 14:59:55 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4AA4861166 for ; Wed, 16 Jun 2021 14:59:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234340AbhFPPCA (ORCPT ); Wed, 16 Jun 2021 11:02:00 -0400 Received: from mail.kernel.org ([198.145.29.99]:42014 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234293AbhFPPCA (ORCPT ); Wed, 16 Jun 2021 11:02:00 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id D508F6115C; Wed, 16 Jun 2021 14:59:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1623855594; bh=uSU0h88tXbXQee4T+znHkJJnj01kzkxlTTk4NWtzrYw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Q5YUHU7/TGQIFS2BRCY/a8iKIbg7eYSijjF8XJeQHzy34OGvqF17PXR32Dav4FgFF SDFfOtk4ob5KxEdJMtheLY24k91S17aVUUFmhodn+qoJNkbodfa71DClv+pqJfpE/D Ba1ACdyFsD+l9KVD0ls6dha9LZ/OWrofKM/sPKzJprJVza0xrbDMRrxmbVULjCoCCR 1NpdP39qsjjxXIUdVfMiOxFnh59ojxtabE6R+KhjpjGP3A+MeE1XYT4oxjVVianDRj lZKSPd1Y8rHAHoJpgAVyEGJNEud6Tg3xjNN8BqBSNmVq877RLsSP8HvpsoiBP0TmFs gFiuMqcG4Y3/A== Date: Wed, 16 Jun 2021 16:59:47 +0200 From: Mauro Carvalho Chehab To: Ariel D'Alessandro Cc: kieran.bingham@ideasonboard.com, linux-media@vger.kernel.org, hverkuil@xs4all.nl, sean@mess.org, p.zabel@pengutronix.de, laurent.pinchart@ideasonboard.com, 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: <20210616165947.70f73cec@coco.lan> In-Reply-To: <443286a1-b955-1ac1-742d-42b9182a435f@collabora.com> 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> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org 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. > 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... > 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. Thanks, Mauro