From owner-freebsd-emulation@FreeBSD.ORG Mon Aug 7 00:04:20 2006 Return-Path: X-Original-To: freebsd-emulation@FreeBSD.org Delivered-To: freebsd-emulation@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8BBB416A4DF; Mon, 7 Aug 2006 00:04:20 +0000 (UTC) (envelope-from bsam@bsam.ru) Received: from mail.kuban.ru (mail.kuban.ru [62.183.66.246]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0787C43D4C; Mon, 7 Aug 2006 00:04:19 +0000 (GMT) (envelope-from bsam@bsam.ru) Received: from bsam.ru ([83.239.48.144]) by mail.kuban.ru (8.9.1/8.9.1) with ESMTP id k7703wNQ016765; Mon, 7 Aug 2006 04:04:08 +0400 (MSD) Received: from bsam by bsam.ru with local (Exim 4.62 (FreeBSD)) (envelope-from ) id 1G9saS-0006fT-9s; Mon, 07 Aug 2006 04:03:16 +0400 To: Jose Alonso Cardenas Marquez References: <200608041438.k74Ec3Wr096494@repoman.freebsd.org> <20060804195623.564c8a6e@Magellan.Leidinger.net> <20060804203725.efb1e8df.acm@FreeBSD.org> <36280092@bs2.sp34.ru> <95145875@bsam.ru> <20060806174543.62b15d3d.acm@FreeBSD.org> From: Boris Samorodov Date: Mon, 07 Aug 2006 04:03:16 +0400 In-Reply-To: <20060806174543.62b15d3d.acm@FreeBSD.org> (Jose Alonso Cardenas Marquez's message of "Sun, 6 Aug 2006 17:45:43 -0500") Message-ID: <82654251@bsam.ru> User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: Boris Samorodov Cc: Alexander Leidinger , freebsd-emulation@FreeBSD.org, garga@FreeBSD.org Subject: Re: cvs commit: ports/ftp Makefile ports/ftp/linux-curl Makefile distinfo.i386 X-BeenThere: freebsd-emulation@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Development of Emulators of other operating systems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Aug 2006 00:04:20 -0000 On Sun, 6 Aug 2006 17:45:43 -0500 Jose Alonso Cardenas Marquez wrote: > On Sun, 06 Aug 2006 15:15:08 +0400 > Boris Samorodov wrote: > > Here are some more comments: > > > > 1. linux-curl > > > > - portrevision is not bumpted while the package is definitely changed > > (at list the binary is branded); (this question is not only > > for this port only) > > > > - can you descrtibe why did you decide to use @dirrmtry and not @dirrm > > to usr/share/doc/curl-7.13.1 directory (I speek about only the > > document directories here)? (this question is not only for this port > > only) > I fixed linux-curl/pkg-plist file and i bumped PORTVERSION on > linux-curl, linux-openssl and linux-e2fsprogs. Great! > > 2. security/linux-krb5-libs > > > > Netchild and I agreed to install config files to linux etc directory > > after renaming them to *.dist (ex. etc/nsswitch.conf at linux_base-fc4 > > port). > > > > That gives a user a good example and don't override the real > > config (if any). And how to deal with real config file (i.e. install a > > symlink while installing or remove it while deletting the port) you > > may see at linux_base_fc4 pkg-install and pkg-plist files. > > > > BTW, maybe it's a right thing to create a symlink to the real FreeBSD > > kerberos configuration file (if any). > I added pkg-install file and i made some changes on pkg-plist and > Makefile files. Also i made a patch file for integrate linux-krb5-libs Why did you use @dirrmtry at security/linux-krb5-libs/pkg-plist? ;-) Actually, it seems to me that when in doubt you prefer to use @dirrmty macro. I'd say that it should be quite the opposite. This macro should be used only when you are sure that those directories are shared between ports and there is no (so to say) "master port" which deals with those directories. My opinion is to use @dirrmtry with care... > into linux_base-fc4. you can see all these changes at: > http://hellfire.no-ip.org/ports/ > I tested all my linux-ports on TB without problems. Please, can you > take a look of linux_base-fc4.diff file? Good work! And here are some comments: - you didn't show a diff for the distinfo; - portrevision is not bumpted; ;-) - at pkg-install script only empty directories are created, so there is no need to create a usr/kerberos one; - assume that you didn't check linux_base-fc4.diff file on TB, because I can't manage to find a line with "@dirrm usr/kerberos" at the plist file. Other than that seems OK. May be Alexander has something to add. Thanks! WBR -- Boris Samorodov (bsam) Research Engineer, http://www.ipt.ru Telephone & Internet SP FreeBSD committer, http://www.FreeBSD.org The Power To Serve