Page MenuHomeFreeBSD

astro/google-earth: Update to 7.1.5.1557
ClosedPublic

Authored by woodsb02 on Aug 28 2016, 1:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 5, 9:11 PM
Unknown Object (File)
Nov 21 2024, 4:54 PM
Unknown Object (File)
Oct 31 2024, 5:14 AM
Unknown Object (File)
Oct 31 2024, 5:13 AM
Unknown Object (File)
Oct 20 2024, 3:55 PM
Unknown Object (File)
Oct 3 2024, 5:50 PM
Unknown Object (File)
Oct 3 2024, 1:18 PM
Unknown Object (File)
Oct 3 2024, 2:58 AM
Subscribers
None

Details

Summary

astro/google-earth: Update to 7.1.5.1557

  • Switch to USES=makeself to minimize Makefile
  • Improve COMMENT and pkg-descr, to clarify exactly what this port provides
  • Do not set WRKSRC, as the default works just fine
  • Fix whitespace at the end of LINUX_DRI_DESC (pet portlint)
  • Install files with COPYTREE_SHARE instead of CP, to ensure permissions are set correctly
  • Remove redundant -f after ${RM}

Changes this release:

http://www.gearthblog.com/blog/archives/2015/05/new-version-google-earth-7-1-5.html
Test Plan

poudriere testport astro/google-earth

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

woodsb02 retitled this revision from to astro/google-earth: Update to 7.1.5.1557.
woodsb02 updated this object.
woodsb02 edited the test plan for this revision. (Show Details)
woodsb02 added reviewers: adamw, mat, koobs.
astro/google-earth/Makefile
37 ↗(On Diff #19756)

tar arguments should always have dashes. If users have ${TAR} redefined to anything else, using bare xf will cause it to break.

As an example:

$ tar -v xf foo.tar
tar: Must specify one of -c, -r, -t, -u, -x

While here, you can reduce the compound into

${TAR} -C ${WRKSRC} -xf google[...]
astro/google-earth/Makefile
67 ↗(On Diff #19756)

Changing from CP to COPYTREE_SHARE is a great step.

Just make sure there are no files that need to be executable or something in there, like, maybe, googleearth :-)

  • Ensure ${TAR} arguments have dashes
  • Ensure binaries are executable by installing with ${INSTALL_PROGRAM}
woodsb02 added inline comments.
astro/google-earth/Makefile
37 ↗(On Diff #19756)

I did add the dashes, but I did not compound this because it would need to be longer.

Instead of:
@cd ${WRKSRC} && ${TAR} -xf googleearth-linux-x86.tar && ${RM} googleearth-linux-x86.tar

It would have to be:
${TAR} -C ${WRKSRC} -xf ${WRKSRC}/googleearth-linux-x86.tar && ${RM} ${WRKSRC}/googleearth-linux-x86.tar

mat edited edge metadata.

Looks good to me

This revision is now accepted and ready to land.Aug 29 2016, 1:31 PM
This revision was automatically updated to reflect the committed changes.
woodsb02 marked an inline comment as done.