vishesh92 commented on PR #14418:
URL: https://github.com/apache/cloudstack/pull/14418#issuecomment-6211987607

   > Points to consider:
   > 
   >     * Build JDK: the Build-Depends order puts openjdk-17-jdk ahead of 
openjdk-21-jdk. On a host with both installed, the build picks whichever apt 
resolves first. That’s probably fine, but it’s worth confirming which JDK the 
Debian 13 and Ubuntu 26.04 build actually uses.
   
   On distros that ship both (e.g. Ubuntu 24.04), a fresh install still pulls 
OpenJDK 17 since it's the first alternative, so existing setups see no change. 
OpenJDK 21 is only used when it's the only one available (Debian 13) or already 
installed.
   
   >     * Remaining mysql.connector users: tools/marvin/marvin/dbConnection.py 
and some tools/docker files still use mysql.connector directly. They are test 
tooling, so this is probably out of scope, but they’ll have the same problem on 
these distros.
   I don't think we need to make any changes for this in the PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to