cvs commit: ports/www/p5-HTML-Defaultify Makefile

Takefu takefu at
Wed Jun 15 00:56:16 UTC 2011


2)a) It agrees with this change.
Other reviews listed above is not a negative.
Not too positive.

Andrej Zverev さんは書きました (2011/06/15 1:44):

> Hello.
> I would like to suggest following patch.
> 1) More preferable to depend on CPAN infrastructure less on private author
> sites.
> 2) Fix pkg-descr:
>     a) Description text from module
>     b) WWW: to CPAN (we prefer this I hope and it contrain full module
> documentaion).
> 3) Some Makefile rearrangement.
> Patch:

--- p5-HTML-Defaultify.patch begins here ---
diff -ruN /usr/ports/www/p5-HTML-Defaultify/Makefile www/p5-HTML-Defaultify/Makefile
--- /usr/ports/www/p5-HTML-Defaultify/Makefile	2011-06-15 09:18:52.000000000 +0900
+++ www/p5-HTML-Defaultify/Makefile	2011-06-15 09:23:36.000000000 +0900
@@ -15,12 +15,9 @@
 MAINTAINER=	takefu at
 COMMENT=	Perl module to populate HTML forms with default values

+RUN_DEPENDS=	p5-HTML-Parser>=0:${PORTSDIR}/www/p5-HTML-Parser

 MAN3=		HTML::Defaultify.3

-.include <>
-RUN_DEPENDS+=	p5-HTML-Parser>=0:${PORTSDIR}/www/p5-HTML-Parser
-.include <>
+.include <>
diff -ruN /usr/ports/www/p5-HTML-Defaultify/pkg-descr www/p5-HTML-Defaultify/pkg-descr
--- /usr/ports/www/p5-HTML-Defaultify/pkg-descr	2011-03-04 22:58:35.000000000 +0900
+++ www/p5-HTML-Defaultify/pkg-descr	2011-06-15 09:48:10.000000000 +0900
@@ -1,3 +1,10 @@
-HTML::Defaultify - Pre-fill default values into an existing HTML form
+This module lets you take an existing HTML page with forms in it, and fill in
+the form fields according to defaults you give. It's most useful in CGI
+scripts. One common use is to handle invalid user input-- show them the same
+form with their previously entered values filled in, and let them correct any
+errors. Another use might be when letting users edit a database record, such as
+their account information-- show them an input form with the current values
+filled in. There are other uses.

--- p5-HTML-Defaultify.patch ends here ---

More information about the cvs-all mailing list