Created attachment 144367 [details] Patchfile for Makefile The port security/duo installs the following configuration files: - /usr/local/etc/login_duo.conf - /usr/local/etc/login_duo.conf.sample - /usr/local/etc/pam_duo.conf - /usr/local/etc/pam_duo.conf.sample Currently the permissions on these files are set to 700 and the owner is root:wheel. When you run the installed program login_duo it will fail as it expects the configuration file login_duo.conf to be owned by the user sshd only. The attached patch fixes two issues: 1. Preserve the owner of the files which were set in the staging/work directory correctly 2. Set the permissions to 600 which should be sufficient
Over to maintainer
Created attachment 144369 [details] Perms fix, libtool, no USE_GCC The real fix should be in the pkg-plist. All the Makefile needs to do is install the sample config. The @sample directive will take care of copying .conf.sample to .conf. The @mode and @owner directives will set the permissions and owner, respectively. Since wheel is the default group, there is no reason to set it. While I was looking at this, I noticed an unneeded patch (patch-lib__Makefile.am) it does nothing since automake isn't being run on it. patch-lib__Makefile.in just fixes the pkgconfig path...that can just be done with USES=pathfix. I also added USES=libtool to drop the .la files and removed the docs install hack because that is taken care of by staging. Finally, this seems to build fine with clang, so the USE_GCC can be dropped. https://redports.org/buildarchive/20140703131500-14885/
jhale@ patch approved.
Committed my patch in r360401. Thanks!