Page MenuHomeFreeBSD

net-mgmt/pmacct: Rework/fix plist logic
AbandonedPublic

Authored by otis on Jan 19 2020, 1:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 6:58 AM
Unknown Object (File)
Aug 24 2023, 1:58 PM
Unknown Object (File)
Jul 31 2023, 10:13 PM
Unknown Object (File)
Jul 31 2023, 6:34 AM
Unknown Object (File)
Mar 16 2023, 1:14 AM
Unknown Object (File)
Dec 31 2022, 2:23 AM

Details

Reviewers
pi
Summary

Fix substitution of SQLDOC in plist. Previously, SQLDOC was always overwritten to "@comment " by subsequent condition (except from the last one).

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 28850
Build 26856: arc lint + arc unit

Event Timeline

  • net-mgmt/pmacct: Fix the SQLDOC substitution logic
  • net-mgmt/pmacct: Use also pkgconfig to help database library detection
  • Remove obsolete/unused options
  • net-mgmt/pmacct: Update Makefile

you could get rid of most of the if-fery

net-mgmt/pmacct/Makefile
33–34

why not

MYSQL_USES=mysql
MYSQL_CONFIGURE_ENABLE=mysql
46–47
MLAYER2_CONFIGURE_ENABLE=...
  • net-mgmt/pmacct: Rework port options a bit.
  • Rework options processing logic

you could get rid of most of the if-fery

net-mgmt/pmacct/Makefile
46–47

WIthout leading M.

pi requested changes to this revision.Feb 22 2020, 4:53 PM

I've tried to apply the diff, but it failed. Can you update the patch to the most recent version of the Makefile and
test, wether it applies ?

This revision now requires changes to proceed.Feb 22 2020, 4:53 PM
otis marked an inline comment as done.

Fix plist logic and pgsql module

Please check the diff again. I updated the port to 1.7.4p1 yesterday, and this diff does not look similar to your first version of the patch ?

New diff revision will be created.