Page MenuHomeFreeBSD

graphics/osgearth: Update to 2.7
ClosedPublic

Authored by woodsb02 on May 13 2016, 8:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 11:27 PM
Unknown Object (File)
Mon, Mar 18, 4:45 PM
Unknown Object (File)
Sun, Mar 3, 1:57 PM
Unknown Object (File)
Jan 20 2024, 12:46 AM
Unknown Object (File)
Jan 12 2024, 3:09 PM
Unknown Object (File)
Dec 25 2023, 7:07 AM
Unknown Object (File)
Dec 25 2023, 7:07 AM
Unknown Object (File)
Dec 25 2023, 7:07 AM
Subscribers

Details

Summary

graphics/osgearth:

  • Update to 2.7 (this new version also allows osgearth to build against osg-3.4.0)
  • Rename and update patch of src/osgEarth/ThreadingUtils.cpp (pet portlint)
  • Add patch to removed unnecessary FirstPersonManipulator include from osgearth_skyview [1]

Changes this release:
http://docs.osgearth.org/en/latest/releasenotes.html#version-2-7-july-2015

[1] https://github.com/gwaldron/osgearth/commit/bd36290687f6884c17c622ca57442e60c1ec50b8#diff-091cce4318b0ef98b4ce05f80e3d0762

PR: 209236
Approved by: lbartoletti@tuxfamily.org (maintainer), xxx (mentor)

Test Plan

poudriere testport graphics/osgearth

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 3723
Build 3766: arc lint + arc unit

Event Timeline

woodsb02 retitled this revision from to graphics/osgearth: Update to 2.7.
woodsb02 updated this object.
woodsb02 edited the test plan for this revision. (Show Details)
woodsb02 added reviewers: adamw, mat, koobs.
graphics/osgearth/Makefile
14

This should be minizip:archivers/minizip.

graphics/osgearth/distinfo
1–3

Please update your ports tree and run make makesum again.

As per comments from mat:

  • fix BUILD_DEPENDS for minizip
  • redo makepatch
graphics/osgearth/Makefile
6–7

You should keep the DISTVERSIONPREFIX line, the 2.7 bit is a branch name, and not a tag, so it's going to move, and by doing so, breaking this port.

As per comments from mat:

  • keep DISTVERSIONPREFIX=${PORTNAME}- to prevent the port breaking in the future

Also:

  • Remove skyview patch as it is not needed with DISTVERSIONPREFIX
  • Adjust pkg-plist to suit port with DISTVERSIONPREFIX
  • Fix COMMENT (pet portlint)
graphics/osgearth/Makefile
6–7

TIL. In fact, I have put it back in, but not sure I fully understand. I tried to read up about the GitHub download URL format, and whilst there was quite a lengthy discussion about old vs new APIs, there was not much mention of branches. So, can I please ask for an explanation of why we want the filename to be gwaldron-osgearth-osgearth-2.7_GH0.tar.gz, and what that means for branches vs tag names?

graphics/osgearth/Makefile
6–7

The branch name is "2.7". When upstream commits to that branch, the branch names stays the same but the tarball changes, making checksum fail after every commit.

The tag name is "osgearth-2.7". As long as the tag isn't slipped (which is very bad behaviour), the tarball is fixed.

GitHub's API is really flexible. If you give it a branch name, a tag name, or a commit hash, it'll send you a tarball of that thing. There's no way to tell it which one of those you wanted (which is okay, because those names can't overlap anyway). The tarballs names look strange but that was an intentional choice to ensure that distfiles could never conflict.

Does that answer the question?

woodsb02 added inline comments.
graphics/osgearth/Makefile
6–7

Ah yes, I follow now. Sorry, it took a little while to click, but I now see that the tags/releases on the osgearth GitHub page are not labelled "2.7" but instead "osgearth-2.7", and therefore without the DISTVERSIONPREFIX it was not actually downloading the release, but the latest commit of the 2.7 branch! Thanks.

Its all done, and this should be ready to commit.

adamw edited edge metadata.
This revision is now accepted and ready to land.May 14 2016, 3:17 PM
This revision was automatically updated to reflect the committed changes.
woodsb02 marked an inline comment as done.