Hi Jasper, hi Steven, hi Wijnand,

Jasper Lievisse Adriaanse wrote on Sun, Apr 29, 2007 at 12:04:40PM +0200:
> i made some more modifications:
> - SUBST_VARS isn't needed

I fear you broke it.  The bsd.port.mk fake target does not work as it
should and the post-install target fails for me.  The reason is that
the jabber configuration files now get installed to ${WRKINST}/etc/
instead of ${WRKINST}/etc/jabberd14, thus crashing the command
  mv jabber.xml jabber.mysql.xml
during post-install.  I dimly remember some trouble i had in the
very beginning of setting up the port, telling ${WRKSRC}/Makefile
about ${SYSCONFDIR}/jabberd14.  In your version of the port, this
problem is back: $(sysconfdir) in ${WRKSRC}/Makefile again is /etc
and not /etc/jabberd14 as it should.

Besides, the port Makefile contains 
  J_CONFDIR= ${SYSCONFDIR}/jabberd14
and related instrumentation to put jabber config files in there.
Yet, ${PKGDIR}/PLIST has
  @sample ${SYSCONFDIR}/jabber.xml   # and two similar lines
This does not look consistent.

There may be an additional problem with ${PKGDIR}/PLIST.  It still
contains
  @newgroup _jabberd14:585
  @newuser _jabberd14:585:585:daemon:Jabber Server: [...]
Do not forget to adjust the UID/GID at commit time.

> - don't hardcode paths in README.OpenBSD
> - better MESSAGE

This looks good.

> with this diff, it looks ok to me.
> steven, ok with you too?
> http://humppa.nl/~jasper/jabberd14.tgz
 
I hate saying that, but seeing all those commit messages saying 
"Use xenocara expat" and having LIB_DEPENDS+= expat::textproc/expat
in this port, i think i ought to update the port to use xenocara
expat, too.
 
I will look into these points as soon as i find time, hopefully
this evening.

Thanks for all your help with the port,
  Ingo

Reply via email to