Created attachment 165776 [details] remove PLIST_DIRSTRY
Reporter is committer, changes are blanket, implicit approval. Assign accordingly, keep maintainer CC'd
Looks sane to me, please go ahead. You also need to bump PORTREVISION in the relevant ports since it changes the package manifest.
A commit references this bug: Author: pawel Date: Wed Jan 20 16:18:03 UTC 2016 New revision: 406778 URL: https://svnweb.freebsd.org/changeset/ports/406778 Log: Remove deprecated PLIST_DIRSTRY PR: 206378 Submitted by: myself Approved by: rakuco Changes: head/devel/qt5/Makefile.sqldrivers
The change is incomplete, you have to bump PORTREVISION in the affected ports.
Disagree, plist of slave ports don't change and pkg is smart enough now to remove directories you install files to on deinstall so I don't see any reason to PORTREVISION bump here. In last 5 weeks I've converted 50+ ports with similar changes and did not get any comments about bumps.
It looks like I've interpreted the "PORTREVISION must be increased each time a change is made to the port that changes the generated package in any way" too strictly, so closing again. Still, you could have said you disagreed with comment #2 before.