Opened 10 years ago
Closed 6 years ago
#44197 closed update (fixed)
launch4j: update to 3.4 and tidy portfile
Reported by: | steve+macports@… | Owned by: | panayotis@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | kurthindenburg (Kurt Hindenburg) | |
Port: | launch4j |
Description
launch4j is now at version 3.4 upstream.
I'm attaching a proposed Portfile that includes these changes:
- Tidy up
- Normalise whitespace - replace tabs with spaces.
- Use absolute instead of relative path for symlink to executable. Looking through my /opt/local/bin, this seems to be the convention in MacPorts.
- Reorganise modifications to extracted files and add comments. First delete the various groups of unwanted files, then move the documentation into place, then install the program files.
- Add license info.
- Update to version 3.4
- Update version and distname, replace checksums with types advised in guide.
- Delete / move newly added files in distribution.
- Remove patchfile and instead use reinplace to set the absolute path to launch4j.jar in launch4j script. The upstream script looks for the jar in the same directory as the script, which won't work when the script is executed via a link. The patchfile (which no longer works against version 3.4) fixed this for the symlink, but broke the real script. An alternative is to use a wrapper script (that just execs the real script) instead of a symlink, as used by clang-3.x for example.
I'm new here, so I'd appreciate some guidance on whether to provide one big diff, or break my changes up into several patches and attach them here, or to split some of this off into a separate ticket. Cheers.
Attachments (3)
Change History (10)
Changed 10 years ago by steve+macports@…
Attachment: | Portfile-3.4 added |
---|
comment:1 Changed 10 years ago by steve+macports@…
comment:2 follow-up: 3 Changed 10 years ago by mf2k (Frank Schima)
Cc: | panayotis@… removed |
---|---|
Owner: | changed from macports-tickets@… to panayotis@… |
Version: | 2.3.1 |
Per the guidelines, please instead attach a unified diff of the Portfile so we can easily see what changes you are proposing. Please keep whitespace changes in a separate patch from functional changes.
Changed 10 years ago by steve+macports@…
Attachment: | Portfile-launch4j-whitespace.diff added |
---|
Changed 10 years ago by steve+macports@…
Attachment: | Portfile-launch4j-3.4.diff added |
---|
comment:5 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)
The whitespace diff committed r126184
The other patch still applies cleanly.
comment:6 Changed 6 years ago by amake (Aaron Madlon-Kay)
This ticket is potentially obsoleted by https://github.com/macports/macports-ports/pull/4286
comment:7 Changed 6 years ago by Aaron Madlon-Kay <amake@…>
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Note that the copies of ld and windres bundled with launch4j have changed between versions 3.0.2 and 3.4. They were previously ppc-only, and are now x86_64. To get launch4j 3.0.2 to work on x86, I replaced those files with symlinks to the ones provided by i386-mingw32-binutils. This workaround might also work for anyone wanting to use launch4j 3.4 on ppc.