Bug 198137 - devel/tradcpp : possible source of FreeBSD 11 regressions
Summary: devel/tradcpp : possible source of FreeBSD 11 regressions
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Baptiste Daroussin
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2015-03-01 17:46 UTC by John Marino
Modified: 2015-03-22 16:40 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (bapt)


Attachments

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-03-01 17:46:08 UTC
see bug 197562

The fix for libgpg-error using tradcpp had to be reverted due to configure failure:

checking how to run the C preprocessor... tradcpp
configure: error: in `/wrkdirs/usr/ports/security/libgpg-error/work/libgpg-error-1.17':
configure: error: C preprocessor "tradcpp" fails sanity check
See `config.log' for more details

(I don't have config.log)

This is the second reversion, a similar thing happened for x11/sessreg.

I don't know where the problem lies, tradcpp or what, but this is a clear regression from FreeBSD 10.  I'm opening the PR on the general regression and starting with tradcpp.
Comment 1 Antoine Brodin freebsd_committer freebsd_triage 2015-03-02 18:57:19 UTC
In the freebsd ports,  tradcpp is almost never used for preprocessing C code.  It is used 99% of time to preprocess Imakefile.c files which are not real C code and produce a Makefile.

tradcpp is unable to preprocess some headers from freebsd 8.4 or 9.3 so I don't think it's a regression.
Comment 2 John Marino freebsd_committer freebsd_triage 2015-03-02 19:15:13 UTC
I disagree.

With the patch for libgpg-error, tradcpp worked fine on FreeBSD 10.   The same patch on FreeBSD 11 fails to configure.  The only difference is FreeBSD.  Granted it's the big difference, but that's the definition of regression -- something that worked before stopped working.
Comment 3 John Marino freebsd_committer freebsd_triage 2015-03-02 19:15:42 UTC
that said, "CPP+= -P" is probably the workaround to try before tradcpp.
Comment 4 John Marino freebsd_committer freebsd_triage 2015-03-22 16:40:03 UTC
I'm not going to use tradcpp anymore, so I'm withdrawing the PR.