svn commit: r350135 - head/share/mk

[Guido Falsi]

On 26/07/19 05:10, Emmanuel Vadot wrote:
> On Thu, 25 Jul 2019 00:17:31 +0200
> Guido Falsi <madpilot at FreeBSD.org> wrote:
> 
>> On 25/07/19 00:14, Guido Falsi wrote:
>>> On 19/07/19 17:11, Emmanuel Vadot wrote:
>>>> Author: manu
>>>> Date: Fri Jul 19 15:11:32 2019
>>>> New Revision: 350135
>>>> URL: https://svnweb.freebsd.org/changeset/base/350135
>>>>
>>>> Log:
>>>>   bsd.confs.mk: Test the correct value for the destination package
>>>>   
>>>>   Reviewed by:	bapt, gjb
>>>>   Differential Revision:	https://reviews.freebsd.org/D20961
>>>>
>>>
>>> Hello,
>>>
>>> I don't know the actual workings of this file and have not checked it in
>>> depth, but this change is causing "make package" to not package
>>> /etc/rc.d/sshd (and others, all the ones being defined in
>>> ${group}PACKAGE variables), which are then missing when upgrading machines.
>>>
>>> Reverting this change makes packaging things work again. I think this
>>> change is actually wrong and the previous version was correct.
>>>
> 
>  /etc/rc.d/sshd is in the FreeBSD-sshd package, the other ones are also
> in their own package now, I'll check them all to see if that make sense.

It makes perfect sense for that file to be in the FreeBSD-sshd package
and that was my first guess when looking for the missing file, but in my
case it was not present in that package.

By reverting your change it appears to end up in the FreeBSD-runtime
package.

> 
>> After sending this message I also noticed, it causes the spurious
>> "FreeBSD-zfs" package to be generated again.
> 
>  Not so spurious, it now contains the rc.d zfs related files.
>  I think that I'll remove it anyway, it doesn't make sense.

It does, but, again reverting your change, those files end up in the
runtime package.

-- 
Guido Falsi <madpilot at FreeBSD.org>