From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 864EE42E22 for ; Wed, 8 Jun 2022 22:50:31 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id B841568B172; Thu, 9 Jun 2022 01:50:28 +0300 (EEST) Received: from NAM02-DM3-obe.outbound.protection.outlook.com (mail-dm3nam02olkn2011.outbound.protection.outlook.com [40.92.43.11]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3377C68B172 for ; Thu, 9 Jun 2022 01:50:22 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=a+el9gWFDS3vyVVndjAbOTicBtemSV2hvyUWjCVE8zU+bVDw6PJr4Gmb/09STas3g7Qt89wg7PbUwXTD9IalFVD0eNSVK8kLDXYTcMialXls1Ivg+iZPQLCLEOJi1swA7hZVowMWunaOszG+q6+2HNkiWkb1v/UsTPBuLno2lEPSi7DNAoOeyMeExoxGqt7LGvI/Hjq9GCwLB+t1W0THbkOFUNc8ISRNGqNGiniQLID3J3wNLFvmg8fU8JvxS+mYO9Og4bxLTARTcT4l0D+yVg0KcTFdTrXYz13UV9d9h89RekCWgJDVRMsNox4v7YmF/bLfuYjTwc/S3e61KtuGiQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=b74d6PaHHDgyEcMwo0Y5L0BhMyIFQJZi0Ir4mFbd9sI=; b=mTrOOj/mAe5/CYLLFBPwCOry0uO0v2x+cvmzWxUwz6cNMloJLVCcL941ILLe49sbO3y//eUrQQdNeKzLMBgGldtt58R0DBY3tNMauBdbwX+gZE5Phqc5to1BonFQzm/tRL0VdkgF8FzPpJhGql/o+yeWLPtqEPH2iN0SS0jUcyEA/HniPhQtNi7jnbz5t1tDwCivvsj1rRQmrDA1m5MZ+xB9M1QEfRK3rFKKf/jKC++jDnaZSgdg5uh5a9cXFapnP8dUgQcQBVWolkfC/TI1Ru1IGWDVuvtOYcRtYtzmPPsK7iuoU38dMfGOobSgVD0i/czudD6Vplvh/H7mg4VRBw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=b74d6PaHHDgyEcMwo0Y5L0BhMyIFQJZi0Ir4mFbd9sI=; b=W2WqiC6Oo3AatfX/DVFk0gXyNjp1b8dl2X/ueBlR/W/ieaO0quPdJvaVVHZ6Naz2DQZi4hXdzS2OgvkmMXI25KvGei6B2WtiJ+uu4ExeAld7s9SDEvvTSaqAtBbOqpWDuGVc6lIjO/R6YH9wxtwEUTQHfdVQRwFQZpPVOKWVA41mxYnWstca++w3hQqLUKF742BpJG3n/mDQbMvYBxbXP9YSxiVn3B4essSJZXlMvt7KWgdyCnAOojhuLt0/aabt5w4HqoIZeBdpm3McVaCm9BJlSnPYwEQKEemO1JrCRT6GScxfYPhJYo7GesTeBGEPw0RUHJQaskCTIn0E98SzLg== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by BN0P223MB0053.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:146::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5314.13; Wed, 8 Jun 2022 22:50:18 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a%7]) with mapi id 15.20.5314.019; Wed, 8 Jun 2022 22:50:18 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v6] libx264: Set min build version to 158 Thread-Index: AQHYcPwx51y6fdcHqEysYWR3Xx526K1GCOCAgAAksUA= Date: Wed, 8 Jun 2022 22:50:18 +0000 Message-ID: References: <137512e-90a-d163-d7a0-8d7e9309724@passwd.hu> In-Reply-To: <137512e-90a-d163-d7a0-8d7e9309724@passwd.hu> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [omkd8XkWkjovXzxdT2/RSjPJp8st7Yqj] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 5fba82f6-fe82-40a1-a040-08da49a142ec x-ms-traffictypediagnostic: BN0P223MB0053:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: fsj2Iry/ECpDdQKoG5PTfVIjPzRpRVsJbcAOgw8pSJDU92YcWcXml9ArZuWXW+BaQIqNpHrfIiWmDLGSuifzepYXeoJJJ8Zx5lj3IyRYUCKIaNWRIicshx8sTzN2QPQrnky6fmWcozs3+PqD5TpX7iHQttShSOQukn5F3R62glzzP+n/xmIH47TJ4YIpm4Xlvp+l3RgsQuD6S0cpoi8hwjHQ2cV5jSRmLrhsjf5hdu1rs+GTDIZdqw0Et5n43FHJ7hORO/BPd3bHh2tEQyYHdhJrjSr5mGqK9j754RNmBTHXtg8t9b2SEkeM0DB0HqzVd9pdC2dymWSvsayPCErwDFqglTLMLgcqm2dr7gVw9bHE3tcMhKlR01E9W5tOZY6FGXK1do9+YAgQIzQRnQoFmXc/pq7SLiWCqq6eGnjOx8Q7gwnK52FNF2dAKxdOn4UNVJXnN7zx+I+hZikqTs+8YavhZht8EjMlb3zNg3B8bH4sGRAOvV8RwyRwMyCbwXv35XQ0FKkGFymfGsJEUpYVuS0mkYOit39bsmekefK7KgaWm5ENnd9ES4kwBaMXjVrouTyGGpgzFlAceb3X7AxtvmBZ/1ALrt4w1vl7fRxitI+IrzgqsgMf4ieYWqp13IjrO05oMfD9KKJuNdJm/VMQlguXC6Yx54Mc+wLsYbrE/zM= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?Rk5nVDhvWDlhMm5rMnRBWHBHdnY3M01aMjBqbHJHaC9QenBEMENvbWRoakdF?= =?utf-8?B?WXk5SzlSdnNMRG1pdUc4ZktqWlI2TkVhZGRWU1BSQ3VVSEJMSjBSdjJBdFNU?= =?utf-8?B?b0QzMTVRSWJ6VlpJUThTdE9IdGJycXlXQzVaY3M3enhKZFdXdmh5dVpRUTRT?= =?utf-8?B?RzUxa2hqMnlGeHNzVGxiM0c5TUxVNGJta3lhRlVlUWNJR1ducmlWdU5lNnFS?= =?utf-8?B?Z1FuU25iakNJdkZ1eVZoRkpRN1JoT2orUzh0QnN2U0tEWlNibWRicU5UaHRk?= =?utf-8?B?QzNtaVErUzB6cHNXcHNGL3BRbDZVQk1vc2RuVVBoUHJZN3MwV0VBSlN5NUY2?= =?utf-8?B?Q2srOUZkSUFxR3p2V3M4L0R2OE1lbUhyY3NsazFrbUhONWtocmJhL2NuUmtQ?= =?utf-8?B?OEFsRk85OUxRa20rTndWRXJVa2tqZ1NxTSthUmg5dDg1M1hHa2pPR3owWVdr?= =?utf-8?B?U2crYWlsUVE0YmJEcndNRmthVW5mbWNCZTVXNG43UXhjWmFtQ2dXYXFCZmdl?= =?utf-8?B?b0w3Ylpwc0l0ZkJ3SW54WTFvK05qT0x4MUpJeWpQeHhUVXJGQUJiY29tY0I5?= =?utf-8?B?dEdNVHZBWElqREhuMjEzRDlzTUQwbzdhWnZLc09zWHV0M1U2OHdjeE1SdmIx?= =?utf-8?B?QVF6RHl0dE93NWlXUG9mbi9EUTNEb25OZEVzUmJrbmlGN1h4Snd6c1VFU1cv?= =?utf-8?B?WUloMnllcmFhWXBLM3A5STcyeVpkOG9USlNJR3c3S2hlQUVOb0MwZXRGNmlG?= =?utf-8?B?dUlBRnIxZ1VrNjBVSFk3aE5KN3hwMXVnSHI3MktSdjVHdVFTUGNZYjNXTDJ4?= =?utf-8?B?SzZyTDlCUjVRTFV1TENaM3o3Z1BJZzFxNW9sNFpjUjNaZHhUV21VNEVJUWgz?= =?utf-8?B?SWxRb2o2dTZNZ1czaW9CcHdQSytjWDlNYmpGQW1qeVRlSTRrMk8xV3F3Nyt5?= =?utf-8?B?bEgxcDhjRisxUmxBaHp1Q01MUEZENndHYWo0ZEFRN3BrWFNabnQzWldIbTNv?= =?utf-8?B?WThMV0c5ZUk5RTN3WGd0ZTIvblQ4b0phSVVpTE8zR3FnUDNDUWpKRDhLVGxK?= =?utf-8?B?UW1uSXpxQ0NJN0xhYVp2WnkwV0wxSXZ1cHF3bWNMUk5QSzZ0M3Z2VVI1blJy?= =?utf-8?B?TzdXNm1uWUZCeGNpdXhCVk00dGdvVGw4bnhPYWgxa28vdmw5RUVaZzJhRDR6?= =?utf-8?B?RVFCcm4zM0hzQXhqUWF6Mk5mNnVDeit0MWVrNWpQZmtOamJuejNvamxGcWtE?= =?utf-8?B?Tk00U0dWTFZPS2xIUWozTE5MR25RVEJJb0xlNWVkdmNKcDJzS2I3UTh4c29D?= =?utf-8?B?QVdCSG5WN0U1NC84UVdOL0s1WGQ4bGNBU1FJTUoyZDBZdkFsS0c2Y1JQWUxj?= =?utf-8?B?RUxEVmNQblNMOUJrOFdLN01mVXhMdExxcFRjZW03bFhYWFNON3dLcUtVbktS?= =?utf-8?B?TzgxK3VONkdaTGxnbnNOOWZXNmo4a0ZXcHpxa2F4OHFMOEVMM1NLdDBqUmhU?= =?utf-8?B?ZEhYVjRRMm41UC9QSDlCUHpVbzFoWDdna1EzSVZsZGw0WWdWd3F1REpTYk9B?= =?utf-8?B?dGg5N3A3ZGVaRzFMbkRWZTNlTUlyODV0VDVnWG1GcTNLbS9NMFJuM3NKYlJC?= =?utf-8?B?dmtOOGFZTnZra3BwcUl2ZE9Zd0oyWnozL2tkdzg5UFpSWTIweHE4VkdXVmxq?= =?utf-8?B?TE5pSktDRG1CNDBGaXM1U0RDaGptT0w4NVl6bTA3ZmkyWU9kemZHZ0Exd2dh?= =?utf-8?B?dEdyMEZIK0tCNVlZaGgrTS8xeVNhR1RsLzRKcUNUK2VzTys3dlV2Z0xoZGlX?= =?utf-8?B?ZWZJamEwSUZoY2FwYkordVQ1a0ZqbExKclBvNUs1bEN4RzBncGhvZWNLTGRa?= =?utf-8?B?SnNCTmpSTDQ5c2hGckliY2FNM2Q0a3FQeGNNd214by9SdjFZSSt0Q0VvVjJF?= =?utf-8?Q?LqH3xYuztz8=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 5fba82f6-fe82-40a1-a040-08da49a142ec X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Jun 2022 22:50:18.7400 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN0P223MB0053 Subject: Re: [FFmpeg-devel] [PATCH v6] libx264: Set min build version to 158 X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: > -----Original Message----- > From: ffmpeg-devel On Behalf Of Marton > Balint > Sent: Wednesday, June 8, 2022 10:22 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH v6] libx264: Set min build version to 158 > > > > On Thu, 26 May 2022, Matt Oliver wrote: > > > From: Matt Oliver > > > > Was "[PATCH] libx264: Do not explicitly set X264_API_IMPORTS" > > > > Setting X264_API_IMPORTS only affects msvc builds and it breaks > > linking to static builds (although is required for shared builds). > > This flag is set by x264 in its pkgconfig as required since build > > 158 (a615f027ed172e2dd5380e736d487aa858a0c4ff) from July 2019. > > So this patch updates configure to require a newer x264 build that > > correctly sets the imports flag. > > > > The requirement for 158 is applied for msvc builds only, > > no change is made for all other cases. > > > > Co-authored-by: softworkz > > Signed-off-by: softworkz > > Signed-off-by: Matt Oliver > > --- > > libx264: Set min build version to 158 > > > > I'm submitting this patch on behalf of Matt with his permission. > > > > There was agreement that the >= 158 version requirement should be > > applied to MSVC builds only. > > > > v2: restrict the version requirement to msvc builds > > v3: fix unintended author change > > v4: add missing braces > > v5: fixed condition (again ;-) > > v6: hope I got it now.. > > > > Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr- > ffstaging-30%2Fsoftworkz%2Fsubmit_x264_api_imports_matt-v6 > > Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging- > 30/softworkz/submit_x264_api_imports_matt-v6 > > Pull-Request: https://github.com/ffstaging/FFmpeg/pull/30 > > > > Range-diff vs v5: > > > > 1: 8c4fe7ffc2 ! 1: 47843fb51e libx264: Set min build version to 158 > > @@ configure: enabled libvpx && { > > - check_cpp_condition libx262 x264.h > "X264_MPEG2" > > +enabled libx264 && check_pkg_config libx264 x264 "stdint.h > x264.h" x264_encoder_encode && > > + require_cpp_condition libx264 x264.h > "X264_BUILD >= 118" && { > > -+ "$toolchain" != msvc || > require_cpp_condition libx264 x264.h "X264_BUILD >= 158"; } > > ++ [ "$toolchain" != "msvc" ] || > > ++ require_cpp_condition libx264 x264.h > "X264_BUILD >= 158"; } > > enabled libx265 && require_pkg_config libx265 x265 x265.h > x265_api_get && > > require_cpp_condition libx265 x265.h > "X265_BUILD >= 70" > > enabled libxavs && require libxavs "stdint.h xavs.h" > xavs_encoder_encode "-lxavs $pthreads_extralibs $libm_extralibs" > > > > > > configure | 9 ++++----- > > libavcodec/libx264.c | 4 ---- > > 2 files changed, 4 insertions(+), 9 deletions(-) > > > > diff --git a/configure b/configure > > index f115b21064..d17361f37f 100755 > > --- a/configure > > +++ b/configure > > @@ -6656,11 +6656,10 @@ enabled libvpx && { > > enabled libwebp && { > > enabled libwebp_encoder && require_pkg_config libwebp "libwebp >= > 0.2.0" webp/encode.h WebPGetEncoderVersion > > enabled libwebp_anim_encoder && check_pkg_config libwebp_anim_encoder > "libwebpmux >= 0.4.0" webp/mux.h WebPAnimEncoderOptionsInit; } > > -enabled libx264 && { check_pkg_config libx264 x264 "stdint.h > x264.h" x264_encoder_encode || > > - { require libx264 "stdint.h x264.h" > x264_encoder_encode "-lx264 $pthreads_extralibs $libm_extralibs" && > > - warn "using libx264 without pkg-config"; > } } && > > x264 without pkg-config feature got removed. If this is intentonal, > then maybe you should mention this in the commit message? I cannot honestly say that I would be sure about this part. Matt had this removed in his original patch and objections were made about the version requirement, but none about the removal of the "non-pkg-config" condition. Would there be any reasons to keep it? > > > - require_cpp_condition libx264 x264.h > "X264_BUILD >= 118" && > > - check_cpp_condition libx262 x264.h > "X264_MPEG2" > > Why is the x262 check got silently removed? This does not seem to belong > to this commit.` Matt had removed it and there was a comment about it saying that it would by dysfunctional for a long time already. By a funny coincidence, Gyan has submitted a patch for complete removal of this: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20220527082922.994-1-ffmpeg@gyani.pro/ Whether it belongs into this patch or not could be seen from two sides: On one side, you could say that THIS patch is about updating and adapting the x264 conditions to the state of time, but you could also say that it must rather be in Gyan's patch (which it is anyway). Just let me know when you think I should change it. PS: Thanks a lot for pushing the other patches! Best wishes, softworkz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".