RFR: 8297963: Partially fix string expansion issues in the autoconf UTIL macros [v3]
Magnus Ihse Bursie
ihse at openjdk.org
Fri Dec 2 14:40:05 UTC 2022
On Fri, 2 Dec 2022 14:06:17 GMT, Julian Waters <jwaters at openjdk.org> wrote:
>> make/autoconf/util.m4 line 70:
>>
>>> 68: # Proceed at the risk of your own sanity, and don't say I didn't warn you when you inevitably suffer the same torment that I went through should you decide
>>> 69: # that you need to dive into it to fix anything
>>> 70: # ~Julian
>>
>> 🤣 While I do appreciate humor and a bit of personal touch in comments (and I fully sympathize with your description!), this feels a bit over the top. Can you tone it down a bit and make it slightly more professional (it is okay to keep a bit of the sentimentality I think, to fully make the point on how difficult it is to modify).
>
> Haha, sorry for letting the pent up bitterness overflow into the comments, I'll do just that. I've lost track of how much coffee and milk I've consumed while trying to get this to work...
> Probably enough to shorten my lifespan by several decades at this point, if I'm being honest
I made like two or three false starts at even getting this macro to work in the first place, after thinking "oh, that would be a good idea", trying different solutions, thinking "nah, it is impossible" to coming back to it later again with a new approach. And even with the correct approach it was basically a lot of trial and horror of permutating different m4 keywords until you got the wanted result... So I fully understand what you have been going through. At least you can call yourself "hard-core m4 programmer" now as a result. 🎖️
-------------
PR: https://git.openjdk.org/jdk/pull/11458
More information about the build-dev
mailing list