Opened 7 years ago

Last modified 7 years ago

#11810 new Patches

privilege __sun over sun for strict ISO compilers

Reported by: Richard PALO <richard@…> Owned by: Vladimir Prus
Milestone: To Be Determined Component: build
Version: Boost 1.59.0 Severity: Problem
Keywords: Cc:

Description

when compilers use strict ISO options such as -std=c++11 they only generate for SunOS the macro '__sun' and not 'sun'.

I propose minimally the attached patches, although there seems to be a bit of overkill with tests involving both in the source tree.

That is to say, the documented way to test is something like the following:

#ifdef __sun
/* this is SunOS */
#endif

#if defined(__sun) && defined(__SVR4)
/* this in particular is Solaris */
#endif

Attachments (3)

0001-libs-config-convert-sun-__sun-for-strict-ISO-compile.patch (1.0 KB ) - added by Richard PALO <richard@…> 7 years ago.
patchset for libs/config convert 'sun' => 'sun'
0001-libs-math-convert-sun-__sun-for-strict-ISO-compilers.patch (1.5 KB ) - added by Richard PALO <richard@…> 7 years ago.
patchset for libs/math convert 'sun' => 'sun'
0001-tools-build-convert-sun-__sun-for-strict-ISO-compile.patch (2.9 KB ) - added by Richard PALO <richard@…> 7 years ago.
patchset for tools/build convert 'sun' => 'sun'

Download all attachments as: .zip

Change History (6)

by Richard PALO <richard@…>, 7 years ago

patchset for libs/config convert 'sun' => 'sun'

by Richard PALO <richard@…>, 7 years ago

patchset for libs/math convert 'sun' => 'sun'

comment:1 by Richard PALO <richard@…>, 7 years ago

third patchset refused?

by Richard PALO <richard@…>, 7 years ago

patchset for tools/build convert 'sun' => 'sun'

comment:2 by viboes, 7 years ago

Component: Noneconfig
Owner: set to John Maddock

comment:3 by John Maddock, 7 years ago

Component: configbuild
Owner: changed from John Maddock to Vladimir Prus

Reassigning, as this is a Boost.Build issue.

Note: See TracTickets for help on using tickets.