On Thu, Sep 06, 2018 at 01:09:52PM -0300, Elias M. Mariani wrote: > 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.
Yup better :-) Are we sure there's a dependency installing SYSCONFDIR/xdg/ ? make plist should tell us anyway. Thanks. > > 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 > -- Antoine