[Haven't actually looked at this patch.  But the general idea of using
Dos* functions instead of "attrib" is definitely the way to go.

-kingdon]

Date: Tue, 07 Apr 1998 16:55:22 +0200
From: tobi@bland.fido.de (Tobias Ernst)
Subject: cvs 1.9.26 OS/2 patch submittal
To: bug-cvs@gnu.org (UUCP)

Hello!

I have created a patch to CVS 1.9.26. It only applies to the OS/2 versions of
CVS located in the os2/ and emx/ subdirectories and gives a factor 3 speed
improvement on "cvs co" and other actions that require changing the read only
attribute of a file. The patch does not change the behaviour of CVS in any way,
so it should be transparent to the user (besides the speed increase). All I did
to achieve this improvement was to replace the xchmod() routine that just did a
system("attrib.exe") by a self-written one that calls the corresponding OS/2
API directly.

I have attached two GNU context-style diffs: os2.dif is for the files in the
os2/ subdirectory, and emx.dif is for the files in the emx/ subdirectory. I
uuencoded them to avoid my newsreader from mangling up linebreaks or similar.

I hope I did abide to all rules for submitting a CVS patch. This is the first
time that I submit a patch for a GNU software, so please be forbearing if I
should have made something wrong. :-)

Kind Regards,
Tobias Ernst.

begin 666 emx.dif
M*BHJ("XN+RXN+V-V<RTQ+CDN,C8O96UX+V9I;&5S=6)R+F,)5'5E($9E8B`Q
M-R`Q-CHS,CHR."`Q.3DX"BTM+2`N+V9I;&5S=6)R+F,)5'5E($%P<B`@-R`Q
M-CHT-CHU."`Q.3DX"BHJ*BHJ*BHJ*BHJ*BHJ*@HJ*BH@,3@L,C,@*BHJ*@HM
M+2T@,3@L,C0@+2TM+0H@("`@(&9I;&4@<WES=&5M('-E;6%N=&EC<RX@("HO
M"B`@"B`@(VEN8VQU9&4@(F-V<RYH(@HK("-I;F-L=61E(#QO<S(N:#X@("`@
M("`@("\J(&9O<B!$;W-1=65R>5!A=&A);F9O("HO"B`@(VEN8VQU9&4@/'-Y
M<R]P87)A;2YH/@H@(`H@("\J"BHJ*BHJ*BHJ*BHJ*BHJ*B!M:V1I<E]I9E]N
M965D960@*&YA;64I"BHJ*B`S,#<L,S4T("HJ*BH*("`O*@H@("`J($-H86YG
M92!T:&4@;6]D92!O9B!A(&9I;&4L(&5I=&AE<B!A9&1I;F<@=W)I=&4@<&5R
M;6ES<VEO;G,L(&]R(')E;6]V:6YG"B`@("H@86QL('=R:71E('!E<FUI<W-I
M;VYS+B`@16ET:&5R(&-H86YG92!H;VYO<G,@=&AE(&-U<G)E;G0@=6UA<VL@
M<V5T=&EN9RX*+2`@*B!4:&4@14U8(&1O8R`H,"XY8RP@96UX;&EB+F1O8RD@
M<V%Y<R!T:&%T(&-H;6]D('-E=',O8VQE87)S('1H92!R96%D;VYL>0HM("`J
M(&)I="X@($)U="!I="!A;'=A>7,@<V5E;65D('1O(&)E(&$@;F]O<"!W:&5N
M($D@=')I960@:70N("!4:&5R969O<F4L"BT@("H@22=V92!C;W!I960@;W9E
M<B!T:&4@(F%T=')I8B(@8V]D92!F<F]M(&]S,B]F:6QE<W5B<BYC+@H@("`J
M+PH@('9O:60*("!X8VAM;V0@*&9N86UE+"!W<FET86)L92D*("`@("`@8VAA
M<B`J9FYA;64["B`@("`@(&EN="!W<FET86)L93L*("!["B$@("`@(&-H87(@
M*F%T=')I8E]C;60["B$@("`@(&-H87(@*F%T=')I8E]O<'1I;VX["B$@("`@
M(&-H87(@*G=H;VQE7V-M9#L*(2`@("`@8VAA<B`J<#L*(2`@("`@8VAA<B`J
M<3L*("`*+2`@("`@:68@*"%I<V9I;&4@*&9N86UE*2D*+2`@("`@>PHM(`EE
M<G)O<B`H,"P@,"P@(F-A;FYO="!C:&%N9V4@;6]D92!O9B!F:6QE("5S.R!I
M="!D;V5S(&YO="!E>&ES="(L"BT@"2`@("`@("!F;F%M92D["BT@"7)E='5R
M;CL*+2`@("`@?0HM(`HM("`@("!A='1R:6)?8VUD(#T@(F%T=')I8B`B.R`O
M*B!.;RP@<F5A;&QY/R`J+PH@(`HA("`@("!I9B`H=W)I=&%B;&4I"B$@("`@
M("`@("!A='1R:6)?;W!T:6]N(#T@(BUR("([("`O*B!M86ME('=R:71E86)L
M92`J+PHA("`@("!E;'-E"B$@("`@("`@("!A='1R:6)?;W!T:6]N(#T@(BMR
M("([("`O*B!M86ME(')E860M;VYL>2`J+PHA("`@("`@("`@"B$@("`@('=H
M;VQE7V-M9"`]('AM86QL;V,@*'-T<FQE;B`H871T<FEB7V-M9"D*(2`@("`@
M("`@("`@("`@("`@("`@("`@("`@*R!S=')L96X@*&%T=')I8E]O<'1I;VXI
M"B$@("`@("`@("`@("`@("`@("`@("`@("`@("L@<W1R;&5N("AF;F%M92D*
M(2`@("`@("`@("`@("`@("`@("`@("`@("`@*R`Q*3L*(2`*(2`@("`@<W1R
M8W!Y("AW:&]L95]C;60L(&%T=')I8E]C;60I.PHA("`@("!S=')C870@*'=H
M;VQE7V-M9"P@871T<FEB7V]P=&EO;BD["B$@"B$@("`@("\J($-O<'D@9FYA
M;64@=&\@=&AE(&5N9"!O9B!W:&]L95]C;60L('1R86YS;&%T:6YG("\@=&\@
M7"X*(2`)("`@071T<FEB(&1O97-N)W0@=&%K92`O(&)U="!M86YY('!A<G1S
M(&]F($-64R!R96QY"B$@("`@("`@(&]N(&)E:6YG(&%B;&4@=&\@=7-E(&ET
M+B`@*B\*(2`@("`@<"`]('=H;VQE7V-M9"`K('-T<FQE;B`H=VAO;&5?8VUD
M*3L*("`@("`@<2`](&9N86UE.PH@("`@("!W:&EL92`H*G$I"B`@("`@('L*
M("`):68@*"IQ(#T]("<O)RD*+2TM(#,P."PS,C<@+2TM+0H@("\J"B`@("H@
M0VAA;F=E('1H92!M;V1E(&]F(&$@9FEL92P@96ET:&5R(&%D9&EN9R!W<FET
M92!P97)M:7-S:6]N<RP@;W(@<F5M;W9I;F<*("`@*B!A;&P@=W)I=&4@<&5R
M;6ES<VEO;G,N("!%:71H97(@8VAA;F=E(&AO;F]R<R!T:&4@8W5R<F5N="!U
M;6%S:R!S971T:6YG+@H@("`J+PH@('9O:60*("!X8VAM;V0@*&9N86UE+"!W
M<FET86)L92D*("`@("`@8VAA<B`J9FYA;64["B`@("`@(&EN="!W<FET86)L
M93L*("!["B$@("`@($9)3$535$%455,S(',["B$@("`@(&-H87(@*F)A8VMS
M;&%S:&EF:65D+"`J<"P@*G$["B`@"B`@"B$@("`@(&)A8VMS;&%S:&EF:65D
M(#T@>&UA;&QO8R`H<W1R;&5N*&9N86UE*2`K(#$I.PHA("`@("!P(#T@8F%C
M:W-L87-H:69I960["B`@("`@('$@/2!F;F%M93L**R`*("`@("`@=VAI;&4@
M*"IQ*0H@("`@("!["B`@"6EF("@J<2`]/2`G+R<I"BHJ*BHJ*BHJ*BHJ*BHJ
M*B!X8VAM;V0@*&9N86UE+"!W<FET86)L92D**BHJ(#,U.2PS-C@@*BHJ*@H@
M("`@("!]"B`@("`@("IP(#T@)UPP)SL*("`*(2`@("`@<WES=&5M("AW:&]L
M95]C;60I.PHA("`@("!F<F5E("AW:&]L95]C;60I.PH@('T*("`*("`O*@H@
M("`J(%)E;F%M92!A(&9I;&4@86YD(&1I92!I9B!I="!F86EL<PH@("`J+PHM
M+2T@,S,R+#,V-B`M+2TM"B`@("`@('T*("`@("`@*G`@/2`G7#`G.PH@(`HA
M(`HA("`@("!I9B`H1&]S475E<GE0871H26YF;R@H4%-:*6)A8VMS;&%S:&EF
M:65D+"!&24Q?4U1!3D1!4D0L"B$@("`@("`@("`@("`@("`@("`@("`@("`@
M("A05D])1"DF<RP@<VEZ96]F*',I*2`A/2`P*0HA("`@("!["B$@"65R<F]R
M("@P+"`P+"`B8V%N;F]T(&-H86YG92!M;V1E(&]F(&9I;&4@)7,[(&ET(&1O
M97,@;F]T(&5X:7-T(BP*(2`)("`@("`@(&9N86UE*3L*(2`@("`@("`@(&9R
M964@*&)A8VMS;&%S:&EF:65D*3L*(2`)<F5T=7)N.PHA("`@("!]"B$@"B$@
M("`@(&EF("@A=W)I=&%B;&4I("`@("`@("`@("`@("`@("`@("`@("`@("`O
M*B!W<FET:6YG(&YO="!A;&QO=V5D/R`J+PHA("`@("`@("`@<RYA='1R1FEL
M92!\/2!&24Q%7U)%041/3DQ9.R`@("`@("`@+RH@='5R;B!O;B!R96%D+6]N
M;'D@*B\*(2`@("`@96QS92`@("`@("`@("`@("`@("`@("`@("`@("`@("`@
M("`@("`@("\J('=R:71I;F<@86QL;W=E9"`J+PHA("`@("`@("`@<RYA='1R
M1FEL92`F/2!^1DE,15]214%$3TY,63L@("`@("`@+RH@='5R;B!O9F8@<F5A
M9"UO;FQY("HO"B$@"B$@("`@(&EF("A$;W-39710871H26YF;R@H4%-:*6)A
M8VMS;&%S:&EF:65D+"!&24Q?4U1!3D1!4D0L("A05D])1"DF<RP*(2`@("`@
M("`@("`@("`@("`@("`@("`@('-I>F5O9BAS*2P@,$PI("$](#`I"B$@("`@
M('L*(2`)97)R;W(@*#`L(#`L(")C86YN;W0@8VAA;F=E(&UO9&4@;V8@9FEL
M92`E<R(L"B$@"2`@("`@("!F;F%M92D["B$@("`@("`@("!F<F5E("AB86-K
M<VQA<VAI9FEE9"D["B$@"7)E='5R;CL*(2`@("`@?0HA(`HA("`@("!F<F5E
M("AB86-K<VQA<VAI9FEE9"D["B`@?0H@(`HK(`HK(`H@("\J"B`@("H@4F5N
M86UE(&$@9FEL92!A;F0@9&EE(&EF(&ET(&9A:6QS"B`@("HO"BHJ*B`N+B\N
M+B]C=G,M,2XY+C(V+V5M>"]#:&%N9V5,;V<)5'5E($9E8B`Q-R`Q-CHS,CHR
M."`Q.3DX"BTM+2`N+T-H86YG94QO9PE4=64@07!R("`W(#$V.C4P.C,X(#$Y
M.3@**BHJ*BHJ*BHJ*BHJ*BHJ"BHJ*B`Q+#,@*BHJ*@HM+2T@,2PY("TM+2T*
M*R!7960@07!R(#`W(#$V.C,P.C`P(#$Y.3@@(%1O8FEA<R!%<FYS="`\=&]B
M:4!B;&%N9"YF:61O+F1E/@HK(`HK("`@("`@("`@*B!F:6QE<W5B<BYC("AX
M8VAM;V0I.B!297!L86-E9"!S>7-T96T@8V%L;"!T;R`B871T<FEB+F5X92(@
M=VET:"!A"BL@("`@("`@("!D:7)E8W0@8V%L;"!T;R!T:&4@8V]R<F5S<&]N
M9&EN9R!/4R\R($%022X@5&AI<R!G:79E<R!A(&9A8W1O<B`S"BL@("`@("`@
M("!S<&5E9"!I;7!R;W9E;65N="!O;B!C=G,@8V\N"BL@"B`@5'5E($9E8B`Q
M-R`P,CHR.#HR,"`Q.3DX("!.;V5L($-R86=G("`\;F]E;$!S=VES:"YR960M
M8F5A;BYC;VT^"B`@"B`@"2H@9FEL97-U8G(N8R`H;&%S=%]C;VUP;VYE;G0I
F.B!R971U<FX@=&AE('1O<"UL979E;"!D:7)E8W1O<GD@=VAE;@H`
`
end

begin 666 os2.dif
M*BHJ("XN+RXN+V-V<RTQ+CDN,C8O;W,R+V9I;&5S=6)R+F,)5'5E($9E8B`Q
M-R`Q-CHS,CHS,B`Q.3DX"BTM+2`N+V9I;&5S=6)R+F,)5'5E($%P<B`@-R`Q
M-CHT,#HP-B`Q.3DX"BHJ*BHJ*BHJ*BHJ*BHJ*B!M:V1I<E]I9E]N965D960@
M*&YA;64I"BHJ*B`R.34L,S,X("HJ*BH*("`@*B!A;&P@=W)I=&4@<&5R;6ES
M<VEO;G,N("!!9&1I;F<@=W)I=&4@<&5R;6ES<VEO;G,@:&]N;W)S('1H92!C
M=7)R96YT('5M87-K"B`@("H@<V5T=&EN9RX*("`@*B\*("!V;VED"B`@>&-H
M;6]D("AF;F%M92P@=W)I=&%B;&4I"B`@("`@(&-H87(@*F9N86UE.PH@("`@
M("!I;G0@=W)I=&%B;&4["B`@>PHA("`@("!C:&%R("IA='1R:6)?8VUD.PHA
M("`@("!C:&%R("IA='1R:6)?;W!T:6]N.PHA("`@("!C:&%R("IW:&]L95]C
M;60["B$@("`@(&-H87(@*G`["B$@("`@(&-H87(@*G$["B$@"B$@("`@(&EF
M("@A:7-F:6QE("AF;F%M92DI"B$@("`@('L*(2`)97)R;W(@*#`L(#`L(")C
M86YN;W0@8VAA;F=E(&UO9&4@;V8@9FEL92`E<SL@:70@9&]E<R!N;W0@97AI
M<W0B+`HA(`D@("`@("`@9FYA;64I.PHA(`ER971U<FX["B$@("`@('T*(2`*
M(2`@("`@871T<FEB7V-M9"`](")A='1R:6(@(CL@+RH@3F\L(')E86QL>3\@
M*B\*("`*+2`@("`@:68@*'=R:71A8FQE*0HM("`@("`@("`@871T<FEB7V]P
M=&EO;B`]("(M<B`B.R`@+RH@;6%K92!W<FET96%B;&4@*B\*+2`@("`@96QS
M90HM("`@("`@("`@871T<FEB7V]P=&EO;B`]("(K<B`B.R`@+RH@;6%K92!R
M96%D+6]N;'D@*B\*("`@("`@("`@(`HA("`@("!W:&]L95]C;60@/2!X;6%L
M;&]C("AS=')L96X@*&%T=')I8E]C;60I"B$@("`@("`@("`@("`@("`@("`@
M("`@("`@("L@<W1R;&5N("AA='1R:6)?;W!T:6]N*0HA("`@("`@("`@("`@
M("`@("`@("`@("`@("`K('-T<FQE;B`H9FYA;64I"B$@("`@("`@("`@("`@
M("`@("`@("`@("`@("L@,2D["B$@"B$@("`@('-T<F-P>2`H=VAO;&5?8VUD
M+"!A='1R:6)?8VUD*3L*(2`@("`@<W1R8V%T("AW:&]L95]C;60L(&%T=')I
M8E]O<'1I;VXI.PHA(`HA("`@("`O*B!#;W!Y(&9N86UE('1O('1H92!E;F0@
M;V8@=VAO;&5?8VUD+"!T<F%N<VQA=&EN9R`O('1O(%PN"B$@"2`@($%T=')I
M8B!D;V5S;B=T('1A:V4@+R!B=70@;6%N>2!P87)T<R!O9B!#5E,@<F5L>0HA
M("`@("`@("!O;B!B96EN9R!A8FQE('1O('5S92!I="X@("HO"B$@("`@('`@
M/2!W:&]L95]C;60@*R!S=')L96X@*'=H;VQE7V-M9"D["B`@("`@('$@/2!F
M;F%M93L*("`@("`@=VAI;&4@*"IQ*0H@("`@("!["B`@"6EF("@J<2`]/2`G
M+R<I"BTM+2`R.34L,S$T("TM+2T*("`@*B!A;&P@=W)I=&4@<&5R;6ES<VEO
M;G,N("!!9&1I;F<@=W)I=&4@<&5R;6ES<VEO;G,@:&]N;W)S('1H92!C=7)R
M96YT('5M87-K"B`@("H@<V5T=&EN9RX*("`@*B\**R`*("!V;VED"B`@>&-H
M;6]D("AF;F%M92P@=W)I=&%B;&4I"B`@("`@(&-H87(@*F9N86UE.PH@("`@
M("!I;G0@=W)I=&%B;&4["B`@>PHA("`@("!&24Q%4U1!5%53,R!S.PHA("`@
M("!C:&%R("IB86-K<VQA<VAI9FEE9"P@*G`L("IQ.PH@(`H@(`HA("`@("!B
M86-K<VQA<VAI9FEE9"`]('AM86QL;V,@*'-T<FQE;BAF;F%M92D@*R`Q*3L*
M(2`@("`@<"`](&)A8VMS;&%S:&EF:65D.PH@("`@("!Q(#T@9FYA;64["BL@
M"B`@("`@('=H:6QE("@J<2D*("`@("`@>PH@(`EI9B`H*G$@/3T@)R\G*0HJ
M*BHJ*BHJ*BHJ*BHJ*BH@>&-H;6]D("AF;F%M92P@=W)I=&%B;&4I"BHJ*B`S
M-#,L,S4P("HJ*BH*("`@("`@?0H@("`@("`J<"`]("=<,"<["B`@"B$@("`@
M('-Y<W1E;2`H=VAO;&5?8VUD*3L*(2`@("`@9G)E92`H=VAO;&5?8VUD*3L*
M("!]"B`@"B`@"BTM+2`S,3DL,S0Y("TM+2T*("`@("`@?0H@("`@("`J<"`]
M("=<,"<["B`@"B$@"B$@("`@(&EF("A$;W-1=65R>5!A=&A);F9O*"A04UHI
M8F%C:W-L87-H:69I960L($9)3%]35$%.1$%21"P*(2`@("`@("`@("`@("`@
M("`@("`@("`@("`@*%!63TE$*29S+"!S:7IE;V8H<RDI("$](#`I"B$@("`@
M('L*(2`)97)R;W(@*#`L(#`L(")C86YN;W0@8VAA;F=E(&UO9&4@;V8@9FEL
M92`E<SL@:70@9&]E<R!N;W0@97AI<W0B+`HA(`D@("`@("`@9FYA;64I.PHA
M("`@("`@("`@9G)E92`H8F%C:W-L87-H:69I960I.PHA(`ER971U<FX["B$@
M("`@('T*(2`*(2`@("`@:68@*"%W<FET86)L92D@("`@("`@("`@("`@("`@
M("`@("`@("`@("\J('=R:71I;F<@;F]T(&%L;&]W960_("HO"B$@("`@("`@
M("!S+F%T=')&:6QE('P]($9)3$5?4D5!1$].3%D[("`@("`@("`O*B!T=7)N
M(&]N(')E860M;VYL>2`J+PHA("`@("!E;'-E("`@("`@("`@("`@("`@("`@
M("`@("`@("`@("`@("`@("`@+RH@=W)I=&EN9R!A;&QO=V5D("HO"B$@("`@
M("`@("!S+F%T=')&:6QE("8]('Y&24Q%7U)%041/3DQ9.R`@("`@("`O*B!T
M=7)N(&]F9B!R96%D+6]N;'D@*B\*(2`*(2`@("`@:68@*$1O<U-E=%!A=&A)
M;F9O*"A04UHI8F%C:W-L87-H:69I960L($9)3%]35$%.1$%21"P@*%!63TE$
M*29S+`HA("`@("`@("`@("`@("`@("`@("`@("`@<VEZ96]F*',I+"`P3"D@
M(3T@,"D*(2`@("`@>PHA(`EE<G)O<B`H,"P@,"P@(F-A;FYO="!C:&%N9V4@
M;6]D92!O9B!F:6QE("5S(BP*(2`)("`@("`@(&9N86UE*3L*(2`@("`@("`@
M(&9R964@*&)A8VMS;&%S:&EF:65D*3L*(2`)<F5T=7)N.PHA("`@("!]"B$@
M"B$@("`@(&9R964@*&)A8VMS;&%S:&EF:65D*3L*("!]"B`@"B`@"BHJ*B`N
M+B\N+B]C=G,M,2XY+C(V+V]S,B]#:&%N9V5,;V<)5'5E($9E8B`Q-R`Q-CHS
M,CHS,B`Q.3DX"BTM+2`N+R]#:&%N9V5,;V<)5'5E($%P<B`@-R`Q-CHT,3HT
M,"`Q.3DX"BHJ*BHJ*BHJ*BHJ*BHJ*@HJ*BH@,2PS("HJ*BH*+2TM(#$L.2`M
M+2TM"BL@5V5D($%P<B`P-R`Q-CHS,#HP,"`Q.3DX("!4;V)I87,@17)N<W0@
M/'1O8FE`8FQA;F0N9FED;RYD93X**R`**R`@("`@("`@("H@9FEL97-U8G(N
M8R`H>&-H;6]D*3H@4F5P;&%C960@<WES=&5M(&-A;&P@=&\@(F%T=')I8BYE
M>&4B('=I=&@@80HK("`@("`@("`@9&ER96-T(&-A;&P@=&\@=&AE(&-O<G)E
M<W!O;F1I;F<@3U,O,B!!4$DN(%1H:7,@9VEV97,@82!F86-T;W(@,PHK("`@
M("`@("`@<W!E960@:6UP<F]V96UE;G0@;VX@8W9S(&-O+@HK(`H@(%1U92!&
M96(@,3<@,#(Z,S$Z,34@,3DY."`@3F]E;"!#<F%G9R`@/&YO96Q`<W=I<V@N
M<F5D+6)E86XN8V]M/@H@(`H@(`DJ(&9I;&5S=6)R+F,@*&QA<W1?8V]M<&]N
J96YT*3H@<F5T=7)N('1H92!T;W`M;&5V96P@9&ER96-T;W)Y('=H96X*
`
end