[Bug dynamic-link/30100] [patch] add dlmem()

["adhemerval.zanella at linaro dot org" <sourceware-bugzilla@sourceware.org>]

https://sourceware.org/bugzilla/show_bug.cgi?id=30100

--- Comment #4 from Adhemerval Zanella <adhemerval.zanella at linaro dot org> ---
(In reply to Stas Sergeev from comment #3)
> But as indicated above, I have problems running this patch
> through the test-suit. All built tests use the system-wide
> elf interpreter, so after I changed the "struct rtld_global_ro"
> I have to use patchelf on a test binaries to set the interpreter
> to the newly-built one.
> Please tell me how to solve that problem, because of course
> running through the test-suit is a requirement for submission
> to the mail list.
> 

The glibc testsuite has tests that either run through the built loader and
libraries (by invoking the loader directly with --library-path or by building
the tests to use the build loader when configure with
--enable-hardcoded-path-in-tests) or use a minimal container (where you can
override system-wide configuration). You can also test a binary with the glibc
loader by using the 'testrun.sh' script in the build folder.

The former is the default for tests, while the latter requires some extra care
(check the tests-container tests on Makefiles).

So first thing you would like to check is whether the new patch adds any
regression by running 'make check'.  Then you need to add any extra tests to
exercise the new feature, if they require changing any global state (such as
global files, etc.) it is preferable to provide them as tests-containers.

> Please let me know if there are any alternatives, too, like
> eg donating on patreon to someone who can do the remaining work.

-- 
You are receiving this mail because:
You are on the CC list for the bug.