Opened 9 years ago

Closed 9 years ago

#48367 closed update (fixed)

Update jpegoptim to 1.4.3

Reported by: bananastalktome@… Owned by: jingoro@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: kurthindenburg (Kurt Hindenburg)
Port: jpegoptim

Description

Any interest in updating this port from 1.2.4 to the latest release (1.4.3)? I was able to do it with a local portfile, but I had to change destroot.destdir INSTALL_ROOT=${destroot} to destroot.destdir DESTDIR=${destroot} in the Portfile for it to work ("INSTALL_ROOT" is no longer recognized by the jpegoptim configuration, replaced with "DESTDIR").

I've attached an updated Portfile (with new destdir and checksums for 1.4.3).

Attachments (2)

Portfile (973 bytes) - added by bananastalktome@… 9 years ago.
Portfile for version 1.4.3
Portfile-jpegoptim.diff (860 bytes) - added by bananastalktome@… 9 years ago.
Portfile diff for version 1.4.3

Download all attachments as: .zip

Change History (10)

Changed 9 years ago by bananastalktome@…

Attachment: Portfile added

Portfile for version 1.4.3

comment:1 Changed 9 years ago by bananastalktome@…

Cc: bananastalktome@… added

Cc Me!

comment:2 Changed 9 years ago by Ionic (Mihai Moldovan)

If DESTDIR is the new variable, then you can drop it completely because DESTDIR is the default way of doing this.

comment:3 Changed 9 years ago by Ionic (Mihai Moldovan)

Cc: bananastalktome@… removed

Please do not CC yourself. Reporters automatically get notifications.

Also, if possible please provide diffs against the current Portfile, not a complete file.

Changed 9 years ago by bananastalktome@…

Attachment: Portfile-jpegoptim.diff added

Portfile diff for version 1.4.3

comment:4 Changed 9 years ago by bananastalktome@…

Sorry, I've added a diff now (after validating it still works with the removal of DESTDIR). Thanks!

comment:5 Changed 9 years ago by mf2k (Frank Schima)

Thanks! The revision line should be deleted because it starts at 0 when increasing the version and that is the default value.

comment:6 Changed 9 years ago by Ionic (Mihai Moldovan)

Owner: changed from macports-tickets@… to jingoro@…

Oh, and please CC the maintainer next time around. Assigning to him.

comment:7 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:8 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

thanks r141721

Note: See TracTickets for help on using tickets.