On Mon, Mar 19, 2012 at 10:32:44AM -0400, Michael Scheidell wrote:
> 
> 
> On 3/19/12 10:30 AM, Wesley Shields wrote:
> > Isn't it better to use ${INSTALL_DATA} here?
> >
> > -- WXS
> open a pr, let maintainer approve it.

As the person who committed this it is your responsibility to see that
issues raised are properly fixed. If you feel like the maintainer should
be brought in for such a simple change that is your decision to make,
but I shouldn't have to open a PR to address this.

> Id also like to see examples wrapper in a .if/endif and pkg-plist 
> prefixed with %%PORTEXAMPLES%%%EXAMPLES%% .

Would be nice to have also, but not my point. Using ${INSTALL_DATA} is
the right thing to do because it sets the owner and mode properly.

-- WXS
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscr...@freebsd.org"

Reply via email to