[PATCH] Configure.pl disable-rpath option

[gerd]

The disable-rpath option works fine in the file config/inter/libparrot.pm.

But rpath is added in config/init/defaults.pm from the Perl 5 configuration. This patch filter it out if disable-rpath is set.

[gerd]

generated with "diff -u config/init/defaults.pm config/init/defaults.pm.new

[jkeenan]

Replying to gerd:

The disable-rpath option works fine in the file config/inter/libparrot.pm. But rpath is added in config/init/defaults.pm from the Perl 5 configuration. This patch will filter it out if disable-rpath is set.

Gerd,

Thanks for your patch. All I know about rpath I got just now from reading the  Wikipedia article. So, I'm not the best person to evaluate your patch. (Am cc-ing Andy D for that :-) )

But do I correctly understand your point? In other words, if we're going to use the --disable-rpath option, we should very early on make sure we prevent it from getting into the link_dynamic attribute in the Parrot::Configure object at all?

Also, you will have to add --disable-rpath to the list of valid options in (IIRC) lib/Parrot/Configure/Options/Conf/Shared.pm, add it to print_help() in lib/Parrot/Configure/Options/Conf.pm and write some brief documentation of it in Configure.pl.

Thank you very much.

kid51

[allison]

We have a --disable-rpath option, added in r37823 for packagers. Most versions of Perl don't add -rpath to the ccdlflags configuration variable. Where they do, we don't need it anyway since that -rpath is relevant to dynamically linking to Perl 5 (which we never do). We really don't want it when the user has specifically requested disable-rpath.

Applied in r40098.

[gerd]

It works on Fedora and SuSE (Michael Schroeder should agree). The ticket can be closed.

Thanks

[doughera]

Agreed. Actually, I'd go farther -- I can't think of any situation in which parrot ought to keep perl5's rpath setting. I think it could be safely deleted completely independently of any disable-rpath option.