From owner-freebsd-hackers@FreeBSD.ORG Sat Nov 6 07:28:29 2010 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 56353106566C; Sat, 6 Nov 2010 07:28:29 +0000 (UTC) (envelope-from swell.k@gmail.com) Received: from mail-yx0-f182.google.com (mail-yx0-f182.google.com [209.85.213.182]) by mx1.freebsd.org (Postfix) with ESMTP id AA6B38FC0A; Sat, 6 Nov 2010 07:28:28 +0000 (UTC) Received: by yxl31 with SMTP id 31so2721864yxl.13 for ; Sat, 06 Nov 2010 00:28:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:references :date:message-id:user-agent:mime-version:content-type; bh=PBhQHHL1T+Dclbi4ncnw+4IupmKyItwT1vdRawMq3ic=; b=MEJOTRj4F+gnQ4tPYEhZZnKOCKUwwh4wJSyHeJbM3mxqoaDGo8tLdpRvr/gYmQDSkQ SbTwx4L4LXFPEM0VFl5/f3nNWv6il8SLkEkBQ94POyuk+fbJg8+4bAFTN+1+87Dr7xsa WI9zBfkC1qr1EnzqjP2NkTnCGN4vdWcVSg+og= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:references:date:message-id:user-agent :mime-version:content-type; b=SHB3lPz2efg8zqH3ZNAL5JN8Su/UJAWcSO6ooNFaq3Gll9enFN3yex7Mw0/EKren9v JWWiGlE7RfEcYvoW6bMfGhnHi8NQgzS/vDWXYqqixKjeHjQMzpeb+X5xpQPZNw/HA1Uy V57MLyW0q3DyOr6O/tXhUMmhShObBwDd7L6mg= Received: by 10.100.209.5 with SMTP id h5mr551232ang.39.1289028507939; Sat, 06 Nov 2010 00:28:27 -0700 (PDT) Received: from localhost (tor-proxy.31173.se [193.138.216.157]) by mx.google.com with ESMTPS id b22sm2600908anb.15.2010.11.06.00.28.25 (version=SSLv3 cipher=RC4-MD5); Sat, 06 Nov 2010 00:28:27 -0700 (PDT) From: Anonymous To: Baptiste Daroussin References: <861v6zwo9j.fsf@gmail.com> <868w17v3b6.fsf@gmail.com> Date: Sat, 06 Nov 2010 10:28:13 +0300 Message-ID: <86y696q5bm.fsf@gmail.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain Cc: freebsd-hackers@freebsd.org Subject: Re: [PATCH] update to the latest libedit version and remove libreadline deps X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 06 Nov 2010 07:28:29 -0000 Anonymous writes: > Anonymous writes: > >> Baptiste Daroussin writes: > [...] >>> You can find the patch against current here: >>> http://people.freebsd.org/~bapt/update-libedit.patch >> >> $ make depend >> /usr/src/lib/libedit/makelist -h /usr/src/lib/libedit/vi.c > vi.h.tmp && mv vi.h.tmp vi.h >> /usr/src/lib/libedit/makelist: Permission denied >> *** Error code 126 >> >> I guess HOST_SH is defined by build.sh on NetBSD. > > libgdb doesn't build, too Aww, a few more changes were needed to make buildworld happy. %% Index: ObsoleteFiles.inc =================================================================== --- ObsoleteFiles.inc (revision 214854) +++ ObsoleteFiles.inc (working copy) @@ -14,6 +14,36 @@ # The file is partitioned: OLD_FILES first, then OLD_LIBS and OLD_DIRS last. # +# XXXXXXXX: readline removed, ports should use devel/readline ;) +OLD_FILES+=usr/include/readline/chardefs.h +OLD_FILES+=usr/include/readline/keymaps.h +OLD_FILES+=usr/include/readline/rlconf.h +OLD_FILES+=usr/include/readline/rlstdc.h +OLD_FILES+=usr/include/readline/rltypedefs.h +OLD_FILES+=usr/include/readline/tilde.h +OLD_FILES+=usr/lib/libhistory.a +OLD_FILES+=usr/lib/libhistory_p.a +OLD_FILES+=usr/lib/libreadline.a +OLD_FILES+=usr/lib/libreadline_p.a +OLD_FILES+=usr/share/info/history.info.gz +OLD_FILES+=usr/share/info/readline.info.gz +OLD_FILES+=usr/share/info/rluserman.info.gz +OLD_FILES+=usr/share/man/man3/readline.3.gz +OLD_FILES+=usr/share/man/man3/rlhistory.3.gz +OLD_LIBS+=lib/libreadline.so.8 +OLD_LIBS+=usr/lib/libhistory.so +OLD_LIBS+=usr/lib/libhistory.so.8 +OLD_LIBS+=usr/lib/libreadline.so +.if ${TARGET_ARCH} == "amd64" +OLD_FILES+=usr/lib32/libhistory.a +OLD_FILES+=usr/lib32/libhistory_p.a +OLD_FILES+=usr/lib32/libreadline.a +OLD_FILES+=usr/lib32/libreadline_p.a +OLD_LIBS+=usr/lib32/libhistory.so +OLD_LIBS+=usr/lib32/libhistory.so.8 +OLD_LIBS+=usr/lib32/libreadline.so +OLD_LIBS+=usr/lib32/libreadline.so.8 +.endif # 20101101: headers moved to machine/ to x86/ .if ${TARGET_ARCH} == "amd64" || ${TARGET_ARCH} == "i386" OLD_FILES+=usr/include/machine/apicreg.h Index: contrib/gdb/gdb/cli/cli-cmds.c =================================================================== --- contrib/gdb/gdb/cli/cli-cmds.c (revision 214854) +++ contrib/gdb/gdb/cli/cli-cmds.c (working copy) @@ -21,7 +21,6 @@ #include "defs.h" #include "readline/readline.h" -#include "readline/tilde.h" #include "completer.h" #include "target.h" /* For baud_rate, remote_debug and remote_timeout */ #include "gdb_wait.h" /* For shell escape implementation */ Index: contrib/gdb/gdb/cli/cli-setshow.c =================================================================== --- contrib/gdb/gdb/cli/cli-setshow.c (revision 214854) +++ contrib/gdb/gdb/cli/cli-setshow.c (working copy) @@ -18,7 +18,6 @@ Boston, MA 02111-1307, USA. */ #include "defs.h" -#include "readline/tilde.h" #include "value.h" #include #include "gdb_string.h" Index: contrib/gdb/gdb/event-top.c =================================================================== --- contrib/gdb/gdb/event-top.c (revision 214854) +++ contrib/gdb/gdb/event-top.c (working copy) @@ -34,7 +34,6 @@ /* readline include files */ #include "readline/readline.h" -#include "readline/history.h" /* readline defines this. */ #undef savestring Index: contrib/gdb/gdb/top.c =================================================================== --- contrib/gdb/gdb/top.c (revision 214854) +++ contrib/gdb/gdb/top.c (working copy) @@ -48,7 +48,6 @@ /* readline include files */ #include "readline/readline.h" -#include "readline/history.h" /* readline defines this. */ #undef savestring Index: contrib/gdb/gdb/tracepoint.c =================================================================== --- contrib/gdb/gdb/tracepoint.c (revision 214854) +++ contrib/gdb/gdb/tracepoint.c (working copy) @@ -45,7 +45,6 @@ /* readline include files */ #include "readline/readline.h" -#include "readline/history.h" /* readline defines this. */ #undef savestring Index: contrib/gdb/gdb/tui/tui-io.c =================================================================== --- contrib/gdb/gdb/tui/tui-io.c (revision 214854) +++ contrib/gdb/gdb/tui/tui-io.c (working copy) @@ -397,8 +397,6 @@ static void tui_rl_display_match_list (char **matches, int len, int max) { typedef int QSFUNC (const void *, const void *); - extern int _rl_qsort_string_compare (const void*, const void*); - extern int _rl_print_completions_horizontally; int count, limit, printed_len; int i, j, k, l; Index: gnu/lib/Makefile =================================================================== --- gnu/lib/Makefile (revision 214854) +++ gnu/lib/Makefile (working copy) @@ -2,7 +2,7 @@ .include -SUBDIR= csu libgcc libgcov libdialog libgomp libregex libreadline libssp +SUBDIR= csu libgcc libgcov libdialog libgomp libregex libssp # libsupc++ uses libstdc++ headers, although 'make includes' should # have taken care of that already. Index: include/Makefile =================================================================== --- include/Makefile (revision 214854) +++ include/Makefile (working copy) @@ -11,7 +11,7 @@ INCS= a.out.h ar.h assert.h bitstring.h complex.h db.h \ dirent.h dlfcn.h elf.h elf-hints.h err.h fmtmsg.h fnmatch.h fstab.h \ fts.h ftw.h getopt.h glob.h grp.h gssapi.h \ - histedit.h ieeefp.h ifaddrs.h \ + ieeefp.h ifaddrs.h \ inttypes.h iso646.h kenv.h langinfo.h libgen.h limits.h link.h \ locale.h malloc.h malloc_np.h memory.h monetary.h mpool.h mqueue.h \ ndbm.h netconfig.h \ Index: lib/libedit/Makefile =================================================================== --- lib/libedit/Makefile (revision 214856) +++ lib/libedit/Makefile (working copy) @@ -1,9 +1,8 @@ -# $NetBSD: Makefile,v 1.34 2005/05/28 12:02:53 lukem Exp $ # $NetBSD: Makefile,v 1.41 2010/02/03 15:34:43 roy Exp $ # @(#)Makefile 8.1 (Berkeley) 6/4/93 # $FreeBSD$ -WIDECHAR ?= yes +WIDECHAR ?= no WARNS?= 3 LIB= edit SHLIB_MAJOR= 7 @@ -40,6 +40,7 @@ CFLAGS+=-DWIDECHAR .endif LIBEDITDIR?=${.CURDIR} +HOST_SH?=sh INCS= histedit.h INCSDIR=/usr/include Index: lib/libedit/readline/Makefile =================================================================== --- lib/libedit/readline/Makefile (revision 0) +++ lib/libedit/readline/Makefile (revision 0) @@ -5,10 +5,8 @@ .include -.PATH: ${NETBSDSRCDIR}/lib/libedit - INCS= readline.h INCSDIR= /usr/include/readline -INCSYMLINKS= readline.h ${INCSDIR}/history.h +INCSLINKS= readline.h ${INCSDIR}/history.h .include Index: rescue/rescue/Makefile =================================================================== --- rescue/rescue/Makefile (revision 214854) +++ rescue/rescue/Makefile (working copy) @@ -143,7 +143,7 @@ CRUNCH_LIBS+= -lipx .if ${MK_ZFS} != "no" CRUNCH_LIBS+= -lzfs -lnvpair -luutil -lavl .endif -CRUNCH_LIBS+= -lgeom -lbsdxml -ljail -lkiconv -lmd -lreadline -lsbuf -lufs -lz +CRUNCH_LIBS+= -lgeom -lbsdxml -ljail -lkiconv -lmd -lsbuf -lufs -lz .if ${MACHINE_CPUARCH} == "i386" CRUNCH_PROGS_sbin+= bsdlabel sconfig fdisk Index: sbin/gvinum/Makefile =================================================================== --- sbin/gvinum/Makefile (revision 214854) +++ sbin/gvinum/Makefile (working copy) @@ -7,8 +7,8 @@ MAN= gvinum.8 WARNS?= 2 CFLAGS+= -I${.CURDIR}/../../sys -DPADD= ${LIBREADLINE} ${LIBTERMCAP} ${LIBDEVSTAT} ${LIBKVM} ${LIBGEOM} -LDADD= -lreadline -ltermcap -ldevstat -lkvm -lgeom +DPADD= ${LIBEDIT} ${LIBTERMCAP} ${LIBDEVSTAT} ${LIBKVM} ${LIBGEOM} +LDADD= -ledit -ltermcap -ldevstat -lkvm -lgeom .PATH: ${.CURDIR}/../../sys/geom/vinum Index: sbin/gvinum/gvinum.c =================================================================== --- sbin/gvinum/gvinum.c (revision 214854) +++ sbin/gvinum/gvinum.c (working copy) @@ -52,7 +52,6 @@ #include #include #include -#include #include #include "gvinum.h" %%