Page MenuHomeFreeBSD

[PATCH] net/akhq: Clean up port
ClosedPublic

Authored by kiwi on Fri, Feb 7, 4:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 14, 12:32 AM
Unknown Object (File)
Thu, Feb 13, 9:42 AM
Unknown Object (File)
Wed, Feb 12, 8:06 PM
Unknown Object (File)
Wed, Feb 12, 1:18 AM
Unknown Object (File)
Fri, Feb 7, 7:50 PM
Unknown Object (File)
Fri, Feb 7, 4:46 PM
Unknown Object (File)
Fri, Feb 7, 4:46 PM
Subscribers

Details

Summary
net/akhq: Clean up port

In Makefile:
- Better COMMENT
- Use ${ETCDIR} when applicable
- Update PLIST_FILES
- Removed post-install part

In service file:
- Use %WWWOWN%%

In sample configuration file, bind to 127.0.0.1 instead of 0.0.0.0

Sponsored by:   Klara, Inc.
Approved by:    0mp (mentor)

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kiwi requested review of this revision.Fri, Feb 7, 4:21 PM
kiwi created this revision.

Fix PLIST_SUB add SUB_LIST as well

This revision is now accepted and ready to land.Fri, Feb 7, 4:37 PM
This revision was automatically updated to reflect the committed changes.

Additionally why was this committed without maintainer consent (or maintainer timeout)?

Additionally why was this committed without maintainer consent (or maintainer timeout)?

Maintainer asked Klara to push this port (see previous commits), my mentor, 0mp, asked me to fix some things in the port that maintainer didn't do.

In that case it should be referenced in commit message and from what I can tell there's no request anywhere? There's only one commit prior to this one regarding this port so I'm not sure what you're referring to.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=274149

Anyway, having a quick look...

  • Port is still handling the sample file incorrectly
  • NO_EXTRACT is oddly placed and I would argue that CONFIG_FILES also is
  • DIST and PORTVERSION are mixed for no apparent reason
  • Preferably also use PREFIX rather than LOCALBASE in .in file
grep -lir LOCALBASE --include "*.in" /usr/ports/ |wc -l
     403
grep -lir PREFIX --include "*.in" /usr/ports/ | wc -l
    2745