Thread: [patch] for "psql : Allow processing of multiple -f (file) options "

[patch] for "psql : Allow processing of multiple -f (file) options "

From
Vikash3 S
Date:
<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><font face="Default Sans
Serif,Verdana,Arial,Helvetica,sans-serif"size="2"><font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif"
size="2">Hi,<br /><br />Please find the patch regarding trivial changes against To Do item list for <font face="Courier
New,Courier,monospace"size="3"> "psql : Allow processing of multiple -f (file) options ".<br />Looking for valuable
feedback.<br/><br />Thanks,<br />Vikash<br /></font></font></font><div></div></font><p>=====-----=====-----=====<br />
Notice:The information contained in this e-mail<br /> message and/or attachments to it may contain <br /> confidential
orprivileged information. If you are <br /> not the intended recipient, any dissemination, use, <br /> review,
distribution,printing or copying of the <br /> information contained in this e-mail message <br /> and/or attachments
toit are strictly prohibited. If <br /> you have received this communication in error, <br /> please notify us by reply
e-mailor telephone and <br /> immediately and permanently delete the message <br /> and any attachments. Thank you<p> 

Re: [patch] for "psql : Allow processing of multiple -f (file) options "

From
Euler Taveira
Date:
On 09-04-2012 02:43, Vikash3 S wrote:
> Please find the patch regarding trivial changes against To Do item list for
> "psql : Allow processing of multiple -f (file) options ".
> Looking for valuable feedback.
> 
Aren't you forget to cover the single transaction (-1) mode? How would you
handle ON_ERROR_* options? Look at the archives for references. Also, your
disclaimer doesn't seems attractive; make it clear you're contributing code
under the PostgreSQL license.


--   Euler Taveira de Oliveira - Timbira       http://www.timbira.com.br/  PostgreSQL: Consultoria, Desenvolvimento,
Suporte24x7 e Treinamento