From owner-dev-commits-src-all@freebsd.org Thu Jan 7 13:54:01 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 03E634D194A; Thu, 7 Jan 2021 13:54:01 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (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-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DBSQ06hYdz553Q; Thu, 7 Jan 2021 13:54:00 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-qt1-f173.google.com (mail-qt1-f173.google.com [209.85.160.173]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id D401325A1F; Thu, 7 Jan 2021 13:54:00 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-qt1-f173.google.com with SMTP id a6so4239877qtw.6; Thu, 07 Jan 2021 05:54:00 -0800 (PST) X-Gm-Message-State: AOAM533JbhRqoMHzLUdoi/Jxm+I3g5dDN7MKg8fCmn0yXVKCEHj0U+gw 0UHvtfRzhr0hfNmAqYQfVfUdFRF9YAMdinulbgc= X-Google-Smtp-Source: ABdhPJzNMF6KJBuIScGrhcdtj9SuXwz9hyaZoCskfndvd/RK155QE93JXhTzQEYsk68f34et1Vmlr/CocaWLUvBBJvQ= X-Received: by 2002:ac8:4e87:: with SMTP id 7mr8473153qtp.310.1610027640091; Thu, 07 Jan 2021 05:54:00 -0800 (PST) MIME-Version: 1.0 References: <202101051302.105D24lW041218@gitrepo.freebsd.org> In-Reply-To: <202101051302.105D24lW041218@gitrepo.freebsd.org> From: Kyle Evans Date: Thu, 7 Jan 2021 07:53:47 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: git: 821aa63a0940 - main - ncurses: only keep the version with widechar support To: Baptiste Daroussin Cc: src-committers , dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Jan 2021 13:54:01 -0000 On Tue, Jan 5, 2021 at 7:02 AM Baptiste Daroussin wrote: > > The branch main has been updated by bapt: > > URL: https://cgit.FreeBSD.org/src/commit/?id=821aa63a09402935da0a73abf20ba0441562aa07 > > commit 821aa63a09402935da0a73abf20ba0441562aa07 > Author: Baptiste Daroussin > AuthorDate: 2021-01-04 16:29:40 +0000 > Commit: Baptiste Daroussin > CommitDate: 2021-01-05 13:01:32 +0000 > > ncurses: only keep the version with widechar support > > Only keep the widechar version of ncurses as libncursesw.so.9 > > Keep the old name to avoid breaking the ABI compatibility (the non > widechar version libncurses.so.9 is not binary compatible with > libncursesw.so.9) since all ports and base are already only linking > against the widechar version we can simply remove libncurses.so.9 > > Since the .9 version only lived in the dev branch and never ended in a > release, it is simply removed and not added to any binary compat > package. > > Add symlinks to keep build time compatibility for anyone linking against > -lncurses > --- > Makefile.inc1 | 5 +-- > Makefile.libcompat | 4 +- > ObsoleteFiles.inc | 2 + > lib/ncurses/Makefile | 6 +-- > lib/ncurses/config.mk | 11 +----- > lib/ncurses/form/Makefile | 18 ++++++--- > lib/ncurses/formw/Makefile | 5 --- > lib/ncurses/formw/Makefile.depend | 17 --------- > lib/ncurses/menu/Makefile | 18 ++++++--- > lib/ncurses/menuw/Makefile | 5 --- > lib/ncurses/menuw/Makefile.depend | 17 --------- > lib/ncurses/ncurses/Makefile | 71 +++++++++++++++-------------------- > lib/ncurses/ncursesw/Makefile | 7 ---- > lib/ncurses/ncursesw/Makefile.depend | 16 -------- > lib/ncurses/panel/Makefile | 23 ++++++++---- > lib/ncurses/panelw/Makefile | 5 --- > lib/ncurses/panelw/Makefile.depend | 17 --------- > share/mk/src.libnames.mk | 16 ++------ > tools/bsdbox/Makefile.base | 2 +- > tools/tools/ath/athratestats/Makefile | 2 +- > 20 files changed, 87 insertions(+), 180 deletions(-) > > diff --git a/Makefile.inc1 b/Makefile.inc1 > index fecdc1bdca71..a19fde9195cf 100644 > --- a/Makefile.inc1 > +++ b/Makefile.inc1 > @@ -2524,7 +2524,6 @@ _bt_clean= ${CLEANDIR} > ${LOCAL_TOOL_DIRS} \ > ${_jevents} \ > lib/ncurses/ncurses \ > - lib/ncurses/ncursesw \ > ${_rescue} \ > ${_share} \ > usr.bin/awk \ > @@ -2870,7 +2869,7 @@ _prebuild_libs= ${_kerberos5_lib_libasn1} \ > lib/libkiconv lib/libkvm lib/liblzma lib/libmd lib/libnv \ > lib/libzstd \ > ${_lib_casper} \ > - lib/ncurses/ncurses lib/ncurses/ncursesw \ > + lib/ncurses/ncurses \ > lib/libopie lib/libpam/libpam ${_lib_libthr} \ > ${_lib_libradius} lib/libsbuf lib/libtacplus \ > lib/libgeom \ > @@ -2891,7 +2890,7 @@ _prebuild_libs= ${_kerberos5_lib_libasn1} \ > > .if ${MK_DIALOG} != "no" > _prebuild_libs+= gnu/lib/libdialog > -gnu/lib/libdialog__L: lib/msun__L lib/ncurses/ncursesw__L > +gnu/lib/libdialog__L: lib/msun__L lib/ncurses/ncurses__L > .endif > > .if ${MK_GOOGLETEST} != "no" > diff --git a/Makefile.libcompat b/Makefile.libcompat > index 3f23a33aa19b..85859a960d81 100644 > --- a/Makefile.libcompat > +++ b/Makefile.libcompat > @@ -45,7 +45,7 @@ _LC_LIBDIRS.${MK_KERBEROS:tl}+= kerberos5/lib > > _LC_INCDIRS= \ > include \ > - lib/ncurses/ncursesw \ > + lib/ncurses/ncurses \ > ${_LC_LIBDIRS.yes} > > .if ${MK_FILE} != "no" > @@ -94,7 +94,7 @@ build${libcompat}: .PHONY > DIRPRFX=${_dir}/ ${_t} > .endfor > .endfor > -.for _dir in lib/ncurses/ncurses lib/ncurses/ncursesw ${_libmagic} ${_jevents} > +.for _dir in lib/ncurses/ncurses lib/ncurses/ncurses ${_libmagic} ${_jevents} > .for _t in ${_obj} build-tools > ${_+_}cd ${.CURDIR}/${_dir}; \ > WORLDTMP=${WORLDTMP} \ The second occurence of lib/ncurses/ncurses should probably just go away? > diff --git a/lib/ncurses/config.mk b/lib/ncurses/config.mk > index d15be8601136..f199c5829b65 100644 > --- a/lib/ncurses/config.mk > +++ b/lib/ncurses/config.mk > @@ -4,20 +4,11 @@ > > NCURSES_DIR= ${SRCTOP}/contrib/ncurses > > -.if defined(ENABLE_WIDEC) > -LIB_SUFFIX= w > CFLAGS+= -D_XOPEN_SOURCE_EXTENDED -DENABLE_WIDEC > -NCURSES_CFG_H= ${.CURDIR:H}/ncurses/ncurses_cfg.h > -.else > -LIB_SUFFIX= > NCURSES_CFG_H= ${.CURDIR}/ncurses_cfg.h > -.endif > > CFLAGS+= -I. > -.if exists(${.OBJDIR:H}/ncurses${LIB_SUFFIX}) > -CFLAGS+= -I${.OBJDIR:H}/ncurses${LIB_SUFFIX} > -.endif > -CFLAGS+= -I${.CURDIR:H}/ncurses${LIB_SUFFIX} > +CFLAGS+= -I${.CURDIR:H}/ncurses > > # for ${NCURSES_CFG_H} > CFLAGS+= -I${.CURDIR:H}/ncurses Based on this we should really just nuke the entire ncurses dir for older WITHOUT_CLEAN builds, because any objects in this objdir will have been built without -DENABLE_WIDEC. Maybe just key it off of an ncursesw directory existing in the objdir hierarchy and nuke it all in depend-cleanup. Thanks, Kyle Evans