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>