11:34 Felix Palmen (zirias) 

Mk/Uses/compiler.mk: Support newer C++ standards

* Add c++20-lang and c++2b-lang arguments
* Respect LLVM_DEFAULT if it fullfills the requirements

PR:			244810, 266029
Exp-run by:		antoine
Approved by:		bapt, tcberner (mentor)
Differential Revision:	https://reviews.freebsd.org/D35889

    5ca2208

21:35 Jan Beich (jbeich) 

Mk/Uses/compiler.mk: switch to -stdlib=libc++

GCC >= 11 supports -stdlib=libc++ (like Clang) which allows to
simplify USES=compiler:gcc-c++11-lib logic.

PR:		265964
Exp-run by:	antoine

    b00b217

10:00 Tobias C. Berner (tcberner) 

framework: cleanup conditional-indentations in Mk/

Run Tools/scripts/indent_make_if.pl on all of Mk.

These white space changes contribute greatly to the readability of those files.
As we have a version control system, finding out the reasons for the changes
prior to these white space changes is still easily possible

Differential Revision:	https://reviews.freebsd.org/D35024
Reviewed by:		portmgr (rene, bapt)

    aa25396

09:29 Rene Ladan (rene) 

Mk: release portmgr maintainership of most USES files

Differential Revision: https://reviews.freebsd.org/D34936

Reviewed by:	desktop (tcberner), portmgr (tcberner), ruby (yasu), tijl

    767d81e

18:56 Gerald Pfeifer (gerald) 

Mk/Uses/compiler.mk: Avoid USE_GCC=any

USE_GCC=any is gone, replace it by USE_GCC=yes which has been equivalent
for a while now.

Approved by:	portmgr (antoine)

    8084766

17:47 Fernando Apesteguía (fernape) 

*/*: Remove compiler:openmp from Mk/Uses/compiler.mk

The option implied a dependency on gcc but clang got openmp support long ago.

Remove compiler:openmp from Mk/Uses/compiler.mk

For ports using USE=compiler:openmp, just remove it and make them build with
clang.
Fix conditionals when necessary
Bump PORTREVISION where appropriate

If problem arises, they can be addressed by using USE_GCC=yes

An update to the Porter's Handbook will follow.

Approved by: portmgr (bapt)
Differential Revision: https://reviews.freebsd.org/D31971

    07fb2d5

14:27 Mathieu Arnold (mat) 

framework: Remove $FreeBSD$

Where appropriate fiddle with a few other things.

    5d33e04

17:47 jrm 

compiler.mk: Update clang dependency to v10; clean up duplication

PR:		252627
Approved by:	brooks, portmgr (swills)
Exp-run by:	antoine
Differential Revision:	https://reviews.freebsd.org/D28120

 

21:46 jkim 

Support compiler version >= 10 (again).

PR:		244781
Approved by:	portmgr (antoine)

 

19:50 antoine 

Revert r528363, it's broken for llvm < 10

With hat:	portmgr

 

16:41 jkim 

Support compiler version >= 10.

PR:		244781
Approved by:	portmgr (antoine)

 

12:01 antoine 

All supported versions of FreeBSD that install clang have version 6.0.0 or
newer

With hat:	portmgr

 

12:39 antoine 

Add support for USES=compiler:c++17-lang

PR:		222415
Submitted by:	Naram Qashat
With hat:	portmgr

 

23:14 jbeich 

Update USES=compiler:c++14-lang to Clang 6.0 on FreeBSD 10

PR:		227679
Exp-run by:	antoine (only 10.4)

 

19:31 bdrewery 

Don't add 'yes' onto the end of errored std output lookups.

 

23:21 bdrewery 

PORTS_ENV: Reorder CC hash and std hash so CC's comes first.

This is only a cosmetic change.

Approved by:	portmgr (implicit)

 

23:10 bdrewery 

PORTS_ENV: Rename OUTPUT_* to CC_OUTPUT_* to be more clearly related to
compiler.mk.

This is purely a cosmetic change.

Approved by:	portmgr (implicit)

 

19:34 bdrewery 

Add compiler.mk to PORTS_ENV.

This only handles the default CC=cc but is enough to significantly improve
INDEX/Poudriere ports var gathering due to reducing around a dozen cc
fork+exec for every port that all have the same result.  These values
are reused during the port build as well which reduces more fork+exec
contention there.

Approved by:	portmgr (implicit)

 

13:30 bapt 

Replace llvm 3.4 by llvm 5.0, as a choice for people who enforce clang
by default everywhere

This code would actually needs to be probably cleaned up, but that would be
for later

 

10:41 jbeich 

Update USES=compiler:c++14-lang to Clang 5.0 on FreeBSD 10

PR:		224591
Exp-run by:	antoine
Approved by:	portmgr (antoine)

 

20:43 gerald 

Replace USE_GCC=5+ by USE_GCC=yes in the case of USES=compiler:c++14-lang.

When this was added by r392084 on 2015-07-14, the default flavor of GCC
was GCC 4.8 and explicitly requesting GCC 5 (or later) was necessary for
C++ 14 support.  Now that the default version of GCC is GCC 6, after GCC 5
for several months, we can use the preferred notion of USE_GCC=yes instead
of specifying a concrete minimum version.

Among others this helps with cases where GCC 6 is better adjusted for
FreeBSD, notably the well known std::to_string issue (where that is only
enabled with GCC 6 or later).

PR:		222268
Approved by:	portmgr (antoine)
MFH:		2017Q3

 

11:57 bapt 

Use llvm 4.0 for c++14

exp-run:	antoine
Reviewed by:	antoine, swills
Approved by:	swills
Differential Revision:	https://reviews.freebsd.org/D10228

 

19:04 ak 

- Remove checks for .PARSEDIR and vestiges of pmake support

PR:	218832
Approved by:	portmgr (antoine)

 

07:24 rene 

Remove support for FreeBSD < 10.3 from Mk/*

This does not yet convert the files to bmake.

Reviewed by:	mat
Differential Revision:	https://reviews.freebsd.org/D9016

 

05:08 jbeich 

Switch USES=compiler:gcc-c++11-lib to use base libc++ on 10.2+ systems

PR:		215192
Requested by:	dim
Exp-run by:	antoine
Approved by:	antoine

 

17:42 antoine 

All supported versions of FreeBSD that install clang have version 3.4.1 or
newer,  so remove obsolete checks in compiler.mk and objc.mk

Reviewed by:	bapt
Differential Revision:	https://reviews.freebsd.org/D8496

 

01:23 bapt 

Remove the now unneeded ${PORTSDIR} from dependency definition in
The infrastructure Makefiles

PR:		206569
Exp run by:	antoine
Differential Revision:	D5047

 

18:54 antoine 

Extend r405599 to make COMPILER_VERSION and ALT_COMPILER_VERSION more
compatible with clang 3.8 trunk

PR:		206074
With hat:	portmgr

 

23:41 bdrewery 

Make COMPILER_VERSION more compatible with clang 3.8 trunk.

"clang version 3.8.0 (trunk 256945) (based on LLVM 3.8.0svn)" was giving
"38 38" was a result.  Now duplicates for fmake are trimmed and only the first
version found is used for bmake using its :tW.

With hat:	portmgr
In collaboration with:	dim

 

14:55 mat 

Drop 8 support.

With hat:	portmgr
Sponsored by:	Absolight
Differential Revision:	https://reviews.freebsd.org/D3694

 

07:36 bapt 

Readd PORTSDIR for now we will only start removing them after 2016Q1 is branched

This gives more time for tools to get updated, available in packages etc before
bothering users

 

16:49 bapt 

Drop the necessity to add ${PORTSDIR} to dependency line

Modify make describe to automatically prepend ${PORTSDIR} if the path for the
port is not absolute

Checked with poudriere, portmaster, portupgrade

PR:		203685
Exp-run by:	antoine
Differential Revision:	https://reviews.freebsd.org/D3866

 

21:38 brd 

Bump the version of clang to 3.6 when c++14-lang is specified

Discussed with:	brooks
Approved by:	bdrewery (mentor)

 

20:32 brd 

Add support for USES= compiler:c++14

PR:		201076
Approved by:	bapt, bdrewery (mentor)

 

21:46 antoine 

Set CHOSEN_COMPILER_TYPE for USES=compiler:gcc-c++11-lib

PR:		ports/196111
Submitted by:	Jan Beich
With hat:	portmgr

 

17:22 antoine 

Enforce clang 3.4 in USES=c*11*,objc,gnustep as clang 3.3 has some bugs

Requested by:	bapt
PR:		ports/195122
Differential Revision:	https://reviews.freebsd.org/D1181
Exp-run by:	self
With hat:	portmgr

 

18:13 antoine 

Switch from clang33 to clang34 for ports using compiler:*11*, objc or
gnustep on freebsd 8.4 and 9.1

PR:		ports/193555
Differential Revision:	https://reviews.freebsd.org/D935
Reviewed by:	bapt
Exp-run:	self

 

16:36 tijl 

Change the way USES is handled:
- Loop over USES twice, once to define all *_ARGS variables and once to
  include Uses/*.mk.  This allows all Uses/*.mk to examine arguments given
  to other USES entries.
- Always define *_ARGS (possibly empty) and replace commas with spaces.

Similar for _USES_POST.

Adjust all Uses/*.mk:
- defined(u_ARGS) becomes !empty(u_ARGS)
- Eliminate helper variables like _*_ARGS=${*_ARGS:C/,/ /g}
- Some Uses/*.mk used ":" as argument separator instead of ",", but no port
  used this form
- Uses/cran.mk: remove unused variable VALID_ARGS and USES+=fortran which
  has no effect
- Uses/twisted.mk: simplify handling of the case where neither "build" nor
  "run" arguments have been specified

PR:		193931
Exp-run by:	antoine
Approved by:	portmgr (antoine)

 

20:35 nwhitehorn 

Avoid using clang as a fallback compiler on Tier-2 systems for C++0x
or C11 features on platforms where it is not known to work.

 

03:23 bdrewery 

- Fix logic from r359424 to not always apply - only apply on tier2 archs.

With hat:	portmgr

 

22:16 nwhitehorn 

On some Tier-2 platforms (e.g. PowerPC), clang kind of works and so is
installed but is not cc. On such platforms, clang is usually not default
for a reason and so using it for C++11 is unwise. Instead, fall back to
newer GCC. On i386 and amd64, clang works even if it isn't the default,
so continue using it there.

This fixes the build for Boost, among other software, on PowerPC.

Approved by:	bapt

 

15:16 bapt 

Add compiler:gcc-c++11-lib to instrument gcc linking with libc++ on system where
default stl is libc++

 

10:31 gerald 

Move MAINTAINER lines to the end of the initial comment block and
make things a bit more consistent.

Approved by:	portmgr (bapt)

 

17:45 bapt 

Now that default gcc is 4.7 switch back to USE_GCC=yes where appropriate

 

22:09 rene 

Mention the new c++0x argument.

Approved by:	portmgr (bdrewery)

 

07:10 bapt 

Fix test

 

06:56 bapt 

Make gcc 4.6 is sufficient for c++0x

 

06:55 bapt 

Allow to choose clang from base on 9.2 when looking for a C11 compiler
Add compiler:c++0x modeled over compiler:c++11-lang [1]

Requested by:	delphij [1]

 

23:45 rene 

Spell out "nestedfct"

Approved by:	portmgr (bapt)

 

23:22 rene 

Mention nestedfct parameter.

Perform some language fixes while here.

Approved by:	portmgr (bapt)

 

14:14 bapt 

Add compiler:nestedfct which will automatically probe USE_GCC=any

 

21:48 bapt 

Fix a typo

Reported by:	Rainer Hurling <rhurlin@gwdg.de>

 

21:47 bapt 

Swap compiler.mk to use gcc 4.7 when requesting a C++11/C11 compiler and clang
does not fit or the user explicitly want GCC.

 

12:41 bapt 

Add CHOSEN_COMPILER_TYPE which can be clang or gcc

It value correspond to the type of compiler chosen by the famework depending on
the options

Requested by:	rene

 

22:25 bapt 

Make sure clang uses ports ld on FreeBSD 8

 

09:25 bapt 

On FreeBSD 8 clang will need a newer version of binutils.

Reported by:	dbn

 

17:35 bapt 

Allow base clang to be used if version is >= 3.3
This gives 2 new variables to the porters:
ALT_COMPILER_TYPE which can be empty, clang or cc depending on what ${CC} is
ALT_COMPILER_VERSION which will give the porter a 2 digit version of the
alternative compiler

Requested by:	many

 

09:50 rene 

Document "openmp" as a valid argument.

Approved by:	portmgr (bapt)

 

16:46 marino 

Mk/Uses/compiler.mk: Ensure COMPILER_TYPE can only be clang or gcc

The compiler.mk comments and code state that COMPILER_TYPE can only be
of the value "clang" or "gcc".  However, the code that determines this
allows for a possible undefined third state (empty string).  BMake
will emit a lot of errors about badly formatted conditionals if
COMPILER_TYPE is empty.

Since, by definition, if the COMPILER_TYPE is not clang, it must be
gcc, so skip the conditional gcc check and just set it.  The entire
file must be updated if support for additional compilers is desired.

This bug was discovered because the gcc detection code failed to
identify the DragonFly base compiler (GCC 4.7.3) as gcc.

Approved by:	portmgr (bapt)

 

05:42 bapt 

Always use a recent version of gcc in case a port needs openmp support

Submitted by:	gerald

 

05:52 bapt 

Add a new argument openmp to make sure that only gcc is used (for now) is the
port requires openmp

Requested by:	brooks

 

05:45 bapt 

Really really commit the right version

Pointyhat to:	bapt

 

00:46 bdrewery 

Revert r329829 to restore compiler.mk

 

19:41 bapt 

Actually commit the right version

 

18:55 bapt 

New USES=compiler

Supported arguments are:
  - c++11-lang: the port needs a c++11 aware compiler what ever standard
  library it uses, implies features
  - c++11-lib: the port needs a c++11 standard library, implies features
  - c11: the ports needs a c11 aware compiler implies features
  - features: this will create a COMPILER_FEATURES variable which contains
  the list of features ${CC} do support, implies env.
  - env: the COMPILER_TYPE will be set to either gcc or clang.

  By default the uses will try to use clang33 from ports when nothing in
  base is relevant except if the user explicitly defines
  FAVORITE_COMPILER=gcc in his make.conf

Please note that testing tinderbox prior to version: 4.0.1_1 is not able to
properly figure out the dependencies implied by this USES.