python3 fails to build if directory or sysroot contains "*icc*" string

[locutusofborg]

BPO	33341
Nosy	@terryjreedy, @vstinner, @vadmium
Superseder	bpo-28584: ICC compiler check is too permissive
Files	icc-find.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2018-05-17.15:31:11.576>
created_at = <Date 2018-04-23.10:12:22.359>
labels = ['build', '3.8']
title = 'python3 fails to build if directory or sysroot contains "*icc*" string'
updated_at = <Date 2018-05-19.00:38:18.642>
user = 'https://bugs.python.org/locutusofborg'

bugs.python.org fields:

activity = <Date 2018-05-19.00:38:18.642>
actor = 'vstinner'
assignee = 'none'
closed = True
closed_date = <Date 2018-05-17.15:31:11.576>
closer = 'terry.reedy'
components = ['Build']
creation = <Date 2018-04-23.10:12:22.359>
creator = 'locutusofborg'
dependencies = []
files = ['47596']
hgrepos = []
issue_num = 33341
keywords = ['patch']
message_count = 6.0
messages = ['315656', '315805', '315854', '316920', '316945', '316948']
nosy_count = 4.0
nosy_names = ['terry.reedy', 'vstinner', 'martin.panter', 'locutusofborg']
pr_nums = []
priority = 'normal'
resolution = 'duplicate'
stage = 'resolved'
status = 'closed'
superseder = '28584'
type = 'compile error'
url = 'https://bugs.python.org/issue33341'
versions = ['Python 3.8']

[locutusofborg]

Hello, as said, in yocto we pass in $CC environment, also the --sysroot keyword.

If --sysroot contains the "*icc*" or "*gcc*" in path, the build fails because of wrong autoconf checks.

checking in a case switch for *icc* seems bad to me, I propose to change it with something like
"*icc" so we might exclude when icc is not the compiler.

I know this isn't a strong patch, but maybe we can apply it while writing some better detect code.

The build fails as descripted in the yocto bug
https://bugzilla.yoctoproject.org/show_bug.cgi?id=12703

[locutusofborg]

I signed the contributor agreement, even if such patches shouldn't need any kind of license

[terryjreedy]

Thank you. Yes, we require a signed CA for something this non-trivial.

Victor, I don't know who maintains configure. If you don't, maybe you know who does.

[vadmium]

Maybe this is the same as bpo-28584, about the ${CC} variable rather than “sysroot”. In any case, the patch looks unrelated.

[locutusofborg]

This was the patch I wanted to add, but I picked up a wrong one (a vbox one)

In any case, duplicate of bpo-28584

[terryjreedy]

icc-find.patch (and a 2nd version) has been uploaded to bpo-28584 also. I unlinked the erroneous upload 'patch'.