Address review feedback

- Consolidate error logging into single line
- Add MINIMUM_GLIBC_VERSION constant
- Handle version strings without dots in comparison logic

Author: Anubhav Singh

app/src/main/java/org/hyperledger/besu/cli/ConfigurationOverviewBuilder.java

@@ -69,6 +69,7 @@ public class ConfigurationOverviewBuilder {
   private boolean isParallelTxProcessingEnabled = false;
   private RocksDBCLIOptions.BlobDBSettings blobDBSettings;
   private Long targetGasLimit;
+  private static final String MINIMUM_GLIBC_VERSION = "2.28";
 
   /**
    * Create a new ConfigurationOverviewBuilder.
@@ -576,22 +577,17 @@ private void detectJemalloc(final List<String> lines) {
    * @param glibcVersion the detected glibc version
    */
   private void checkGlibcVersion(final String glibcVersion) {
-    if (glibcVersion == null) {
-      // Cannot determine version, log warning but allow startup
-      logger.warn("Unable to determine glibc version. Minimum required version is 2.28");
-      return;
-    }
+  if (glibcVersion == null) {
+    logger.warn("Unable to determine glibc version. Minimum required version is {}", MINIMUM_GLIBC_VERSION);
+    return;
+  }
 
-    final String minVersion = "2.28";
-    if (!isGlibcVersionSufficient(glibcVersion, minVersion)) {
-      logger.error("Insufficient glibc version detected.");
-      logger.error("Required: {} or higher", minVersion);
-      logger.error("Found: {}", glibcVersion);
-      logger.error("Please upgrade your system's glibc to version {} or higher.", minVersion);
-      throw new RuntimeException(
-          "Besu requires glibc version " + minVersion + " or higher. Found: " + glibcVersion);
-    }
+  if (!isGlibcVersionSufficient(glibcVersion, MINIMUM_GLIBC_VERSION)) {
+    logger.error("Insufficient glibc version detected. Required: {} or higher, Found: {}. Please upgrade your system's glibc.", MINIMUM_GLIBC_VERSION, glibcVersion);
+    throw new RuntimeException(
+        "Besu requires glibc version " + MINIMUM_GLIBC_VERSION + " or higher. Found: " + glibcVersion);
   }
+}
 
   /**
    
@@ -603,27 +599,29 @@ private void checkGlibcVersion(final String glibcVersion) {
    */
 
   private boolean isGlibcVersionSufficient(final String current, final String required) {
-    try {
-      final String[] currentParts = current.split("\\.");
-      final String[] requiredParts = required.split("\\.");
-
-      for (int i = 0; i < Math.min(currentParts.length, requiredParts.length); i++) {
-        final int currentNum = Integer.parseInt(currentParts[i].trim());
-        final int requiredNum = Integer.parseInt(requiredParts[i].trim());
-
-        if (currentNum > requiredNum) {
-          return true;
-        }
-        if (currentNum < requiredNum) {
-          return false;
-        }
+  try {
+    final String[] currentParts = current.split("\\.");
+    final String[] requiredParts = required.split("\\.");
+    
+    final int compareLength = Math.max(currentParts.length, requiredParts.length);
+
+    for (int i = 0; i < compareLength; i++) {
+      final int currentNum = i < currentParts.length ? Integer.parseInt(currentParts[i].trim()) : 0;
+      final int requiredNum = i < requiredParts.length ? Integer.parseInt(requiredParts[i].trim()) : 0;
+
+      if (currentNum > requiredNum) {
+        return true;
+      }
+      if (currentNum < requiredNum) {
+        return false;
       }
-      return true; // Versions are equal
-    } catch (final NumberFormatException e) {
-      logger.warn("Unable to parse glibc version: {}", current, e);
-      return true; // Assume sufficient if unable to parse
     }
+    return true;
+  } catch (final NumberFormatException e) {
+    logger.warn("Unable to parse glibc version: {}", current, e);
+    return true;
   }
+}