From: Bryan Vyhmeister <bryan@bsdjournal.net>
Subject: Re: [NEW] comms/direwolf
To: ports@openbsd.org
Date: Wed, 23 Oct 2024 11:05:43 -0700

On Wed, Oct 23, 2024 at 11:27:19AM +0100, Stuart Henderson wrote:
> On 2024/10/22 21:57, Bryan Vyhmeister wrote:
> > On Tue, Oct 22, 2024 at 01:41:52PM -0700, Bryan Vyhmeister wrote:
> > > I came across a half-finished and out-of-date attempt at direwolf in
> > > openbsd-wip on github from five years ago. I updated the port and it is
> > > building fine now. Is anyone able to test some of the features of
> > > direwolf and see if this works correctly? I don't have any hardware to
> > > test this out.
> > 
> > I made some updates and fixes to the port but a few things remain. If I
> > run 'portcheck -N' I get the error that:
> > 
> > manual pages should go under ${PREFIX}/man/ rather than under
> > ${PREFIX}/share/man/
> > 
> > I have tried several different ways of fixing that and it stil appears.
> > THe latest I tried was adding INSTALL_MAN_DIR to FAKE_FLAGS as you can
> 
> remove FAKE_FLAGS (you don't need DESTDIR either) and fix the path in
> CMakeLists.txt. cmake doesn't allow overrides unless cmakelists is setup
> to permit it.

The FAKE_FLAGS and so forth were remnants from before the switch to
cmake for direwolf. Thanks for that info about cmake. I was not aware of
that. I added a patch for CMakeLists.txt to fix those issues.

> drop the trailing . in COMMENT too.

Done.

> why does it have @tag update-desktop-database and desktop-file-utils
> dep?

This was also there before I started updating the port. I don't believe
it is needed so removed it. I also elimitated the bash dependency since
the script that bash is needed for is very simple and should work fine
with our shell. I noticed a dependency on espeak that I had not noticed
before. How does this look? Tarball attached. Thank you.

Bryan