[PATCH] icu-config emits prefix with extra newline

[hv]

Having fetched, built and installed latest release icu library (configured with './runConfigureICU Linux --prefix=/opt/icu-4.2.0.1'), I then tried to configure parrot-1.2.0 with '/opt/perl-5.10.0/bin/perl Configure.pl --prefix=/opt/parrot-1.2.0 --icu-config=/opt/icu-4.2.0.1/bin/icu-config'. The result was an attempt to handle a path with a trailing newline, because of this:

zen% /opt/icu-4.2.0.1/bin/icu-config --prefix
/opt/icu-4.2.0.1

zen%

.. as a result of which parrot's configure emitted:

auto::icu -           Is ICU installed...Unsuccessful stat on filename containing newline at config/auto/icu.pm line 331.
Unsuccessful stat on filename containing newline at config/auto/icu.pm line 336.
...........................not found.

I'm not sure what guarantees *-config scripts are supposed to offer about output, but the to-be-attached patch works for me.

Hugo

[hv]

chop all trailing newlines on icu prefix

[Infinoid]

Hi! Thanks very much for the patch.

I think this is a duplicate of ticket #723, which had a similar patch and got applied a few days ago. So it should be fixed in the latest trunk revision. If you want to verify the fix, you can try a subversion checkout of parrot's "trunk" branch. If it is not fixed, please reply to this ticket and I will reopen it.

Thanks!

Mark

[hv]

Replying to Infinoid: Ah yes, that looks fine (though relying on two chomps in different places in the code seems a little fragile).

Thanks,

Hugo