On Wed, Oct 27, 2004, Andreas Schmidt wrote:
> Peter S. Mazinger wrote:
>
> >Run -bi (instead of -bb) (original rpm uses rpm[buld] -bi <specfile>, I
> >can't tell how openpkg does this though), see if the ant-file has "r" in
> >the temporary install directory
> >Or do it step-by-step
> >-bp (unpackage)
> >-bc (compile) --short-circuit
> >-bi (install) --short-circuit
>
> now i have the corrupted ant-file in
> $pre/RPM/TPM/ant-1.6.2-root/opt/opkg/bin/
>
> to me, it seems the bug is following shtool-call
>
> + /opt/opkg/lib/openpkg/shtool install -c -m 755 -e
> 's;/etc/ant.conf;/opt/opkg/etc/ant/ant.conf;' -e
> 's;^\(ANT_LIB="\)[^"]*\("\);\1/opt/opkg/libexec/ant\2;' -e 's;\r$;;'
> bin/ant /opt/opkg/RPM/TMP/ant-1.6.2-root/opt/opkg/bin/
>
> i think, shtools's regex handles "\r" not as "RETURN" but just as "r".
> so -e 's;\r$;;' will delete every "r" at the end of a line -- making my
> .jar a .ja
>
> most of these invalid changes are in comments, a second critcal one
> seems to be in line 212:
> for i in "${OSXHACK}"/*.ja
Ohhhhhhh, yes. Now it is clear. Yes, the \r is certainly not
portable enough for sed(1). Fortunately it looks that the "ant"
script of the current version doesn't contain any \r characters
(CR) at all, so we can fix this problem by simple removing the
substitution. It is now fixed with ant-1.6.2-20041027 and higher.
See http://cvs.openpkg.org/chngview?cn=19886 for details. Thanks for
figuring out this subtle bug in the packaging of Ant.
Ralf S. Engelschall
[EMAIL PROTECTED]
www.engelschall.com
______________________________________________________________________
The OpenPKG Project www.openpkg.org
User Communication List [EMAIL PROTECTED]
