Various fixes #1579
Various fixes #1579
Conversation
…able paths in "build_then_launch.sh" (paths are case-sensitive on Linux et al.); Added note about requiring Java 17 to top-level "readme.md"; Cleaned-up and corrected examples in top-level "readme.md".
readme.md
Outdated
|
|
||
| * Build the latest version for the platform you are running on: | ||
|
|
||
| `mvn clean verify -DskipTests=true` |
There was a problem hiding this comment.
NOSDK is best here
The SDK is an option to build a eclipse instance to do java development on sloeber
readme.md
Outdated
| `mvn clean verify -Plinux64,latest,NOSDK -DskipTests=true` | ||
|
|
||
| * Build Eclipse + Sloeber for 32-bit Linux: | ||
| `mvn clean verify -Plinux32,latest,NOSDK -DskipTests=true` |
There was a problem hiding this comment.
32 bit eclipse doesn't work anymore.
As you are on it. Can you remove these 32 bit references?
There was a problem hiding this comment.
Sure thing. I saw the website pages are also in this repository so I'll remove the chmod note from the Linux download page as well (can't test that the download page handles tar.gz correctly though since I don't have PHP installed). I'll add a commit with all the additional stuff.
There was a problem hiding this comment.
You can leave download links on the web pages to me. The download links to the web pages are based on files and only updated when I make an official release.
|
I'm currently making a release and though the local build on my system provides a tar file for linux the actions here provide a zip. |
|
It seems the actions recreates the files |
tar.gzFixes issue 1577
Fixes issue 1557
When using
tar.gzinstead ofzipthe Linux file permissions are preserved (i.e.sloeber-ide,jexec, andjspawnhelperare all executable).The download page at http://eclipse.baeyens.it/stable.php?OS=Linux also needs to be updated with the new filename. The text saying to run
chmodcan be removed.build_then_launch.sh(paths are case-sensitive on Linux et al.)readme.mdThis fixes issue 1544
readme.mdSeveral of the examples had
latest.NOSDK(with a period) instead oflatest,NOSDK(with a comma).