
<html><head></head><body><div>Hi Peter,</div><div><span><pre><br></pre></span></div><div>On Mon, 2024-10-21 at 16:15 +0100, Peter Blackman wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Hi,<br></div><div><br></div><div>I'm thinking of fixing these.<br></div><div>The 47 current warnings break down into only 4 flavours.<br></div><div><br></div><div>1) hardening-no-pie<br></div><div>This is a direct consequence of disabling -pie on architectures<br></div><div>where fpc won't build that way. So makes sense to override these.<br></div></blockquote><div>I would rather not hide an obvious problem and let it be visible.</div><div>FPC has a real issue there, and we should not mask it.</div><div>I don't recommend this way of hiding issues.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>2) script-not-executable<br></div><div>These occur only in the lazarus-src package, where its unlikely that<br></div><div>its intended for these scripts to be run by end users. I suggest <br></div><div>overriding these too, rather than chmod them. After all, and end user<br></div><div>can trivially set the bit themselves should they (unlikely) wish to run <br></div><div>them.<br></div></blockquote><div>Generally, upstream takes patches for this kind of issue.</div><div>I would add a patch for changing permissions, include it in current patch set and forward it to upstream.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>3) duplicate-font-file<br></div><div>Genuine issue. Tempting to just remove them. Should the autopkgtest <br></div><div>fail, solution could be to depend on fonts-freefont-ttf and include <br></div><div>symlinks in aggpas directory.</div></blockquote><div>That is probably the best solution. We do the same for CGE.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>4) national-encoding<br></div><div>"Support for national encodings is being phased out.<br></div><div>Please convert the file to UTF-8 using iconv or a similar tool."<br></div><div><br></div><div>This would be a real problem for certain locales if the support <br></div><div>disappears. Could run iconv in dh_install.</div></blockquote><div>We should be careful here as Paul stated in his mail.</div><div>We also need to forward patches to upstream in order not to have to update them for each new release.</div><div style="caret-color: rgb(46, 52, 54); color: rgb(46, 52, 54); font-family: Cantarell; font-size: 14.666667px; font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-tap-highlight-color: rgba(0, 0, 0, 0.4); -webkit-text-stroke-width: 0px; text-decoration: none;"><span><pre>-- <br></pre><pre>Cheers,

Abou Al Montacir

</pre></span></div><br class="Apple-interchange-newline"><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"></blockquote></body></html>