Date: Thu, 16 Mar 2006 18:16:25 +0100 From: Alexander Leidinger <Alexander@Leidinger.net> To: Boris Samorodov <bsam@ipt.ru> Cc: freebsd-emulation@freebsd.org Subject: Re: [patch] for bsd.linux-rpm.mk (add auto sources fetching) Message-ID: <20060316181625.41102cfd@Magellan.Leidinger.net> In-Reply-To: <39907788@serv3.int.kfs.ru> References: <39907788@serv3.int.kfs.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
Am Thu, 16 Mar 2006 18:25:23 +0300
schrieb Boris Samorodov <bsam@ipt.ru>:
> +. if defined(PACKAGE_BUILDING) && ${AUTO_LINUX_SRC} == "yes"
> +DISTFILES+= ${DISTNAME}${SRC_SUFX}
> +MASTER_SITE_SUBDIR+= 3/SRPMS
> +ALWAYS_KEEP_DISTFILES= yes
> . endif
You are assuming this will only be used for some specific directory
layouts without the possibility to override this. I suggest to
introduce an overridable variable... maybe MASTER_SITE_SRC_SUBDIR or
whatever which will be added to MASTER_SITE_SUBDIR then.
Maybe you can change the patch to have a look at a SRC_DISTFILES
variable and add this content to the DISTFILES in the PACKAGE_BUILDING
case instead of hardcoding one file.
Bye,
Alexander.
--
Where do you think you're going today?
http://www.Leidinger.net Alexander @ Leidinger.net
GPG fingerprint = C518 BC70 E67F 143F BE91 3365 79E2 9C60 B006 3FE7
WL http://www.amazon.de/exec/obidos/registry/1FZ4DTHQE9PQ8/ref=wl_em_to/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060316181625.41102cfd>