From owner-freebsd-ports-bugs@freebsd.org Tue Nov 3 08:21:49 2015 Return-Path: Delivered-To: freebsd-ports-bugs@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id BC975A25707 for ; Tue, 3 Nov 2015 08:21:49 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 8D2ED13A8 for ; Tue, 3 Nov 2015 08:21:49 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id tA38Lnfd058993 for ; Tue, 3 Nov 2015 08:21:49 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 203639] security/krb5: readline not enabled by option Date: Tue, 03 Nov 2015 08:21:49 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Some People X-Bugzilla-Who: z7dr6ut7gs@snkmail.com X-Bugzilla-Status: Open X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: cy@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: maintainer-feedback? X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Nov 2015 08:21:49 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203639 --- Comment #2 from John Hein --- I was partly wrong. My description of the way to show the problem wasn't quite right - the CONFIGURE_ARGS were correct, but LIB_DEPENDS was not. But the core problem that readline.mk is not included when using .pre.mk/.post.mk is valid. Try reverting the krb5/Makefile before your 2015-10-19 changes in r399631, or just temporarily modify krb5/Makefile to use .include & .include (instead of using bsd.port.options.mk & bsd.port.mk which were in the suggested patch here and also part of the r399631 changes). Then if you do 'make -V LIB_DEPENDS', you should see that devel/readline is not in the list of LIB_DEPENDS. To further illustrate the issue, you could also add a .warning in Mk/Uses/readline.mk. Then run the 'make -V LIB_DEPENDS' again and see that readline.mk is never included. Then use bsd.port.options.mk/bsd.port.mk (instead of .pre.mk/.post.mk) with the .warning still in readline.mk. You should observe that readline.mk is then included. This happens generally when one uses .pre.mk/.post.mk and test for PORT_OPTIONS in the port Makefile before .post.mk. This is why the typical idiom will include bsd.port.options.mk before the PORT_OPTIONS tests. This is harder to notice for USES=readline, because the effects of missing readline.mk are typically not catastrophic in most cases these days - since /usr/lib/libreadline.so almost always exists at the moment unless you're using -current and have a world built WITHOUT_BINUTILS and WITHOUT_GDB _and_ you don't have the readline port installed _and_ the port in question doesn't detect the missing readline lib and handle it gracefully. Those are lots of conditions necessary to cause the missed readline.mk include to be noticed by the typical observer. -- You are receiving this mail because: You are on the CC list for the bug.