[Bug 216005] New port: deskutils/mytetra: Smart cross-platform manager for information collecting

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216005

Tobias Kortkamp <tobik at freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tobik at freebsd.org

--- Comment #1 from Tobias Kortkamp <tobik at freebsd.org> ---
It has been over a year unfortunately.  Can you confirm that you still
want this in the ports tree?  The port looks fine to me at first
glance.

> 1. It uses ftime(3), which is deprecated and requires -lcompat.
>   Hope that's still going to be supported on 11 and 12.

It's still available, but is it not possible to replace it with
gettimeofday(2)?  Looking at [1] it seems that this would be
relatively trivial to do.

> 2. It tried to install some stuff to /usr/share. Is it okay to hard code
>   PREFIX = /usr/local in the .pro file?

In general, no.  That would mean that users cannot install it into a
different prefix (e.g. /opt).  But you could use a REINPLACE_CMD (i.e
sed) to replace it with ${PREFIX} in post-patch.  For example misc/owrep
does this.

[1] https://github.com/freebsd/freebsd/blob/master/lib/libcompat/4.1/ftime.c

-- 
You are receiving this mail because:
You are the assignee for the bug.