Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Dec 2016 11:10:04 -0600
From:      John Marino <freebsd.contact@marino.st>
To:        Rene Ladan <rene@FreeBSD.org>, ports-committers@freebsd.org, svn-ports-all@freebsd.org, Gleb Smirnoff <glebius@FreeBSD.org>
Subject:   Re: svn commit: r428233 - head/Mk/Uses [kmod.mk]
Message-ID:  <f6b1e2c2-57f6-3e13-47e5-efb8b5167761@marino.st>
In-Reply-To: <5cdc9a0f-4b88-ca81-7c0c-f5e4fc2f8a93@marino.st>
References:  <201612091847.uB9Il9uR007592@repo.freebsd.org> <5cdc9a0f-4b88-ca81-7c0c-f5e4fc2f8a93@marino.st>

next in thread | previous in thread | raw e-mail | index | archive | help
On 12/10/2016 10:42, John Marino wrote:
> On 12/9/2016 12:47, Rene Ladan wrote:
>> Author: rene
>> Date: Fri Dec  9 18:47:09 2016
>> New Revision: 428233
>> URL: https://svnweb.freebsd.org/changeset/ports/428233
>>
>> Log:
>>   Uses/kmod.mk: add support for debug versions of kernel modules.
>>
>>   Submitted by:    glebius via private mail
>>
>> Modified:
>>   head/Mk/Uses/kmod.mk
>>
>> Modified: head/Mk/Uses/kmod.mk
>> ==============================================================================
>>
>> --- head/Mk/Uses/kmod.mk    Fri Dec  9 17:50:43 2016    (r428232)
>> +++ head/Mk/Uses/kmod.mk    Fri Dec  9 18:47:09 2016    (r428233)
>> @@ -34,13 +34,19 @@ MAKE_ENV+=    KMODDIR="${KMODDIR}" SYSDIR="
>>  PLIST_FILES+=    "@kld ${KMODDIR}"
>>
>>  STRIP_CMD+=    --strip-debug # do not strip kernel symbols
>> +KERN_DEBUGDIR?=    ${DEBUGDIR}
>>  .endif
>>
>>  .if defined(_POSTMKINCLUDED) && !defined(_INCLUDE_USES_KMOD_POST_MK)
>>  _INCLUDE_USES_KMOD_POST_MK=    yes
>>
>>  _USES_install+=    290:${STAGEDIR}${KMODDIR}
>> +_USES_install+=    291:${STAGEDIR}${KERN_DEBUGDIR}${KMODDIR}
>> +
>>  ${STAGEDIR}${KMODDIR}:
>>      @${MKDIR} ${.TARGET}
>>
>> +${STAGEDIR}${KERN_DEBUGDIR}${KMODDIR}:
>> +    @${MKDIR} ${.TARGET}
>> +
>>  .endif
>>
>
>
> After this commit, I am seeing:
> make: "/xports/Mk/Uses/kmod.mk" line 50: warning: duplicate script for
> target "/construction/x11-drivers/drm-kmod/stage/boot/modules" ignored
> make: "/xports/Mk/Uses/kmod.mk" line 47: warning: using previous script
> for "/construction/x11-drivers/drm-kmod/stage/boot/modules" defined here
>
> The cause is that if KERN_DEBUGDIR is not defined, the targets ending in
> ${KMODDIR} are identical.  There needs to be a guard checking for the
> definition of KERN_DEBUGDIR.
>
> Thanks,
> John

Resending -- I was having DNS issues that caused my mail not to be 
delivered to @FreeBSD addresses for a while.

---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?f6b1e2c2-57f6-3e13-47e5-efb8b5167761>