[PATCH] Update RPATH to make loading libjawt possible

Anthony Petrov anthony.petrov at oracle.com
Tue Aug 14 10:23:44 UTC 2012


I see your point. As I told you we'll be open-sourcing a JAWT-specific 
test under 7190587. This test already provides support for Linux, 
Solaris, and Windows platforms, and even does a little more than just 
loads the jawt library.

So it looks there's still some value in Kumar's test as well. Yes, it 
will need to be rewritten should we replace the RPATH mechanism with 
something else in the future. It's up to Kumar to decide whether he 
wants such test to be present in the repo. Personally, I don't mind.

--
best regards,
Anthony

On 8/13/2012 7:04 PM, Omair Majid wrote:
> On 08/13/2012 10:39 AM, Anthony Petrov wrote:
>> The test looks great, and I like that it doesn't depend on the JAWT
>> machinery, but tests the actual problematic RPATH entry only.
> 
> I generally go for tests that verify behaviour (that jawt-linked
> programs are working) rather than implementation details (which changed,
> for example, when we switched from LD_LIBRARY_PATH to RPATHS).
> 
>> +1 from me.
> 
> Yes, good to have something that guards us from changing something
> unintentionally. Looks fine to me too.
> 
> Cheers,
> Omair



More information about the build-dev mailing list