Skip to content

Fix for linux#145

Open
Adisk06 wants to merge 3 commits intogatech-csl:masterfrom
Adisk06:master
Open

Fix for linux#145
Adisk06 wants to merge 3 commits intogatech-csl:masterfrom
Adisk06:master

Conversation

@Adisk06
Copy link
Copy Markdown

@Adisk06 Adisk06 commented Nov 11, 2024

jes.sh script rewrite + minor changes to make it usable on linux for students

1. Automatic Java Detection

- The script now automatically searches for the appropriate Java installation on the system, removing the need for manual configuration.
- It first checks for the presence of Java 8 (Java 1.8) and sets the JAVA_HOME environment variable accordingly.

2. Custom Java Directory Support

- Users can now specify a custom Java installation path by creating a file named java_path in the root directory of the JES installation.
- This file should contain the full path to the desired Java installation, allowing for easy management of the Java version used by JES without modifying the system's global JAVA_HOME.

3. Line Ending Compatibility

- The original jes.sh script required manual end-of-line conversion (using dos2unix) to be usable on Linux, as it was designed with Windows-style line endings (CRLF).
- This version of the script eliminates the need for such conversion, ensuring compatibility with Linux systems that expect Unix-style line endings (LF).
Changes
- It includes specific versioning, making it easier to track the version of JES.
- It validates the presence of the icon file and provides error messages when necessary, preventing broken links.
- It is clearer and more consistent in structure, with improved error handling and better user feedback.
- It provides a more user-friendly experience by hardcoding the application title (JES), while ols version had unnecessary complexity with dynamic placeholders.
added explanation to how to add java 8 path for jes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant