<div dir="auto"><div class="gmail_extra" dir="auto"><div class="gmail_quote">El 15 nov. 2017 8:00 a.m., "Ximin Luo" <<a href="mailto:infinity0@debian.org">infinity0@debian.org</a>> escribió:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Pino Toscano:<br>
<div class="quoted-text">> In data martedì 14 novembre 2017 11:14:00 CET, Ximin Luo ha scritto:<br>
>> You're using __FILE__ inappropriately, none of the documentation<br>
>> guarantees or implies that you can access __FILE__ as a real<br>
>> filesystem path. "Surely for relative paths" is your justification<br>
>> for your own broken behaviour.<br>
><br>
> Again, this is your own echo chamber: "__FILE__ is broken, everybody<br>
> using it is broken no matter what".<br>
><br>
<br>
</div>It's not my "own echo chamber", I pointed you to lots of docs that confirm your usage of __FILE__ is not appropriate here.<br>
<br>
Adrian Bunk also mentioned the C89 to me on IRC yesterday. Again, the wording gives no guarantee that __FILE__ should represent a real filesystem path.<br>
<br>
If my previous words were a bit terse I am sorry for that, but I don't appreciate comments like "Any of your solutions get a big, fat, and giant nope" after trying to explain the problem and present you with no less than 4 alternative simple unintrusive solutions.<br>
<div class="quoted-text"><br>
>> You can either accept my one line patch suggestion, or fix it some<br>
>> other way.<br>
><br>
> I am not interested in working around broken changes introduced blindly<br>
> for very doubtful reasons.<br>
><br>
>> I'm not going to change the GCC patch, it does nothing wrong.<br>
><br>
> Let me add also another POV to this approach: do you really expect<br>
> Debian to carry this important diversion for GCC upstream?  I really<br>
> doubt GCC will accept this.<br>
><br>
<br>
</div>I'm in the process of getting the patch into GCC. We certainly don't intend to keep this as a Debian-specific thing. [1] If they don't accept it, you don't need to patch your package - but I'd say the use of __FILE__ is still not the best, since no documentation implies it can be used to find files, and all the examples only mention error messages.<br>
<br>
And to be clear, in case Holger's earlier messages were missed, the FTBFS is currently only on <a href="http://tests.r-b.org" rel="noreferrer" target="_blank">tests.r-b.org</a> and not on the official Debian archive.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I see your point here. I'm currently on the phone, please allow me some time to get into a proper keyboard and make a. Proposals that I think will work for both sides.</div><div class="gmail_extra" dir="auto"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div><br></div></div>