From owner-freebsd-emulation@FreeBSD.ORG Wed Mar 4 09:23:37 2009 Return-Path: Delivered-To: freebsd-emulation@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3A4C01065673 for ; Wed, 4 Mar 2009 09:23:37 +0000 (UTC) (envelope-from bsam@ipt.ru) Received: from services.ipt.ru (services.ipt.ru [194.62.233.110]) by mx1.freebsd.org (Postfix) with ESMTP id E87438FC0A for ; Wed, 4 Mar 2009 09:23:36 +0000 (UTC) (envelope-from bsam@ipt.ru) Received: from bb.ipt.ru ([194.62.233.89]) by services.ipt.ru with esmtp (Exim 4.54 (FreeBSD)) id 1LenKB-000AVf-Mj; Wed, 04 Mar 2009 12:23:35 +0300 To: Alexander Leidinger References: <14905754@bb.ipt.ru> <20090304093137.10733oaty524f0is@webmail.leidinger.net> From: Boris Samorodov Date: Wed, 04 Mar 2009 12:23:35 +0300 In-Reply-To: <20090304093137.10733oaty524f0is@webmail.leidinger.net> (Alexander Leidinger's message of "Wed\, 04 Mar 2009 09\:31\:37 +0100") Message-ID: <11370520@bb.ipt.ru> User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.3 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: freebsd-emulation@freebsd.org Subject: Re: [patch] Mk/bsd.linux-rpm.mk: introduce some linux-f8 infrastructure 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: Wed, 04 Mar 2009 09:23:37 -0000 Alexander Leidinger writes: > Quoting Boris Samorodov (from Tue, 03 Mar 2009 18:55:01 +0300): > >> I'm going to commit an attached patch. This patch is a noop for >> current ports structure (no need to ask an approval from portmgr) >> but is needed for new linux-f8 ports. > > While I agree with you... does portmgr agree with you too? Sometimes > they have different opinions than committers about what should be > tested in an exp-build. Agreed, I'll contact portmgr with this patch too. >> There are two points (both taken from emulators/linux_base-f8): >> . define LINUX_OSRELEASE variable (linux-f8 packages should be >> build with compat.linux.osrelease=2.6.16); > > Why do you use ECHO_CMD? Is there some whitespace which needs to be deleted? Can't recall for sure but I've got some unwanted results while doing some experiments (was it an output to stderr if no linuxulator kldloaded?). >> . define MASTER_SITE_SUBDIR and MASTER_SITE_SRC_SUBDIR when >> LINUX_DIST_VER=8 (no such ports ATM). > > Why is there ifndef MASTER_SITES (only in the context of your patch, This ifndef was introduced by sat@ at Rev 1.9. I'm not aware of the real need. > so not really related)? Is there a difference for "?=" between an > empty MASTER_SITES and an undefined one? Yea, after ifndef introduction "?=" lost it's purpose. Alexander, I fully agree with your concerns, I'm aware of them. But for now I try to change as little as it is possible so to face less problems. Since new infrastructure and linux-f8 port are introduced work should go further. ;-) Thanks for your feedback. WBR -- Boris Samorodov (bsam) Research Engineer, http://www.ipt.ru Telephone & Internet SP FreeBSD Committer, http://www.FreeBSD.org The Power To Serve