Bug 199177 - [patch] x11-toolkits/wxgtk30 : support c++11 instead of c++0x, plus dragonfly
Summary: [patch] x11-toolkits/wxgtk30 : support c++11 instead of c++0x, plus dragonfly
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: John Marino
URL:
Keywords: patch, patch-ready
Depends on: 199600
Blocks:
  Show dependency treegraph
 
Reported: 2015-04-05 18:07 UTC by John Marino
Modified: 2015-04-22 13:40 UTC (History)
1 user (show)

See Also:
marino: maintainer-feedback+


Attachments
patch: Disables TR1 support, converts to c++11 (3.52 KB, patch)
2015-04-05 18:07 UTC, John Marino
no flags Details | Diff
List of 62 ports built in poudriere (1.11 KB, text/plain)
2015-04-05 18:09 UTC, John Marino
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description John Marino freebsd_committer freebsd_triage 2015-04-05 18:07:57 UTC
Created attachment 155203 [details]
patch: Disables TR1 support, converts to c++11

wxgtk30 uses the TR1 functions in its installed headers.  These functions are not present in c++11 standard, and modern gcc (e.g. gcc4.9, gcc5) don't even recognized TR1 (gcc 4.7 did).

The following patch forcibly removes TR1 functions, which requires the passing of -std=c++11 in the CXXFLAGS.  This allows wxgtk30 headers to be used by latest gcc compilers.

While here:

 - remove $PTHREAD_LIBS from configure.
It wasn't populated, and the default of "-pthread" is fine

 - bring in dragonfly support from dports.


TESTING:
I used poudriere and built the following 62 ports before and after the patch was applied.  These 62 ports are listed in INDEX-10 as requiring wxgtk30.  The result was the same before and after: all 62 build successfully.
Comment 1 John Marino freebsd_committer freebsd_triage 2015-04-05 18:09:37 UTC
Created attachment 155204 [details]
List of 62 ports built in poudriere

This is the list of ports built in poudriere.

These are the 62 primary ports -- on the first build with sanity checking, I ended up building 363 ports!
Comment 2 John Marino freebsd_committer freebsd_triage 2015-04-05 18:10:27 UTC
I'm looking for maintainer approval to commit the patch.
Comment 3 Chris Hutchinson 2015-04-06 17:06:39 UTC
(In reply to John Marino from comment #2)
> I'm looking for maintainer approval to commit the patch.

Sorry for the delay, John. I'm afraid I didn't get the
prior messages.

I've examined your proposed patch. It looks good.
Thanks for all the work you put into this!

Please consider this reply, as "maintainer approval".

Thanks again, John.

--Chris
Comment 4 John Marino freebsd_committer freebsd_triage 2015-04-06 17:23:15 UTC
Thanks, all three messages were written within 3 minutes of each other.
Comment 5 Chris Hutchinson 2015-04-06 18:02:10 UTC
(In reply to John Marino from comment #4)
> Thanks, all three messages were written within 3 minutes of each other.
I don't know about your Mondays. But mine seem to start off
*really* busy. Which led to my having overlooked that small
detail. :-)

Thanks for taking to time to point that out, John, and thanks again,
for all the work you did on this.

--Chris
Comment 6 commit-hook freebsd_committer freebsd_triage 2015-04-06 18:50:11 UTC
A commit references this bug:

Author: marino
Date: Mon Apr  6 18:49:54 UTC 2015
New revision: 383466
URL: https://svnweb.freebsd.org/changeset/ports/383466

Log:
  x11-toolkits/wxgtk30: Support c++11 over c++0x, plus DragonFly

  The wxgtk30 port will use TR1 headers if the capability is detected.
  These TR1 functions were experimental and not included in the same form
  in the final c++11 standard.  This patch forcibly disabled TR1 support,
  which requires the explicit setting of -std=c++11 in CXXFLAGS>

  A private "exp-run" was performed, all 62 ports dependent on wxgtk2
  successfully built before and after after the patch was applied on two
  separate runs (FreeBSD 10 amd64 jail).

  While here, remove the incorrect $PTHREAD_LIB setting and let it default
  to -pthread.  Also bring in the configure patch fragement from DragonFly.

  PR:		199177
  Submitted by:	marino
  Approved by:	maintainer (Chris Hutchinson)

Changes:
  head/x11-toolkits/wxgtk30/Makefile
  head/x11-toolkits/wxgtk30/files/patch-configure
  head/x11-toolkits/wxgtk30/files/patch-setup.h.in
Comment 7 John Marino freebsd_committer freebsd_triage 2015-04-06 18:52:22 UTC
Here, Easter Monday is a holiday. :)
Comment 8 Chris Hutchinson 2015-04-22 13:40:28 UTC
@John, @Jan
Thanks for taking the time to create the reference.

--Chris