From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jEWHC-0000x7-51 for ptxdist@pengutronix.de; Wed, 18 Mar 2020 11:50:18 +0100 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1jEWHB-0007GP-Rw for ptxdist@pengutronix.de; Wed, 18 Mar 2020 11:50:17 +0100 Date: Wed, 18 Mar 2020 11:50:17 +0100 From: Michael Olbrich Message-ID: <20200318105017.GE2448@pengutronix.de> References: <20200213172208.22452-6-matthias.schiffer@ew.tq-group.com> <20200220083947.28574-1-matthias.schiffer@ew.tq-group.com> <20200221074712.GC4597@pengutronix.de> <1c7e6793cb9fa772af19e0691c8140307471d803.camel@ew.tq-group.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1c7e6793cb9fa772af19e0691c8140307471d803.camel@ew.tq-group.com> Subject: Re: [ptxdist] (EXT) Re: [PATCH v2] syslogng: update to 3.25.1 List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ptxdist-bounces@pengutronix.de Sender: "ptxdist" To: ptxdist@pengutronix.de On Fri, Feb 21, 2020 at 09:30:30AM +0100, Matthias Schiffer wrote: > On Fri, 2020-02-21 at 08:47 +0100, Michael Olbrich wrote: > > On Thu, Feb 20, 2020 at 09:39:47AM +0100, Matthias Schiffer wrote: > > > --with-systemd-journal=$(call ptx/ifdef, > > > PTXCONF_SYSLOGNG_SYSTEMD,system,no) \ > > > --with-systemdsystemunitdir=/usr/lib/systemd/system \ > > > --localstatedir=/var/run \ > > > + --with-ivykis=internal \ > > > + --with-jsonc=$(if $(PTXCONF_SYSLOGNG_JSON),system,no) \ > > > --with-libnet=$(SYSROOT)/usr/bin \ > > > - --with-python=$(PYTHON_MAJORMINOR) > > > + --with-python=$(PYTHON3_MAJORMINOR) > > > > As I noted in my last review, please use configure_helper.py to check > > and > > improve the options. > > - GLOBAL_LARGE_FILE_OPTION should be used > > - the sorting should be corrected > > - there are several enable/disable and with/without options that are > > missing and may pick up dependencies automatically. > > Ah, I didn't know about GLOBAL_LARGE_FILE_OPTION, and mostly used > configure_helper.py to compare the old and the new syslogng version. > > Should I list *all* options that configure supports, even when the > defaults are fine? I thought I had caught everything that has an actual > effect. I guess disabling libcurl etc. could be made explicit as well, > but these deps are only used when the corresponding modules are > enabled. > > I also found the option --disable-all-modules, which overrides the > defaults for various other options to "no" - which would allow us to > make the SYSLOGNG_AUTOCONF section shorter, but the diff shown by > configure_helper.py larger. Which way should I go here? To make updates easier, I prefer a shorter configure_helper.py output: - If it makes sense, then the output should be empty (probably not reasonable here) - options that obviously don't matter are find - stuff like Windows or OSX specific options - some --with-* options for extra search paths ... - any option that avoids / forces a autodetect check _must_ be specified. Never depend on the auto detections. - not adding options that can overwrite a fixed value (paths etc.) is ok - make sure the options are sorted correctly I hope this helps. Michael -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de