<div dir="auto"><div dir="auto"><br></div><div dir="auto">On Tue, Jul 23, 2024, 6:39 PM Jan Beulich <<a href="mailto:jbeulich@suse.com">jbeulich@suse.com</a>> wrote:<br></div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 23.07.2024 12:24, H.J. Lu wrote:<br>
> On Tue, Jul 23, 2024, 6:19 PM Jan Beulich <<a href="mailto:jbeulich@suse.com" target="_blank" rel="noreferrer">jbeulich@suse.com</a>> wrote:<br>
> <br>
>> On 18.07.2024 11:04, Benjamin Drung wrote:<br>
>>> Specifying the compiler flag `-Wl,--package-metadata=<JSON>` might not<br>
>>> work, because the shells might eat the quotation marks and the compiler<br>
>>> splits the JSON at the commas.<br>
>>><br>
>>> Ubuntu tried to using a specs file to set `--package-metadata` but that<br>
>>> turned out to be too fragile. autopkgtests might use the compiler flags<br>
>>> but the needed environment variables are not set in the test<br>
>>> environment. Debugging a crash of an application build with the -spec<br>
>>> parameter lacks the environment variables. People like to iteratively<br>
>>> continue building the software in the build directory while hacking on<br>
>>> the package and then have no environment variable set.<br>
>>><br>
>>> So introduce a `--encoded-package-metadata` linker flag that takes a<br>
>>> percent-encoded JSON. Percent-encoding is used because it is a<br>
>>> standard and simple to implement.<br>
>>><br>
>>> Bug-Ubutru: <a href="https://bugs.launchpad.net/bugs/2071468" rel="noreferrer noreferrer" target="_blank">https://bugs.launchpad.net/bugs/2071468</a><br>
>>> Signed-off-by: Benjamin Drung <<a href="mailto:benjamin.drung@canonical.com" target="_blank" rel="noreferrer">benjamin.drung@canonical.com</a>><br>
>><br>
>> Okay. Please don't forget to add a reference to the PR that you created<br>
>> upon H.J.'s request. I notice H.J. has voiced a readability concern<br>
>> there, but I think that can be taken care of incrementally.<br>
> <br>
> I think we should make the existing option<br>
> usable instead of adding another option.<br>
<br>
Can you at least vaguely explain how you envision doing so, without any<br>
kind of risk of breaking existing uses?<br></blockquote></div><div dir="auto"><br></div><div dir="auto">See my comments in the bug report.</div><div dir="auto"><br></div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Jan<br><br>
</blockquote></div></div>