Differential D10021 Diff 27125 graphics/drm-next-kmod/Makefile

Changeset View

Standalone View

graphics/drm-next-kmod/Makefile

Property	Old Value	New Value
svn:eol-style	null	native \ No newline at end of property
svn:keywords	null	FreeBSD=%H \ No newline at end of property
svn:mime-type	null	text/plain \ No newline at end of property

				# Created by: Johannes Dieterich <jmd@FreeBSD.org>
				# $FreeBSD$

				PORTNAME= drm-next-kmod
				PORTVERSION= g20170404
				CATEGORIES= graphics

				MAINTAINER= jmd@FreeBSD.org
				COMMENT= DRM modules for the linuxkpi-based KMS components

				LICENSE= MIT GPLv2
				LICENSE_COMB= multi
				markjUnsubmitted Done Inline Actions Why MIT? What does it mean to have two licenses specified here? Most of the code is dual BSD/GPLv2-licensed, but some of it is just GPLv2. markj: Why MIT? What does it mean to have two licenses specified here? Most of the code is dual…
				markjUnsubmitted Done Inline Actions Sorry, MIT is indeed right. Again, though, not all of the code is dual-licensed. markj: Sorry, MIT is indeed right. Again, though, not all of the code is dual-licensed.
				jmdAuthorUnsubmitted Done Inline Actions note that it uses multi (i.e., code base has multiple licenses), which I think is fitting. Thanks for the heads-up that there are also BSD2CLAUSE pieces. Added that license. jmd: note that it uses multi (i.e., code base has multiple licenses), which I think is fitting.

				RUN_DEPENDS= gpu-firmware-kmod>=g20170403:graphics/gpu-firmware-kmod

				ONLY_FOR_ARCHS= amd64
				ONLY_FOR_ARCHS_REASON= the new KMS components are only supported on amd64

				USE_GITHUB= yes
				GH_ACCOUNT= FreeBSDDesktop
				GH_PROJECT= kms-drm
				GH_TAGNAME= 0d5ebc0

				USES= kmod

				.include <bsd.port.pre.mk>
				matUnsubmitted Done Inline Actions That should probably be: IGNORE_FreeBSD_10= not supported on 11.x or older, no kernel support IGNORE_FreeBSD_11= not supported on 11.x or older, no kernel support That way you do not need to include bsd.port.options.mk. mat: That should probably be: ``` IGNORE_FreeBSD_10= not supported on 11.x or older, no kernel…
				jmdAuthorUnsubmitted Done Inline Actions The issue is that not all revisions of HEAD do even compile this port. It depends on changes to the linuxkpi that were only recently upstreamed. Hence the version check. What's your recommendation? jmd: The issue is that not all revisions of HEAD do even compile this port. It depends on changes to…

				jbeichUnsubmitted Done Inline Actions Maybe use `<bsd.port.options.mk>` instead. Nothing later depends on USE_* or USES defining special variables. jbeich: Maybe use `<bsd.port.options.mk>` instead. Nothing later depends on USE_* or USES defining…
				.if ${OPSYS} == FreeBSD && ${OSVERSION} < 1200019
				IGNORE= not supported on 11.x or older, no kernel support
				.endif

				.include <bsd.port.post.mk>