Changes between Initial Version and Version 1 of Ticket #22088


Ignore:
Timestamp:
2022-05-25T16:16:55+02:00 (4 years ago)
Author:
taylor.smock
Comment:

attachment:22088.patch​ does the following:

  • Ensure that the java command will be executable (L43​)
  • Makes the script more resistant to missing links or executables (dpkg and /etc/alternatives/java specifically)

This will hopefully allow us to encourage downstream distributors to use the start script, with minor modifications (hopefully just L71​).

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #22088

    • Property Milestone → 22.06
    • Property Summary Debian start script does not properly check if a path does not exist or is not executable → [RFC PATCH] Debian start script does not properly check if a path does not exist or is not executable

  • Ticket #22088 – Description

    initial v1  
    11I've been working on getting downstreams to use our start script (with some ''minor'' patches) to hopefully avoid future issues where we get bug reports that we've fixed in the start script.
     2
     3Steps to reproduce (on Debian/Ubuntu):
     41. If `/usr/lib/jvm/default-java/bin/java` exists, move it (example: `mv /usr/lib/jvm/default-java/bin/java /usr/lib/jvm/default-java/bin/java.bak`)
     52. Attempt to run the start script
     63. If you moved `/usr/lib/jvm/default-java/bin/java` in step 1, move it back (example: `mv /usr/lib/jvm/default-java/bin/java.bak /usr/lib/jvm/default-java/bin/java`)
     7
     8You may also be able to reproduce by unsetting `JAVA_HOME` or setting it to an empty string (e.g., `JOSM_HOME="" ./josm`)
    29
    310From https://github.com/flathub/org.openstreetmap.josm/issues/55,