Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 16 Aug 2014 06:51:49 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 189257] [NEW PORT] databases/percona56-xtradb-cluster
Message-ID:  <bug-189257-13-DW129juN2g@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-189257-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-189257-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=189257

--- Comment #16 from John Marino <marino@FreeBSD.org> ---
Scott, this looks real good.

I have only two concerns.

1) on this line:
Xpost-install:
X    @${MKDIR} ${STAGEDIR}/var/db/mysql
X    @${RM} -rf ${STAGEDIR}${PREFIX}/xinetd.d

The "@" on the RM command masks the file removal.  It is against policy to mask
installation steps (with mkdir being the exception so the first line is ok)


2) you have "flo@freebsd.org" being listed as the maintainer.
This is unusual.  Did flo agree to this?  I see on the description 
"I've been talking to flo who's the maintainer of the percona database software
amongst others, he told me to create a PR to get this port committed as he
didn't have time."

That doesn't mean he agreed to maintain the port.

So either somebody else needs to be listed as the maintainer, or flo needs to
explicitly approve this PR and agree to the maintainer.  Do you want me add him
to the PR or should somebody else be the maintainer?

-- 
You are receiving this mail because:
You are the assignee for the bug.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-189257-13-DW129juN2g>