Differential D22026

mail/milter-callback: don't add -fnested-functions on head on powerpc*
ClosedPublic
Actions

Authored by pkubaj on Oct 14 2019, 5:45 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 17 2024, 5:57 PM

	Unknown Object (File)
	Dec 20 2023, 3:23 AM

	Unknown Object (File)
	Dec 19 2023, 8:49 AM

	Unknown Object (File)
	Nov 10 2023, 8:55 AM

	Unknown Object (File)
	Nov 8 2023, 8:56 AM

	Unknown Object (File)
	Nov 7 2023, 10:40 PM

	Unknown Object (File)
	Oct 7 2023, 7:48 AM

	Unknown Object (File)
	Oct 6 2023, 9:35 PM

View All 14 Files

Subscribers

Contributor Reviewers (ports)

Details

Reviewers

Commits

rP514519: Conditionalize the inclusion of the -fnested-functions flag on GCC.

Summary

With powerpc* soon switching to clang on head, GCC from ports will be used and it doesn't accept -fnested-functions.

Test Plan

linimon
mat
tcberner

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

pkubaj created this revision.Oct 14 2019, 5:45 PM

Herald added subscribers: Contributor Reviewers (ports), mat. · View Herald TranscriptOct 14 2019, 5:45 PM

Harbormaster completed remote builds in B27026: Diff 63262.Oct 14 2019, 5:45 PM

linimon added a reviewer: linimon.Oct 15 2019, 12:23 AM

As per my prior feedback, this does indeed fix the problem on elfv2 but breaks it on elfv1/13:

/wrkdirs/usr/ports/mail/milter-callback/work/milter-callback-1.6.0/cache.c: In function 'extendCache':
/wrkdirs/usr/ports/mail/milter-callback/work/milter-callback-1.6.0/cache.c:338: error: nested functions are disabled, use -fnested-functions to re-enable

This revision was not accepted when it landed; it landed in state Needs Review.Oct 15 2019, 12:43 PM

Closed by commit rP514519: Conditionalize the inclusion of the -fnested-functions flag on GCC. (authored by linimon). · Explain Why

This revision was automatically updated to reflect the committed changes.

linimon added a commit: rP514519: Conditionalize the inclusion of the -fnested-functions flag on GCC..

Revision Contents
Changeset List

Path

Size

head/

mail/

milter-callback/

16 lines

Diff 63288

head/mail/milter-callback/Makefile

Loading...