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:

  1. Tidy up
    1. Normalise whitespace - replace tabs with spaces.
    2. Use absolute instead of relative path for symlink to executable. Looking through my /opt/local/bin, this seems to be the convention in MacPorts.
    3. 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.
    4. Add license info.
  2. Update to version 3.4
    1. Update version and distname, replace checksums with types advised in guide.
    2. Delete / move newly added files in distribution.
    3. 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)

Portfile-3.4 (3.2 KB) - added by steve+macports@… 10 years ago.
Portfile-launch4j-whitespace.diff (4.7 KB) - added by steve+macports@… 10 years ago.
Portfile-launch4j-3.4.diff (4.0 KB) - added by steve+macports@… 10 years ago.

Download all attachments as: .zip

Change History (10)

Changed 10 years ago by steve+macports@…

Attachment: Portfile-3.4 added

comment:1 Changed 10 years ago by steve+macports@…

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.

comment:2 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.

comment:3 in reply to:  2 Changed 10 years ago by steve+macports@…

OK thanks, will do.

Changed 10 years ago by steve+macports@…

Changed 10 years ago by steve+macports@…

Attachment: Portfile-launch4j-3.4.diff added

comment:4 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

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: newclosed

In b29d8a11e93079a89d8d789369fcdef3a6fd75c2/macports-ports (master):

launch4j: update to 3.12

Fixes: #44197

Note: See TracTickets for help on using tickets.