Bug 198436 - Fix net/mediatomb build with clang 3.6.0
Summary: Fix net/mediatomb build with clang 3.6.0
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks: 197395
  Show dependency treegraph
 
Reported: 2015-03-08 21:23 UTC by Dimitry Andric
Modified: 2015-09-24 19:51 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (leo)


Attachments
Move singleton specialization to before timer.h inclusion (1.07 KB, patch)
2015-03-08 21:23 UTC, Dimitry Andric
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer freebsd_triage 2015-03-08 21:23:34 UTC
Created attachment 154039 [details]
Move singleton specialization to before timer.h inclusion

During the exp-run in bug 197395, it was found that net/mediatomb gives errors with clang 3.6.0:

http://package18.nyi.freebsd.org/data/headamd64PR197395-default/2015-02-11_23h04m51s/logs/errors/mediatomb-0.12.1_12.log

This error is because the timer.h header implicitly defines the Timer singleton already, and such a definition cannot be overridden later by a specialization.

I propose to fix this by defining the singleton specialization *before* including the timer.h file, as in the attached patch.
Comment 1 Bartek Rutkowski freebsd_committer freebsd_triage 2015-04-01 16:41:04 UTC
This fails to build:

http://pd.valinor.palantiri.org/data/84amd64-default/2015-04-01_18h11m14s/logs/errors/mediatomb-0.12.1_12.log

So I am returning it to the pool.
Comment 2 commit-hook freebsd_committer freebsd_triage 2015-09-24 19:49:53 UTC
A commit references this bug:

Author: pi
Date: Thu Sep 24 19:49:20 UTC 2015
New revision: 397755
URL: https://svnweb.freebsd.org/changeset/ports/397755

Log:
  net/mediatomb: remove BROKEN, fix build

  - now uses UIDs/GIDs
  - fix build in certain cases

  PR:		198436, 202467
  Submitted by:	Pavel Nedoshivkin <nedoshivkin@gmail.com>, dim
  Approved by:	maintainer (timeout)

Changes:
  head/GIDs
  head/UIDs
  head/net/mediatomb/Makefile
  head/net/mediatomb/files/config.xml.dist.in
  head/net/mediatomb/files/mediatomb.in
  head/net/mediatomb/files/patch-timer.cc
  head/net/mediatomb/files/pkg-install.in
  head/net/mediatomb/pkg-plist