Re: pkg/43187 (icu-config --version under solaris prints -n 4.2.1)

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

The following reply was made to PR pkg/43187; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg%britannica.bec.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/43187 (icu-config --version under solaris prints -n 4.2.1)
Date: Tue, 3 Apr 2012 10:40:37 +0900

 On Mon, Apr 02, 2012 at 04:45:04PM +0000, Ryo ONODERA wrote:
 > The following reply was made to PR pkg/43187; it has been noted by GNATS.
 > 
 > From: Ryo ONODERA <ryo_on%yk.rim.or.jp@localhost>
 > To: gnats-bugs%NetBSD.org@localhost, uklaus%hgb-leipzig.de@localhost
 > Cc: 
 > Subject: Re: pkg/43187 (icu-config --version under solaris prints -n 4.2.1)
 > Date: Tue, 03 Apr 2012 01:41:26 +0900 (JST)
 > 
 >  From: Uwe Klaus <uklaus%hgb-leipzig.de@localhost>, Date: Mon,  2 Apr 2012 
 > 16:05:06 +0000 (UTC)
 >  
 >  > The following reply was made to PR pkg/43187; it has been noted by GNATS.
 >  > 
 >  > From: Uwe Klaus <uklaus%hgb-leipzig.de@localhost>
 >  > To: gnats-bugs%NetBSD.org@localhost
 >  > Cc: solaris-pkg-people%NetBSD.org@localhost, 
 > pkgsrc-bugs%NetBSD.org@localhost,
 >  >         gnats-admin%NetBSD.org@localhost, ryoon%NetBSD.org@localhost
 >  > Subject: Re: pkg/43187 (icu-config --version under solaris prints -n 
 > 4.2.1)
 >  > Date: Mon, 2 Apr 2012 18:01:59 +0200 (CEST)
 >  > 
 >  >  On Sat, 24 Mar 2012, ryoon%NetBSD.org@localhost wrote:
 >  >  
 >  >  > Synopsis: icu-config --version under solaris prints -n 4.2.1
 >  >  >
 >  >  > State-Changed-From-To: open->feedback
 >  >  > State-Changed-By: ryoon%NetBSD.org@localhost
 >  >  > State-Changed-When: Sat, 24 Mar 2012 17:01:06 +0000
 >  >  > State-Changed-Why:
 >  >  > At least on Solaris/i386 10, icu-4.8.1nb2's icu-config does not have 
 > echo -n.
 >  >  > Then this problem is not reproduced.
 >  >  > Could you test again with current pkgsrc tree?
 >  >  "icu-config --version" prints now
 >  >  4.8.1
 >  >  
 >  >  the version number with one additional newline.
 >  
 >  I see.
 >  The following patch will solve new problem.
 >  It looks ok, I will commit it after the freeze.
 >  
 >  Could you test the patch?
 >  
 >  Thank you.
 >  
 >  $NetBSD$
 >  
 >  --- config/icu-config-bottom.orig   2011-07-19 21:16:46.000000000 +0000
 >  +++ config/icu-config-bottom
 >  @@ -352,11 +352,11 @@ do
 >          ;;
 >   
 >      --version)
 >  -        echo $ECHO_N $VERSION
 >  +        echo $ECHO_N $VERSION ${ECHO_C}
 >           ;;
 >   
 >       --unicode-version)
 >  -        echo $ECHO_N $UNICODE_VERSION
 >  +        echo $ECHO_N $UNICODE_VERSION ${ECHO_C}
 >           ;;
 >   
 >      --help)
 
 ...or just use printf "%s".
 
 Joerg