You are right. I just review the change. My proposed change was different: https://marc.info/?l=openbsd-ports&m=153559615313702
Already sent a mail to rsadowski@ to consult about this. Cheers. Elias. 2018-09-06 13:03 GMT-03:00 Antoine Jacoutot <ajacou...@bsdfrog.org>: > On Thu, Sep 06, 2018 at 12:58:36PM -0300, Elias M. Mariani wrote: >> A fix for this is already in the CVS: >> https://marc.info/?l=openbsd-ports-cvs&m=153621035406823 > > That's not a fix. > The @sample line was removed altogether instead of sampling the needed > directory. > >> Cheers. >> Elias. >> >> 2018-09-06 3:55 GMT-03:00 Marc Espie <es...@nerim.net>: >> > On Thu, Sep 06, 2018 at 06:56:35AM +0200, Solene Rapenne wrote: >> >> Hello, I installed umbrello which brings kservice as a dependency and an >> >> error >> >> showed during the install process. >> >> >> >> solene@t480 /usr/ports $ doas pkg_add -vi umbrello >> >> Update candidates: quirks-3.7 -> quirks-3.7 >> >> quirks-3.7 signed on 2018-09-04T22:05:13Z >> >> File /etc/xdg/menus/applications-kde5.menu could not be installed: >> >> No such file or directory >> >> umbrello-17.12.3p1:kservice-5.46.0: ok >> >> >> >> The file /etc/xdg/menus/applications-kde5.menu is present in the PLIST of >> >> devel/kf5/kservice like this: >> >> >> >> share/examples/xdg/menus/applications-kde5.menu >> >> @sample ${SYSCONFDIR}/xdg/menus/applications-kde5.menu >> >> >> >> I built the package, checked the content of the package tarball and it >> >> doesn't >> >> have a /etc folder in it. Why? >> > >> > Probably because at some point there was a runtime dependency on stuff >> > that already had >> > @sample ${SYSCONFDIR}/xdg/menus/ >> > in it, but that dependency is no longer listed, but the plist hasn't been >> > updated ? >> > >> > > -- > Antoine
