From nobody Tue Sep 30 19:25:00 2025 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4cbp0w6H2wz697fB; Tue, 30 Sep 2025 19:25:04 +0000 (UTC) (envelope-from mad@madpilot.net) Received: from vogon.madpilot.net (vogon.madpilot.net [159.69.1.99]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4cbp0w5QkVz3X0w; Tue, 30 Sep 2025 19:25:04 +0000 (UTC) (envelope-from mad@madpilot.net) Authentication-Results: mx1.freebsd.org; none Received: from localhost (mail [IPv6:fd5c:5351:d272::3]) by vogon.madpilot.net (Postfix) with ESMTP id 4cbp0v4bnkzLlyr; Tue, 30 Sep 2025 21:25:03 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=madpilot.net; h= content-transfer-encoding:content-type:content-type:in-reply-to :from:from:content-language:references:subject:subject:date:date :message-id:received; s=cyq4qetkgngm; t=1759260301; x= 1761074702; bh=e967tN4J40YzaEJSbFfQVPgySv+n8VoUF2ioFNgOwTI=; b=n xz2+ZYbujiFUl9XK6yvsGqzMvxJvKfmAEuuFjHbRmfsn0Bgj6g8SYY2XeAONKHhv XQ+XShem7SlWowHROg+inxZBBucNgGagmjtTFqiCtXmKKz1y0DmioZYlYnANEHPc 7VvAT+cEb6FesFUxlAoGUSknIoQlvpb4gY835tL3DB4QFKUxu6OaawWAf+kVPHn7 6MyM7GEwTMTY4R01uK4JJ3o8V9BsFWMcFWAJW+C+R29+aCgBj9+CE/ePS2uqByzL EzzByEIN/TWpT0u6BiFDgAZuXy7k++ORmFrXiUvpFoKSGx2/RU8iG/T5dStpclMl 7Q8N5lboO/PdlS6XWzFjQ== Received: from vogon.madpilot.net ([IPv6:fd5c:5351:d272::3]) by localhost (vogon.madpilot.net [IPv6:fd5c:5351:d272::3]) (amavis, port 10026) with ESMTP id MxDRaVz-xeKZ; Tue, 30 Sep 2025 21:25:01 +0200 (CEST) Message-ID: Date: Tue, 30 Sep 2025 21:25:00 +0200 Subject: Re: git: 06afa0a55e0a - main - pkgbase: move ncurses into its own package To: Dima Panov , Kyle Evans , src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org References: <202509301222.58UCM0MP003913@gitrepo.freebsd.org> Content-Language: en-US From: Guido Falsi Autocrypt: addr=mad@madpilot.net; keydata= xsBNBE+G+l0BCADi/WBQ0aRJfnE7LBPsM0G3m/m3Yx7OPu4iYFvS84xawmRHtCNjWIntsxuX fptkmEo3Rsw816WUrek8dxoUAYdHd+EcpBcnnDzfDH5LW/TZ4gbrFezrHPdRp7wdxi23GN80 qPwHEwXuF0X4Wy5V0OO8B6VT/nA0ADYnBDhXS52HGIJ/GCUjgqJn+phDTdCFLvrSFdmgx4Wl c0W5Z1p5cmDF9l8L/hc959AeyNf7I9dXnjekGM9gVv7UDUYzCifR3U8T0fnfdMmS8NeI9NC+ wuREpRO4lKOkTnj9TtQJRiptlhcHQiAlG1cFqs7EQo57Tqq6cxD1FycZJLuC32bGbgalABEB AAHNHkd1aWRvIEZhbHNpIDxtYWRAbWFkcGlsb3QubmV0PsLAeQQTAQgAIwIbAwIeAQIXgAUL CQgHAwUVCgkICwQWAgMBBQJS79AgAhkBAAoJEBrmhg5Wy9KTc0kH/RO64ORBlTbTHaUaOj8F Je5O5NU2Pt9Cyt5ZWBRvxntr1zPTJGKRPS9ihlIfqT4ZvEngQGp57EUyFbCpI0UWasTerImM tt5WACnGmCzUTB39UXx8Oy4b1EgWeTJQ747e/F1mQLXTNa6ijRBE9fYlTb4gAkPN88/wVV9v 3PZozKLTg16ghBzHM/P7Lk8L7clPEZChX1FTa/6eSt3nvzfCuTMZbBPJF/ph+q1KyPqRgVfh tyhu5dvgMoPz/ni41IfeSrkJTD5RXzdyGR9q4Z1NYeBsLkRjC4LxKAP5KqUsvlOUjKvO1byj ApYdMarol+IGkaSk9e3zVYAJkWKjn/ni8XbOwU0EUxB7QQEQAKFhrDceoPdK/IHDSmoj6SQY isvM7VdhcleS7E9DoEAVt7yMbf6HbbMVTTY6ckvwTWQssywLBXNVqxgc4WLJjzfUhgef+WE7 5M3+WFYlOVQLGZY/zEVgma1raYnOHNAOzeHLDmEXjbZP6vGAeDyBbGfQPpE7qGYZ7ubeT3Xw QO+PklcCrvOPj2ZPcAxGNS2xVU/LzONqCrJqLMJSIcCdsbiSP4G5PnDFHtMokaTY6OEr8OEQ fOAerhcHUa/z7Uu8YtmaqKH+QGkE/WEgaRqSiTnv0JOTD+DxehaqvoKPPZ++2NpCZMHB2i6A /xifmQwEiIjEXtcueBRzkNUQkxhqZyS13SrhocL9ydtaVPBzZatAEjUDDEJmAMLVFs45qfyh MiNapHJo2n3MW/E5omqCvEkDdWX/en3P7CK2TemeaDghMsgkNKax/z0wNo5UZCkOPOz0xpNi UilOVbkuezZZNg65741qee2lfXhQIaZ66yT7hphc/N/z3PIAtLeze4u1VR2EXAuZ2sWAdlKC NTlJMsaU/x70BV11Wd/ypnVzM68dfdQIIAj1iMFAD/lXGlEUmKXg5Ov2VQDlTntQoanCYrAg +8CttPzjrydgLZFq3hrtQmfc0se5yv1WHS69+BsUOG09RvvawUDZxUjW19kyeN9THaNRgow3 kSuArUp6zSmJABEBAAHCwF8EGAEIAAkFAlMQe0ECGwwACgkQGuaGDlbL0pMN5wgA4bCkX/qw EVC06ToeR6C2putmSWQMgpDaqrv65Hubo+QGmg2P4ewTYQQ4g6oYWS03qHxqVVWhKz7FjfrV +dH8qbCLfSgIcvdBha7ayGZVrsiuMLKGbw36fcmkZPpSDOfHcP0XH8Z+u9CWj0xUkTxAlZ/7 i6gYSUpG2JWNtdmE/X8VVEyXusCLwy0K0BI60A/4dRTIX3C4QKrJ3ZbUXegz70ynjHf+lQMZ 9IZKASoRMuS5FozPQh6abvmwZEPdf5I9riUElzvHrqJ8Bx0t3Pujdoth+yNHpnBxrtO8LkQd rQ58P0SwcaIX33T2U9pG8bhu5YVR88FQ8OQ0cEsPBpDncg== In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:24940, ipnet:159.69.0.0/16, country:DE] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Rspamd-Queue-Id: 4cbp0w5QkVz3X0w List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org On 9/30/25 21:15, Dima Panov wrote: > Hello! > > It fails :( > > /opt/poudriere/jails/160amd64/usr/src/release/packages/generate-ucl.lua: > fail to parse(/opt/poudriere/jails/160amd64/usr/src/release/packages/ > template.ucl): error while parsing /opt/poudriere/jails/160amd64/usr/ > src/release/packages/ucl/ncurses-all.ucl: at the end of chunk: > unfinished multiline comment > *** [create-world-package-ncurses-dbg-lib32] Error code 1 > I was able to get through this error by locally removing the comment and adding empty lines around each statement in the file. Not sure which of the two worked, but if I should guess it was removing the comment. But I have no idea why it was failing, really. After this though I stumbled on the snd issue I reported right now. > > On 30.09.2025 15:22, Kyle Evans wrote: >> The branch main has been updated by kevans: >> >> URL: https://cgit.FreeBSD.org/src/commit/? >> id=06afa0a55e0acc00823f08fe17305df13502f235 >> >> commit 06afa0a55e0acc00823f08fe17305df13502f235 >> Author:     Kyle Evans >> AuthorDate: 2025-09-30 12:21:33 +0000 >> Commit:     Kyle Evans >> CommitDate: 2025-09-30 12:21:33 +0000 >> >>      pkgbase: move ncurses into its own package >>      Pushing ncurses into clibs adds extra upgrade risk to a package that >>      otherwise consists of integral libs with non-trivial >> interdependencies. >>      Adding it to -runtime was considered, but the move was historically >>      motivated to some extent by allowing a much smaller set of base >> libs to >>      be installed for smaller VM images that don't really need all of the >>      runtime package. >>      This also fixes the ncurses build to ensure that libmenuw is grouped >>      with the rest of the ncurses libraries, which doesn't seem to >> have been >>      an intentional omission. >>      Reviewed by:    ivy >>      MFC after:      2 days (pkgbase movement) >>      Differential Revision:  https://reviews.freebsd.org/D52786 >> --- >>   lib/ncurses/Makefile.inc             |  1 + >>   lib/ncurses/form/Makefile            |  1 - >>   lib/ncurses/ncurses/Makefile         |  1 - >>   lib/ncurses/panel/Makefile           |  1 - >>   lib/ncurses/tinfo/Makefile           |  1 - >>   release/packages/ucl/ncurses-all.ucl | 39 ++++++++++++++++++++++++++ >> ++++++++++ >>   6 files changed, 40 insertions(+), 4 deletions(-) >> >> diff --git a/lib/ncurses/Makefile.inc b/lib/ncurses/Makefile.inc >> index f2ed84e2e38f..eea49908474c 100644 >> --- a/lib/ncurses/Makefile.inc >> +++ b/lib/ncurses/Makefile.inc >> @@ -1,5 +1,6 @@ >>   # This is to include src/lib/Makefile.inc >> +PACKAGE?=    ncurses >>   WARNS?=    3 >>   .include "../Makefile.inc" >> diff --git a/lib/ncurses/form/Makefile b/lib/ncurses/form/Makefile >> index d4774a5f51a8..7d8963b05ae8 100644 >> --- a/lib/ncurses/form/Makefile >> +++ b/lib/ncurses/form/Makefile >> @@ -3,7 +3,6 @@ >>   SRCDIR=    ${NCURSES_DIR}/form >> -PACKAGE=    clibs >>   LIB=    formw >>   SHLIB_MAJOR=    6 >> diff --git a/lib/ncurses/ncurses/Makefile b/lib/ncurses/ncurses/Makefile >> index 512a452ad6d0..b60760281b39 100644 >> --- a/lib/ncurses/ncurses/Makefile >> +++ b/lib/ncurses/ncurses/Makefile >> @@ -1,4 +1,3 @@ >> -PACKAGE=    clibs >>   SHLIBDIR?=    /lib >>   .include >> diff --git a/lib/ncurses/panel/Makefile b/lib/ncurses/panel/Makefile >> index a51129df42e3..dfb478aa5f5b 100644 >> --- a/lib/ncurses/panel/Makefile >> +++ b/lib/ncurses/panel/Makefile >> @@ -3,7 +3,6 @@ >>   SRCDIR=    ${NCURSES_DIR}/panel >> -PACKAGE=    clibs >>   LIB=    panelw >>   SHLIB_MAJOR=    6 >> diff --git a/lib/ncurses/tinfo/Makefile b/lib/ncurses/tinfo/Makefile >> index 476df54bb72a..8f01557edaee 100644 >> --- a/lib/ncurses/tinfo/Makefile >> +++ b/lib/ncurses/tinfo/Makefile >> @@ -1,4 +1,3 @@ >> -PACKAGE=    clibs >>   SHLIBDIR?=    /lib >>   .include >> diff --git a/release/packages/ucl/ncurses-all.ucl b/release/packages/ >> ucl/ncurses-all.ucl >> new file mode 100644 >> index 000000000000..ded994ebb3aa >> --- /dev/null >> +++ b/release/packages/ucl/ncurses-all.ucl >> @@ -0,0 +1,39 @@ >> +/* >> + * Copyright 2018-2023,2024 Thomas E. Dickey >> + * Copyright 1998-2017,2018 Free Software Foundation, Inc. >> + * >> + * Permission is hereby granted, free of charge, to any person >> obtaining a >> + * copy of this software and associated documentation files (the >> + * "Software"), to deal in the Software without restriction, including >> + * without limitation the rights to use, copy, modify, merge, publish, >> + * distribute, distribute with modifications, sublicense, and/or sell >> + * copies of the Software, and to permit persons to whom the Software is >> + * furnished to do so, subject to the following conditions: >> + * >> + * The above copyright notice and this permission notice shall be >> included >> + * in all copies or substantial portions of the Software. >> + * >> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, >> EXPRESS >> + * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF >> + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND >> NONINFRINGEMENT. >> + * IN NO EVENT SHALL THE ABOVE COPYRIGHT HOLDERS BE LIABLE FOR ANY >> CLAIM, >> + * DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR >> + * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR >> + * THE USE OR OTHER DEALINGS IN THE SOFTWARE. >> + * >> + * Except as contained in this notice, the name(s) of the above >> copyright >> + * holders shall not be used in advertising or otherwise to promote the >> + * sale, use or other dealings in this Software without prior written >> + * authorization. >> + */ >> + >> +comment = "ncurses terminal control library" >> +desc = <> +The ncurses package is a subroutine library for terminal-independent >> +screen-painting and input-event handling which presents a high level >> screen >> +model to the programmer.  ncurses implements the System V Release 4.0 >> (SVR4) >> +curses interface, and is also backward compatible with traditional >> BSD curses. >> + >> +This package ships with a limited termcap database, but it is >> designed to use >> +a terminfo database provided by ports, such as in misc/terminfo-db. >> +EOD >> > -- Guido Falsi