On Mon, Jun 06, 2022 at 10:39:52AM +0100, Stuart Henderson wrote:
> On 2022/06/06 11:12, Omar Polo wrote:
> > some tweaks are needed:
> 
> I agree with all your tweaks. Generally ok to import (as long as
> MAINTAINER really is correct). The pledge/unveil stuff mentioned in
> README does not really nice though, this is not something users should
> have to fiddle with for basic operation, I would drop that section from
> the readme and leave it for advanced users to find for themselves if
> they want to be responsible for keeping it up to date/fixing things.

I've applied tweaks from Omar, added epicrc.sample to files/ and install
it to ${PREFIX}/share/epic5/script instead of embedding in README, also
removed pledge/unveil example from this file as Stuart suggested, left
only a note, that further information about the syscalls usage is in
UPDATES document.

Regarding maintainer - Joey will be the "official" one, so the variable
is set correctly, but we both will take care of this port.

Appreciated for the review.

Attachment: epic5.tar.gz
Description: application/tar-gz

Reply via email to