NetBSD Problem Report #56444

From www@netbsd.org  Wed Oct  6 10:47:52 2021
Return-Path: <www@netbsd.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id C9EAD1A921F
	for <gnats-bugs@gnats.NetBSD.org>; Wed,  6 Oct 2021 10:47:52 +0000 (UTC)
Message-Id: <20211006104751.ABEFD1A9239@mollari.NetBSD.org>
Date: Wed,  6 Oct 2021 10:47:51 +0000 (UTC)
From: aqua_dabbler@mac.com
Reply-To: aqua_dabbler@mac.com
To: gnats-bugs@NetBSD.org
Subject: A new package: textproc/pgFormatter-5.1
X-Send-Pr-Version: www-1.0

>Number:         56444
>Category:       pkg
>Synopsis:       A new package: textproc/pgFormatter-5.1
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bsiegert
>State:          closed
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Wed Oct 06 10:50:00 +0000 2021
>Closed-Date:    Sat Nov 06 15:10:34 +0000 2021
>Last-Modified:  Sat Nov 06 15:10:34 +0000 2021
>Originator:     Kogule Ryo
>Release:        
>Organization:
>Environment:
>Description:
A new package: textproc/pgFormatter-5.1

Homepage: https://github.com/darold/pgFormatter/

A PostgreSQL SQL syntax beautifier that can work as a console program or as a
CGI.

This SQL formatter/beautifier supports keywords from SQL-92, SQL-99, SQL-2003,
SQL-2008, SQL-2011 and PostgreSQL specifics keywords. Works with any other
databases too.
>How-To-Repeat:

>Fix:
begin 644 pgFormatter.tgz
M'XL(`'9[76$``^U8;6^;2!#N5_,K5FVDWNELWA=,)$M';.)P]=N!T]ZGLQ98
M,+(-'(O;I)'_^PTV<>RV=I33I>GI/+*UL/.Z,[OS`%ETF>8+4A0T%UX]$XFB
MJ&.,UJ.V&459W8P5(4F59%W6L2R+2)14451>(?Q<`>W2DA4DAU!F:;2<T\-R
M(!:&1^Q4Z]B._Q'*=NK?L=RV\PP^(!^:JAZNOZ+CJOZ**"EJ67^L::^0^`RQ
M?$7_\_J;:)2R(LJI^WL/E7]VFQ3D!GF4+(LXC&F.BBDID$\2]"G-9X@P1)"?
M)BR=4Y3E:923!4KS]3S7[MH\QXVG,5O;"K=;:\<<6V99FA<,S>@M6`P8"O-T
M4<HW#+F^&8W-*,/>J'/55?-^3I(028+=N%E&?3#N/]CDT0<(EJ%/<3$%Z5N4
M%E.:<P$IB$<89:A(4YY[Z>3_`+1[_OMD1L/XV"'X9_3(^8?::MO^#R.<?ZRJ
MRNG\?P]Z@\X&M+AP.V<<U[7'5]<7DY$S_,UJCUNUG;UQSQN;W58MIR$3"A(Q
MX2/F):YCN^.!V;=:M?7MZ%UW<W=VMV]PU3B[NY<]=^M_?JS75US;'%O=H6-;
M;JM6T)L".HJ/'HYI1O,YYOJF.[:<B6N/2[&SNYW[R<;'>2L@>3H/A-4ZG(E[
M?=&QG6_$P($Q>S"&OP7L;!:QW&\L&<W9KYL\\&D><5?#OC4RN["(:5%D[%P0
M(F@D2X_WTX50>=H].5Q[V.];`\C98]V4Z]EM:^""Y6PCR?Z:-^:Q3Q-&.>Z#
M\\YUVJT:Q`V7L(*5<'97)70R&#K6^Q4WLIP>GHS,]KL>K+15`\OI7C!\1OS9
M/&8%=^U:DYXYZ%[#4B!Q;U"2)N"E,VRODP->1HYU:?^Q$MB4Y%0(4G_C[\)T
M+4B5/7#'9J]GCNWA8`(J[B]KI8T^\"'+D.MVSW1=:\VKA?%-(R/%E%4\T.]:
M_'861+*<-@`^PCA:YK22ZENN^X7<97P3)Q$B'N#,L@"D*:?Y2O[2[EGNGM4R
M\BR:;`"'S]+@WK_5V9-K4/26U?>ERVAX1A;9G-:WBSO`C]YR'%=6KA$GT!'F
M\W..!S$$%E&<H/:4)!'MI1&JRHP<R^ST+730(P?YK-(\Z9AC<X76I8==4)8>
MU,H)>#`;ET$]I)ZG20#&(!@^3OSY,J#H-<\+\)M#!,+ZV`B+-("6QB]FK[^2
M6LP$CP4\[/\U^Z6[T(E>BG8;QZAL*,_@XQ'\E^!QO\)_C'6]Q']54T[X_UWH
M5T"T!4V*A\<``(`UPD`#G52=Z;TUZ`P=TVE?K80C<'-0LV=?K/9T+C9X>,MG
MBR=HE6\73U/H'5>XL`<[C?ZP7-\<2%^`@G146JFDMZ&<GV^7K#Q5LUSVTY5Z
MI=(#K.^EY1ZE#O`K[#K`W2#:`>:W,>ZE-_F)#M)N\0(XQ'$2IO^VC\>^_V!-
MJ?J_K)??@N#]#VO2J?]_#WIX^W.O3`G]M+L?RM<Y'K+#1Y]_1BTDZ9CJLF8$
M<N@;FN$!2F.L-!55]JA"<$`]CV+5ESFGWY$T\:@M$50PV%.E0/.)2FG0E"0L
M>48S5,&R(H:AI,NJ44:%)?FHK:9&0D^DND[*APD5>X$?>H$2X-"0?&P$?C.@
MBJ9KGDIU326JU]0D52=8#@-=;^I:4_(4)52IK$`@(O7`/Z8&D=4`>QK<BY)O
MB%A70VR$(?:))X>*HC8-'X<R-22/<^//]'CF%'`HRLB[+2@[=<,3G>A$/P;]
)#7SHR!8`'```
`
end

>Release-Note:

>Audit-Trail:
From: nia <nia@NetBSD.org>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: pkg/56444: A new package: textproc/pgFormatter-5.1
Date: Wed, 6 Oct 2021 14:28:10 +0000

 Thanks for your submission.

 A few nits:

 - "GITHUB_TAG=     refs/tags/v5.1" is not necessary.
   It can just be v5.1
 - Setting the distname to just v5.1 will conflict with other packages.
   It should contain the package name.
   Luckily, with github packages, you can set DISTNAME to anything.
   Then you can unset DIST_SUBDIR.
 - New package names should be all-lowercase.

 Therefore, I would propose:

 GITHUB_PROJECT= pgFormatter
 GITHUB_TAG=     v5.1
 DISTNAME=       ${GITHUB_PROJECT}-${GITHUB_TAG}
 PKGNAME=        ${DISTNAME:S/-v/-/1:tl}

From: Leonardo Taccari <leot@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/56444: A new package: textproc/pgFormatter-5.1
Date: Wed, 06 Oct 2021 16:45:14 +0200

 nia writes:
 > [...]  
 >  A few nits:
 >  
 >  - "GITHUB_TAG=     refs/tags/v5.1" is not necessary.
 >    It can just be v5.1
 >  - Setting the distname to just v5.1 will conflict with other packages.
 >    It should contain the package name.
 >    Luckily, with github packages, you can set DISTNAME to anything.
 >    Then you can unset DIST_SUBDIR.
 >  - New package names should be all-lowercase.
 >  
 >  Therefore, I would propose:
 >  
 >  GITHUB_PROJECT= pgFormatter
 >  GITHUB_TAG=     v5.1
 >  DISTNAME=       ${GITHUB_PROJECT}-${GITHUB_TAG}
 >  PKGNAME=        ${DISTNAME:S/-v/-/1:tl}
 > [...]  

 I have not tested it but I think that's a bit easier to follow
 (rationale same of what you have proposed :)):

 DISTNAME=       pgFormatter-5.1
 PKGNAME=        ${DISTNAME:tl}
 GITHUB_PROJECT= pgFormatter
 GITHUB_TAG=     v${PKGVERSION_NOREV}

From: "Kogule, Ryo" <aqua_dabbler@mac.com>
To: gnats-bugs@netbsd.org
Cc: pkg-manager@netbsd.org,
 gnats-admin@netbsd.org,
 pkgsrc-bugs@netbsd.org
Subject: Re: pkg/56444: A new package: textproc/pgFormatter-5.1
Date: Thu, 7 Oct 2021 07:56:08 +0900

 > Oct 6, 2021 23:50=E3=80=81Leonardo Taccari <leot@NetBSD.org> wrote:
 >=20
 > DISTNAME=3D       pgFormatter-5.1
 > PKGNAME=3D        ${DISTNAME:tl}
 > GITHUB_PROJECT=3D pgFormatter
 > GITHUB_TAG=3D     v${PKGVERSION_NOREV}

 I made an updated one following this suggestion.

 By the way, these parameters are automatically generated by
 url2pkg-21.1.0nb1 and I did pkglint-21.2.6 over it.  I'm glad
 if one of them points these flaws out.



 begin 644 pgformatter.tgz
 M'XL(`#\F7F$``^U8ZV^;2!#OU_!7K-I(O=3D/%P/(TD2P5V\3AZM>!T]XW:X$%
 M(V.@@*])*__O-X!?2>L\3I?F3O4HUMK,;]Z[,TO2P$^R!2D*FG&OGHEXGE=3DE
 M&56K4J^\(-7KFA"6L*!*@J`((N*QQ$O2*R0_ET/[M,P+DH$K\R181O0P#F"^
 M?X^>=3D1S;]7]"Z5[]NX;=3DL9[!!N1#D:3#]1=3D5N:Z_(/)8E,KZ*X+R"O'/X,LW
 M])/77T?C)"^"C-I_]%'YR6_B@EPCAY)E$?HAS5`Q(P5R28P^)]D<D1P1Y"9Q
 MGD04I5D29&2!DJQZSG1Z)LLPDUF85[IV6VM/7;Y,TR0K<C2G-Z#1RY&?)8L2
 MW]"$LWK5ZE6`O7'&K+\U-\\P1B3V]OW.4^J"<G>GDT4?P=3DD<?0Z+&:!O4%+,
 M:,9XI"`.R6F.BB1AF9=3D._G^`]L__@,RI']YW"/X9/7#^H;9XV_\EL>S_LJ1*
 MQ_/_(^@-.AW2HFUW3QFF:]J3H3XP6B=3DI<+'9%0V9Q<SX?:]FG'[=3D@,Z+:,7T
 MS,GE57LZMD:_&YW)+;D-;Z+W6B=3D_G7X%%1\,RS9'P^EP9!D?5DQ'GQB]D64:
 M=3DNNDH-<%-!,7[4YH2K-(9@:Z/3&LJ6U.2MCIU[W?T]K"><LC61)YW*H*8&I?
 MM;NF56)O>[=3DB0)DYG,#'`'8Z#_+,;2QSFN7OZA2P218PEZ.!,=3D9[$.NL*-+\
 MG.,"Z"%+AW63!;>VM!<FQW1&@X$QA.`?:J1,W^P80[M,;XW,/T6-*'1IG%.&
 M^6B]MZU.ZP3\AJ\0P8JKDE;F>I.QL6'UY>E8[[SO0Z2M$]"<W'*&38D[C\*\
 M8*YL8]K7A[TK"`42]P;%20Q6NJ-.E1RP,K:,"_//%9?/2$8Y+W%K>VW=3D-B!5
 MYM">Z/V^/BD+!B+V;Y50+0]\R#+DNM/7;=3DNH>"=3D^>-U(23'+USR0[QGL]BE`
 MTHPV8'+X8;#,Z!HU,&S[#NXBO`[C`!$'1LRR@"%3/F;7^`NS;]BWM):>I\&T
 M;F-LFG@;^T;W%JY!T=3DO\[#:Z](;-R2*-Z-DVN`/\X"W#,&7E&F$,S2"*SAD6
 M8`@THC!&G1F)`]I/`K0N,[(,O3LPT$&+#.1SG>9I5Y_H*U25'G9!67H0*Q_`
 MG6Q2.K5+/4MC#Y2!,VP8N]'2H^@URW+P%X$'7'5LN$7B03=3DC%_/7WZ`6<\[)
 M/1;V?\5^Z09TI!>E_?D_+KO*,]AX8/YC`:O;^:]BOKK_*_QQ_O\(>@=3DC;4'C
 M8G<-@"E0C1GHHM/-%+`ZEWVSO:J:2Y2X)*KZ['>0'XQA=3DV25^!5WSW0Z*%E;
 MV9-IU^/SADT73Y`JWT.>)M"_7Z!M#O?FPF'<0!_B.S,$WXL6U^BM*^?GVY#%
 MITJ683]=3DJ%\*[6X!^QUA.]0.\->C[@"W'H`'F-\?B2]]''XZVB^)!T<SC/WD
 MW[;QT/]_%''7_\M9`.]_"A:/_?]'T.[MS[[4,?IEORG>>0MD(5%L\.57U$)8
 ME:DJ*)HG^*ZF:`X,;%D6FZ(D.%0DLD<=3DA\J2*S#6H(L5_K%J>9"60;6$/<4E
 M$J5>$V,9.UK3E\"(R/L^5@5)*WV5L?!8M4V%^`Y/5964MPU)=3DCS7=3DSS1DWT-
 MN[+FN4V/BHJJ.!)5%8E(3E/!DDIDP?=3D4M:DJ3>R(HB]1002?>.J`*S+5B"!Y
 MLJ/`;QZ[&B^KDB]KOB^[Q!%\492:FBO[`M6PP]CA%_KHU(I@FQ>0<U/0_-@/
 0CW2D(ST?_0V6&XT&`!P`````
 `
 end

Responsible-Changed-From-To: pkg-manager->bsiegert
Responsible-Changed-By: bsiegert@NetBSD.org
Responsible-Changed-When: Thu, 04 Nov 2021 10:12:39 +0000
Responsible-Changed-Why:
take


From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/56444 CVS commit: pkgsrc/databases
Date: Sat, 6 Nov 2021 15:09:26 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Sat Nov  6 15:09:26 UTC 2021

 Modified Files:
 	pkgsrc/databases: Makefile
 Added Files:
 	pkgsrc/databases/p5-pgFormatter: DESCR Makefile PLIST distinfo

 Log Message:
 New package, p5-pgFormatter. From Kogule Ryo in PR pkg/56444.

 A PostgreSQL SQL syntax beautifier that can work as a console program or as a
 CGI.

 This SQL formatter/beautifier supports keywords from SQL-92, SQL-99, SQL-2003,
 SQL-2008, SQL-2011 and PostgreSQL specifics keywords. Works with any other
 databases too.

 Changes from the submitted version in the PR:

 - p5 prefix for the package name
 - clean up of github logic
 - REPLACE_PERL
 - updated distinfo with BLAKE2s


 To generate a diff of this commit:
 cvs rdiff -u -r1.641 -r1.642 pkgsrc/databases/Makefile
 cvs rdiff -u -r0 -r1.1 pkgsrc/databases/p5-pgFormatter/DESCR \
     pkgsrc/databases/p5-pgFormatter/Makefile \
     pkgsrc/databases/p5-pgFormatter/PLIST \
     pkgsrc/databases/p5-pgFormatter/distinfo

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->closed
State-Changed-By: bsiegert@NetBSD.org
State-Changed-When: Sat, 06 Nov 2021 15:10:34 +0000
State-Changed-Why:
Committed to pkgsrc, thank you!


>Unformatted:

Home	PR Database Search