Bug 197864 - Fix devel/libev build with clang 3.6.0
Summary: Fix devel/libev 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 Only Me
Assignee: Vsevolod Stakhov
URL:
Keywords:
Depends on:
Blocks: 197395
  Show dependency tree / graph
 
Reported: 2015-02-20 21:30 UTC by Dimitry Andric
Modified: 2015-02-23 09:39 UTC (History)
0 users

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

Attachments
Fix positioning of _Noreturn attribute for libev (911 bytes, patch)
2015-02-20 21:30 UTC, Dimitry Andric 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

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-02-20 21:30:41 UTC 
Created attachment 153246 [details]
Fix positioning of _Noreturn attribute for libev

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

http://package18.nyi.freebsd.org/data/headamd64PR197395-default/2015-02-11_23h04m51s/logs/errors/libev-4.15_1,1.log

This is because clang 3.6.0 defaults to C11, and libev puts its ecb_noreturn in the wrong place, at the end of a function declaration.

The placement was fixed by upstream here:

http://cvs.schmorp.de/libev/ev.c?r1=1.473&r2=1.474

but this also has a lot of other unneeded changes.  So I propose the following minimal fix, which changes just two lines.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2015-02-20 21:30:41 UTC 
Auto-assigned to maintainer vsevolod@FreeBSD.org
Comment 2 commit-hook freebsd_committer freebsd_triage 2015-02-23 09:39:58 UTC 
A commit references this bug:

Author: vsevolod
Date: Mon Feb 23 09:39:08 UTC 2015
New revision: 379653
URL: https://svnweb.freebsd.org/changeset/ports/379653

Log:
  - Fix positioning of _Noreturn attribute for libev
  - Do not bump portrevision, since a package has not been changed

  PR:		197864
  Submitted by:	dim

Changes:
  head/devel/libev/files/patch-ev.c