### 08:51:05 ### 985 ### 2 ### install /usr/ports/mail/postgrey from /freebsd/Packages/postgrey-1.34_3.tbz ---> Starting install script: ---> Creating Postgrey database directory (/var/db/postgrey) chown: postgrey: illegal group name pkg_add: install script returned error status
Maintainer of mail/postgrey, Please note that PR ports/166943 has just been submitted. If it contains a patch for an upgrade, an enhancement or a bug fix you agree on, reply to this email stating that you approve the patch and a committer will take care of it. The full text of the PR can be found at: http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/166943 -- Edwin Groothuis via the GNATS Auto Assign Tool edwin@FreeBSD.org
State Changed From-To: open->feedback Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
It looks like the step that creates the database directory happens before the user and group creation step when installing from package. The attached patch moves that step to the post-install stage.
Responsible Changed From-To: freebsd-ports-bugs->crees I'll take it.
Hi Darren, Thanks for the quick fix! I've also changed the perl dependencies to packages; SITE_PERL is defined in bsd.perl.mk, and for various reasons we're trying to make the ports tree work without bsd.perl.mk if possible. Is the patch at [1] OK? Chris [1] http://www.bayofrum.net/~crees/patches/postgrey-fix-pkginstall.diff
> Hi Darren, > > Thanks for the quick fix! > > I've also changed the perl dependencies to packages; SITE_PERL is > defined in bsd.perl.mk, and for various reasons we're trying to make > the ports tree work without bsd.perl.mk if possible. > > Is the patch at [1] OK? > > Chris > > [1] > http://www.bayofrum.net/~crees/patches/postgrey-fix-pkginstall.diff Why are there ">=0" version specifications for the dependencies? Isn't that the same as saying any version, which is what you get without a version spec? Note: GNATS does not forward follow-ups, so please make sure you keep ports.maintainer@evilphi.com in the CC list.
On 15 April 2012 23:29, Darren Pilgrim <ports.maintainer@evilphi.com> wrote= : >> Hi Darren, >> >> Thanks for the quick fix! >> >> I've also changed the perl dependencies to packages; SITE_PERL is >> defined in bsd.perl.mk, and for various reasons we're trying to make >> the ports tree work without bsd.perl.mk if possible. >> >> Is the patch at [1] OK? >> >> Chris >> >> [1] >> http://www.bayofrum.net/~crees/patches/postgrey-fix-pkginstall.diff > > > Why are there ">=3D0" version specifications for the dependencies? =A0Isn= 't that > the same as saying any version, which is what you get without a version > spec? Without a version spec, it depends on a binary (i.e. /usr/local/bin/p5-bluurgh). The >=3D forces a package dependency. Of course, if you know the actual versions for the deps, I'll happily add those :) Chris
> Of course, if you know the actual versions for the deps, I'll > happily add those :) The only minimum version requirements are for Perl itself (5.6.0) and BerkleyDB (4.1). Perl 5.6.0 isn't even in the tree anymore and p5-BerkeleyDB has USE_BDB=47+. Chris Rees' patch approved.
crees 2012-04-18 07:54:30 UTC FreeBSD ports repository Modified files: mail/postgrey Makefile mail/postgrey/files pkg-install.in Log: - Fix package installation; pre-install part of script depends on postgrey username existing. PR: ports/166943 Submitted by: Darren Pilgrim <ports.maintainer@evilphi.com> (maintainer) - While here, use packages for perl dependencies to avoid requiring bsd.perl.mk Approved by: Darren Pilgrim <ports.maintainer@evilphi.com> (maintainer) Revision Changes Path 1.34 +9 -7 ports/mail/postgrey/Makefile 1.6 +6 -5 ports/mail/postgrey/files/pkg-install.in _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
State Changed From-To: feedback->closed Committed. Thanks!