Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66]) by www.NetBSD.org (Postfix) with ESMTP id 07BF263B86B for ; Tue, 22 May 2012 11:02:32 +0000 (UTC) Received: by mail.netbsd.org (Postfix, from userid 605) id 90B0814A2D1; Tue, 22 May 2012 11:02:53 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by mail.netbsd.org (Postfix) with ESMTP id 1EE5614A2C5 for ; Tue, 22 May 2012 11:02:51 +0000 (UTC) X-Virus-Scanned: amavisd-new at NetBSD.org Received: from mail.netbsd.org ([127.0.0.1]) by localhost (mail.NetBSD.org [127.0.0.1]) (amavisd-new, port 10025) with ESMTP id ph4aD8Rr1Q9Q for ; Tue, 22 May 2012 11:02:50 +0000 (UTC) Received: from cvs.netbsd.org (cvs.NetBSD.org [IPv6:2001:4f8:3:7:2e0:81ff:fe30:95bd]) by mail.netbsd.org (Postfix) with ESMTP id 79D0514A2AC for ; Tue, 22 May 2012 11:02:50 +0000 (UTC) Received: by cvs.netbsd.org (Postfix, from userid 500) id 0755D175DD; Tue, 22 May 2012 11:02:50 +0000 (UTC) MIME-Version: 1.0 Content-Disposition: inline Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" Date: Tue, 22 May 2012 11:02:50 +0000 From: "John Marino" Subject: CVS commit: pkgsrc/pkgtools/packagekit To: pkgsrc-changes@NetBSD.org Reply-To: marino@netbsd.org X-Mailer: log_accum Message-Id: <20120522110251.0755D175DD@cvs.netbsd.org> Sender: pkgsrc-changes-owner@NetBSD.org List-Id: pkgsrc-changes.NetBSD.org Precedence: bulk Module Name: pkgsrc Committed By: marino Date: Tue May 22 11:02:50 UTC 2012 Modified Files: pkgsrc/pkgtools/packagekit: Makefile distinfo Added Files: pkgsrc/pkgtools/packagekit/patches: patch-src_Makefile.in Removed Files: pkgsrc/pkgtools/packagekit: INSTALL Log Message: pkgtools/packagekit: Fix permissions and directory creation This package would fail when a non-privileged user would try to install it. That's because it was trying to create /usr/pkg/var/log directory. Another directory was getting installed into DESTDIR but no further. Added patch: 1) Removes attempt of Makefile to install ${VARBASE}/log 2) ..and directory ${VARBASE}/cache/PackageKit/downloads creation 3) ..which fixes non-root installations The INSTALL file was removed, instead of installing directories in ${VARBASE}, it was installing directories at /. The functionality was replaced with OWN_DIRS+= in the Makefile and verified to work as expected. To generate a diff of this commit: cvs rdiff -u -r1.1.1.1 -r0 pkgsrc/pkgtools/packagekit/INSTALL cvs rdiff -u -r1.20 -r1.21 pkgsrc/pkgtools/packagekit/Makefile cvs rdiff -u -r1.3 -r1.4 pkgsrc/pkgtools/packagekit/distinfo cvs rdiff -u -r0 -r1.1 \ pkgsrc/pkgtools/packagekit/patches/patch-src_Makefile.in Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.