On 2010/10/19 18:20, matteo filippetto wrote:
> Maybe remember to specify that a chown www:ww is need to make it working.
the user shouldn't have to do this, the package should set appropriate
ownership and permissions already (including things like writable temp
directories where appropriate). so
@chown -R ${SHAREOWN}:${SHAREGRP} ${PREFIX}/achievo
-> use www:www
neater to do this in Makefile to cover general cases and then maybe
some @owner/@mode in PLIST for any special cases - temp dirs etc.
other points:
COMMENT
-> start lowercase, no trailing .
PERMIT_PACKAGE_CDROM = Yes
PERMIT_PACKAGE_FTP = Yes
PERMIT_DISTFILES_CDROM = Yes
PERMIT_DISTFILES_FTP = Yes
-> much easier on the eyes if these line up
please zap blank lines top/bottom in DESCR.
