ports/152399: databases/yasql fail to compile/run due to non-existing function call

[Riccardo Torrini]

>Number:         152399
>Category:       ports
>Synopsis:       databases/yasql fail to compile/run due to non-existing function call
>Confidential:   no
>Severity:       serious
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Nov 19 12:00:18 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator:     Riccardo Torrini
>Release:        FreeBSD 8.1-STABLE i386
>Organization:
>Environment:
System: FreeBSD tiger.fi.esaote.it 8.1-STABLE FreeBSD 8.1-STABLE #16: Tue Oct 19 09:24:48 CEST 2010 root at tiger.fi.esaote.it:/usr/obj/usr/src/sys/TIGER i386


	
>Description:
	yasql try to use perl err() function and fail to compile/run.
	Also give some warning about deprecated functions.

>How-To-Repeat:
	# cd /usr/ports/databases/yasql && make install clean
	# yasql
	syntax error at /usr/local/bin/yasql line 177, near "err"
	syntax error at /usr/local/bin/yasql line 180, near "}"
	Execution of /usr/local/bin/yasql aborted due to compilation errors.

	After substitued all err() with die() yasql start but if you use
	perl -w `which yasql` you can also get this warnings:

	Use of "do" to call subroutines is deprecated at /tmp/ya line 2407.
	Use of "do" to call subroutines is deprecated at /tmp/ya line 2410.
	\1 better written as $1 at /tmp/ya line 2876.
	\1 better written as $1 at /tmp/ya line 2890.
	Unrecognized escape \G passed through at /tmp/ya line 3565.
	Unrecognized escape \S passed through at /tmp/ya line 3565.

	First four lines are self-explained, last two have a missing "\"
	in the help section (to correctly show as "\G" asking for help).



>Fix:

	Change err() with die(), replace \1 with $1, add some missing \
	Also bumped PORTREVISION to 1 (full patch attached)

begin 600 yasql.patch.gz
M'XL(`&%4YDP"`ZU7;7/B.!+^''Y%CX<,$(RQ at 01B+MED&68W=4F8 at FSNKI)9
MRF`!OAB+D>1D4 at GWVZ\EV880YN9ECW)96.IN=3_]J"7YP60"%19?PJ/'/X?6
M at I'[ZH5W1R9!2'1?]IFK5"K;Q'9JMGU8L5N5VC[8AVZ][M9:EIW^H&SC>*Y<
M+F^80S7'KCA.Q3D$QW$;CENKO5!S4.WD!"I-\P#*3;,))R<YR,''7O_J\O2B
M>[2SHPSJGNMN?W#6N\1.QVK5<V79U^]>GZ6=.>B<7G5_Z_7/N@/\]CWAC3Q.
M>`XN3@=7W?YP<':E1 at 8?<!(Y[4%=SGO0,`_5Q#L[^:>+ZR7DGS[VNQ_._KFL
M$C&N:CC&-)I\=4""-0YHS*$-MVAG$N`$Y07EHK+PQ'CFYLIHNM\]N_QX?MKI
M#CL7[Y=0X.;N[GFO<WK^Z^F at N[MKYI^RKZ4Y+>!L_^C_?=#O+).9 at BA7SH&R
M&D1<>&'HXEP_ZS+WY at M,.%C!!-[XF*Z(^,7+G at 3U?:\S*.7\;;R16>55%50E
M]2I)^K:A33YMD]EQ#IMVQ7;P`=MQ;1N?5R19<6NKB8QG+7#VW9KCUAM;>6:;
M^.68CM.0"2^OO`LBB[)@*GF^7Y$/$K;EHCN-YC:>E^$K4SNV6W]-\;(BFXW\
M+JNWG!L)`IR$9"R*Q:0=(/)_7)5,R#_#$3BE&_M3J0UOXV at 43R:$@1Y7JI4\
M?^0RCW[`4-:0V37:Z-I&_PN&I4*0"DF)-84U>B at Q?-Z"8(\@*`31.(Q]`E?!
MG+CN[T&?<)A0!I(W,&6>I`^,2#2>S3UV%T13';33;,BH=:/"UK\'%A6-#HTB
MC#N@$83(:B@*-$YCX4)>^O`TSL:'R<BR9)3:F9'/<2"*3M:Q!!)R`D^(CAXG
MC,E)XM"'B`I(S,EHTMI@@C3L`YJVUJ8W$$AMP at _(3YN`[&<9WQ/04K5+#5SS
MX%`"IYL,N$P$WW&$"X$7\Q,:1WXI"WLC:,R/#^]_?>^Z/>:-0V)9%N"Z1\<A
M*2(6P.F]%X3>"+<#GP7WA/$5!!L`_`5S&1C_I@$FWP1\3A*!$EA at 6+=1"H:.
M\.V"!9$`([-R8Z"<5G_>U/Z$VFV-W6'-D=CI)L,NGZ`_Y`+-3I'[A0)@\=OL
M)Y^A0+X(PB(O+*2YR?NCF1?Y&-(1^*-AHE+,(SQ#S":"3IF9I\P;XH;#98+G
MU"<F9H9,DO?&/&FDDK3!Y*6EKV9S"_]<Z1N*H56XT7\_*1C+W\??K^J_6%'K
MB_8GC0$P(F(T89>2/#E8'%5UT*W*%(]',)Z1\=V0*H(-9W2>>O`6KF8!U\.Z
M^H at 9@5[_M'/>'?[>N^@"B>X#1J,Y0=K<>RQ0),2\21RX3'8@"CRQ)4/@1"B@
M%<P?-8^Q#QYIS+YI^(U1`@6S`OGGM!4N>B5#OGMY_;2FMM2X):CI.I=4!Z?A
M'"CD=*N06R9[B at 00RRD9COE]PB/U7>P,KDOP_/R*4W(0<!#DUJI9CGE\DY+H
MI?(FH?Z7LEIT_'ZX&AK2!8EP"=EKHW/ORS#$[8.G_4F,M8;=,!W<0M,_*LH=
MN5;H`MZ]`]G(Q7I<D(YE7\>%4L;8'?F2\D.!2[J$,TQ#.EIUM-?E?#**IW,^
M+=:Q-O7045D-U"&YD"H4%.NP;B]B at 8G!/3/9OV(NA6]03O.]HBS*:(L?<,\V
ME7L6K#R14%+<P0A[E&N^6$BM%4RCXT426X75MOD?6"!P.H6SB55,)>H[)OS_
MS9;M;%GY2M!__C'P?QRF1;`@&T[+KC6G_PHL/V@==M8JY.9,QO$[?6`S$M&$
MUJUF72[=I$VVI_DCY"/R@,<OA"F/33OKGC`ZYP)/@*NNAQEA9,IHO*#,)VRE
M at FO]]>!_@%>MO:+J?U9]SVJX9.U5;YVJ7N4_II97:N at -EM3B*U4D at O1GQ80M
M#AM&>_VX4VNUFAJ55G-]T\9H=\<TC.?1$"\0D^!+2CYD)O'&,P6'4&6^>"+"
M@(O5K)".J%#^A+UJM9V=#=>'BC=_PJ=RZ9:7B]9>:87)-^0R$)+3Q9J73UIM
M*3-3 at U_DRTU,M3>\DSZ#=80PKCY_P;.1 at 1J&(8\V+_42P.H'#76R3MH$L.X7
M=:OC;B)\A?OCA(8A?9#EZ2$(0V!QI+9-Q7I3_HU47R#`F^(Q'J7$#.3E#U,6
MJ2U2+O_,)&Y7G+#;DUZ_<WZ<W&!@#R1+03OJV!G*ZZ*WO^W;&:PO!O0(]N*K
MORJJ>L9_T1C&7 at 2>[X,'?$8PAC"X(R_*+RX[Y at DL5MX$#VXHIZ*3)Q.U?&6\
MNFBGX.TW-7C[S;7K&&Q!\-OAPK'^8_GQ?+$U\BU:MX-,#??`K;!LT4*H-O2T
MVV<1AI9ZG`*&MJ#PMX($`>]G(W5&T[:\;,.V$HK(LC^/D7DCR1<V]X0 at OL:J
MX335321I4ZR.`D'F</G'^3EP01<<U%E=LDRG0/L[I?+8%:$+%`_J$^0A8)(X
M'NG$3-T1Y556F\+BRC"_"V\LKQ+(%GFUF'N/Z!'%_#$\MI6W2VX37;F(>[J+
GWDSQN"H3J^CBHUM10<`#97>:[V,/0_"BQSF6%:G]7S;=I@<U$P``
`
end


>Release-Note:
>Audit-Trail:
>Unformatted: