Bug 197864

Summary: Fix devel/libev build with clang 3.6.0
Product: Ports & Packages Reporter: Dimitry Andric <dim>
Component: Individual Port(s)Assignee: Vsevolod Stakhov <vsevolod>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (vsevolod)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 197395    
Attachments:
Description Flags
Fix positioning of _Noreturn attribute for libev none

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