Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 526842 - mail-filter/procmail-3.22-r10 with sys-devel/clang - x86_64-pc-linux-gnu-clang-3.5.0: error: unknown argument: '-fno-ipa-cp-clone'
Summary: mail-filter/procmail-3.22-r10 with sys-devel/clang - x86_64-pc-linux-gnu-clan...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: systemwide-clang
  Show dependency tree
 
Reported: 2014-10-25 23:10 UTC by Markus Ullmann
Modified: 2016-06-15 03:36 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,4.77 KB, text/plain)
2014-10-25 23:10 UTC, Markus Ullmann
Details
emerge --info (emerge-info.log,4.99 KB, text/plain)
2014-10-25 23:10 UTC, Markus Ullmann
Details
Add case statement to procmail ebuild for clang compliance (procmail-compilers-case-for-clang.patch,737 bytes, patch)
2015-01-10 20:05 UTC, Fabio Scaccabarozzi
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Ullmann 2014-10-25 23:10:11 UTC
Created attachment 387370 [details]
build.log

Using the following directories for the LOCKINGTESTs:
  /tmp . 
Initiating fcntl()/kernel-locking-support tests
Whoeaaa!  There's something fishy going on here.
You have a look and see if you detect anything uncanny:
*******************************************************
make[2]: Entering directory '/home/tmp/portage/mail-filter/procmail-3.22-r10/work/procmail-3.22/src'
clang -c -O2 -march=core2 -mtune=core2 -msse4.1 -msse4.2 -pipe -fno-inline-functions -fno-ipa-cp-clone   _autotst.c
x86_64-pc-linux-gnu-clang-3.5.0: error: unknown argument: '-fno-ipa-cp-clone'
Makefile:240: recipe for target '_autotst.o' failed
make[2]: *** [_autotst.o] Error 1
make[2]: Leaving directory '/home/tmp/portage/mail-filter/procmail-3.22-r10/work/procmail-3.22/src'
*******************************************************
I suggest you take a look at the definition of CFLAGS* and CC
in the Makefile before you try make again.
Makefile:124: recipe for target '../autoconf.h' failed
make[1]: *** [../autoconf.h] Error 1
Comment 1 Markus Ullmann 2014-10-25 23:10:32 UTC
Created attachment 387372 [details]
emerge --info
Comment 2 Fabio Scaccabarozzi 2015-01-10 20:05:59 UTC
Created attachment 393638 [details, diff]
Add case statement to procmail ebuild for clang compliance

Attached is a patch that fixes the issue.
The patch introduces a "case" switch for the procmail ebuild in order to support compilation with clang.
As can be seen, the append-flags statement for clang appends only -fno-inline-functions, as I couldn't trace back the original report and can't run procmail on my system to actually test it (no configuration).
-fno-ipa-cp-clone gets dropped for clang, as it's a GCC-specific flag.
The package compiles and tests fine with both GCC (4.8.4) and clang (3.5.0).
Comment 3 Austin English (RETIRED) gentoo-dev 2016-06-02 08:25:22 UTC
Sent a PR:
https://github.com/gentoo/gentoo/pull/1583
Comment 4 Austin English (RETIRED) gentoo-dev 2016-06-15 00:36:30 UTC
Robbin/Mgorny, please review/ack/merge
https://github.com/gentoo/gentoo/pull/1583
Comment 5 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2016-06-15 03:04:03 UTC
<hat project="net-mail@gentoo.org">
Why did you drop FAQ & README from dodoc?
Please re-add them, then you can merge your PR yourself.
</hat>
Comment 6 Austin English (RETIRED) gentoo-dev 2016-06-15 03:36:37 UTC
(In reply to Robin Johnson from comment #5)
> <hat project="net-mail@gentoo.org">
> Why did you drop FAQ & README from dodoc?
> Please re-add them, then you can merge your PR yourself.
> </hat>

Misreading of what eapi6 changes were, my mistake.

Fixed by https://github.com/gentoo/gentoo/commit/e3fefa967b840fb928c2cf21930bcf50ea62d713