Page MenuHomeFreeBSD
Differential D9170

audio/teamspeak3-server: unbreak on 11-STABLE
ClosedPublic
Actions

Authored by lifanov on Jan 13 2017, 9:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 9:25 PM
Unknown Object (File)
Sat, Nov 23, 1:54 AM
Unknown Object (File)
Thu, Nov 21, 2:13 PM
Unknown Object (File)
Thu, Nov 21, 2:12 PM
Unknown Object (File)
Thu, Nov 21, 2:11 PM
Unknown Object (File)
Thu, Nov 21, 11:52 AM
Unknown Object (File)
Thu, Nov 21, 6:06 AM
Unknown Object (File)
Wed, Nov 20, 7:35 PM
View All Files
Subscribers
koobs
mat

Details

Reviewers
matthew
Commits
rP431501: audio/teamspeak3-server: unbreak on 11-STABLE
Summary
audio/teamspeak3-server: unbreak on 11-STABLE

 PR: 212237
 Reported by: Jonathan Price <freebsd@jonathanprice.org>
 Submitted by: ultima1252@gmail.com (maintainer)
 Reviewed by: matthew
 Approved by: matthew (mentor)
 Differential Revision: https://reviews.freebsd.org/D9170

Diff Detail

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

Event Timeline

lifanov updated this revision to Diff 23974.Jan 13 2017, 9:40 PM
lifanov retitled this revision from to audio/teamspeak3-server: unbreak on 11-STABLE.
lifanov updated this object.
lifanov edited the test plan for this revision. (Show Details)
lifanov added a reviewer: matthew.
lifanov added a subscriber: koobs.
Herald added a subscriber: mat. · View Herald TranscriptJan 13 2017, 9:40 PM
mat added a comment.Jan 13 2017, 10:16 PM

Have you tested this ? Because it cannot possibly work, OSVERSION is only defined after bsd.port.options.mk is included. (Testing in poudriere won't work, it defines OSVERSION in the environement to fake things up.)

You need to move this lower in the Makefile and change it to something like:

.if OSVERSION > 110000 && OSVERSION < <your version>
BROKEN=
.endif
matthew edited edge metadata.Jan 13 2017, 10:31 PM

Missing the PR number?

audio/teamspeak3-server/Makefile
23 ↗(On Diff #23974)

This doesn't work -- it's too early to reference ${OSVERSION} here. Has to be after the inclusion of bsd.ports.pre.mk or bsd.port.options.mk

Also, probably better to use the style:

.if ${OPSYS} == FreeBSD && ${OSVERSION} >= 1100000 && ${OSVERSION} < 1100507
BROKEN=Will not start on FreeBSD 11, Bug 212493
.endif

rather than mixing an explicit test against OSVERSION with the 'BROKEN_FreeBSD_11' form.

matthew added a comment.Jan 13 2017, 10:33 PM
In D9170#189493, @mat wrote:

Have you tested this ? Because it cannot possibly work, OSVERSION is only defined after bsd.port.options.mk is included. (Testing in poudriere won't work, it defines OSVERSION in the environement to fake things up.)

You need to move this lower in the Makefile and change it to something like:

.if OSVERSION > 110000 && OSVERSION < <your version>
BROKEN=
.endif

The port is marked restricted, so it wouldn't build under poudriere even if it was syntactically correct.

lifanov updated this revision to Diff 23989.Jan 14 2017, 3:08 PM
lifanov edited edge metadata.

fix and switch to a single version check mechanism

I didn't test this previously because of the RESTRICTED, but I tested the version just now with LICENSE* and RESTRICTED commented out and it works.

lifanov updated this object.Jan 14 2017, 3:13 PM
matthew accepted this revision.Jan 14 2017, 4:44 PM
matthew edited edge metadata.

Builds fine on 11-STABLE r310632

This revision is now accepted and ready to land.Jan 14 2017, 4:44 PM
Closed by commit rP431501: audio/teamspeak3-server: unbreak on 11-STABLE (authored by lifanov). · Explain WhyJan 14 2017, 11:31 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
audio/
teamspeak3-server/
6 lines

Diff 24011

View Options

head/audio/teamspeak3-server/Makefile

Loading...