RFR: 8345169: Implement JEP 503: Remove the 32-bit x86 Port

Magnus Ihse Bursie ihse at openjdk.org
Fri Mar 7 15:10:57 UTC 2025


On Fri, 7 Mar 2025 07:21:43 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> It's no that important, no. I'm not sure if previous deprecated ports were handles exactly like this. 
>> 
>> And you can always do like `git log | grep -i "remove .* port"` to find the change it was removed in, and look what it did...
>
> I think leaving a comment describing how to deprecate a port is useful. To look it up in history you have to realise there is something to look up.
> 
> "They who are not reminded of the past will invent a new way to do it in the future."

The `--enable-deprecated-ports` is still there. All that is removed is an if statement and a print line. I know the make syntax can seem intimidating, but just ask me or any other build team member if you need help to recreate such a thing. It is not like it is a complicated algorithm that can be written in many ways. This is just make's equivalant of:


if (some_condition) {
  println("whatever");
}


To me this is just utter nonsense to keep that commented out.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23906#discussion_r1985229429


More information about the build-dev mailing list