tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: review request for mod_pubcookie package

On Mon, Jul 6, 2009 at 6:21 PM, John Nemeth<> 
> On Nov 26, 12:33pm, Tracy Di Marco White wrote:
> }
> } I'd started putting together a mod_pubcookie package ages ago, and
> } finally spent enough time on it to finish it. I haven't fiddled with
> } pkgsrc in a while, so I'd appreciate some review of it.
> } has a pass at the
> } package that untars into www/ap2-pubcookie.
>     A few items:
> - no LICENSE specified

I added that, and updated the package.

> - no DESTDIR support
>  - I don't know if DESTDIR makes sense for apache modules

I wasn't sure what to do here.

> - in patch-ab, are there variables that could used in place of
>  @APACHE_PREFIX@/share/httpd and -I@APACHE_PREFIX@/include/httpd

The configure script seems to use those to modify that, so
I abused them. I really wasn't sure how to do apache modules, so I
abused the ldap one a long time ago when I started this one.

> - various pkglint warnings
> You should install pkgtools/pkglint.  Then after writing/updating a
> package, you should run 'pkglint -Wall -Call -i'.  If you add '-e', it
> will give an explanation of anything it finds.

Wonderful, thanks.  Jason reminded me of pkglint after I'd sent my
mail, but I appreciate the options to give it as well.


Home | Main Index | Thread Index | Old Index