Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 649664 - www-client/chromium-65.0.3325.106 doesn't call correct clang
Summary: www-client/chromium-65.0.3325.106 doesn't call correct clang
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Chromium Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-05 17:04 UTC by Michał Górny
Modified: 2018-03-05 17:27 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-03-05 17:04:40 UTC
[288/616] clang++ [...]
warning: unknown warning option '-Wno-enum-compare-switch'; did you mean '-Wno-enum-compare'? [-Wunknown-warning-option]
warning: unknown warning option '-Wno-tautological-unsigned-zero-compare' [-Wunknown-warning-option]
warning: unknown warning option '-Wno-null-pointer-arithmetic'; did you mean '-Wno-null-arithmetic'? [-Wunknown-warning-option]
warning: unknown warning option '-Wno-tautological-constant-compare'; did you mean '-Wno-tautological-pointer-compare'? [-Wunknown-warning-option]
4 warnings generated.


It should be at least '${CHOST}-clang'. Also, I find it quite silly that you force a specific compiler and at the same time can't pass correct options to it.
Comment 1 Mike Gilbert gentoo-dev 2018-03-05 17:11:36 UTC
Does setting CXX="${CHOST}-clang++" make any practical difference? The package doesn't actually work when cross-compiling anyway.

> Also, I find it quite silly that you force a specific compiler and at the same time can't pass correct options to it.

I find it quite annoying to get such feedback from someone who has spent no effort maintaining this package. The warnings are harmless, so I didn't bother trying to patch out the flags from the build system. If you would like to provide a patch, please do so.
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-03-05 17:14:47 UTC
(In reply to Mike Gilbert from comment #1)
> Does setting CXX="${CHOST}-clang++" make any practical difference? The
> package doesn't actually work when cross-compiling anyway.

When I distcc from 32-bit host, getting 64-bit objects is not very helpful.
Comment 3 Larry the Git Cow gentoo-dev 2018-03-05 17:27:57 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1ae8ae1f1e0cfb78e67e664ec6a655844b5b49b0

commit 1ae8ae1f1e0cfb78e67e664ec6a655844b5b49b0
Author:     Mike Gilbert <floppym@gentoo.org>
AuthorDate: 2018-03-05 17:26:17 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2018-03-05 17:27:50 +0000

    www-client/chromium: include CHOST in CXX override for clang
    
    Closes: https://bugs.gentoo.org/649664
    Package-Manager: Portage-2.3.24_p18, Repoman-2.3.6_p99

 www-client/chromium/chromium-65.0.3325.106.ebuild | 4 ++--
 www-client/chromium/chromium-65.0.3325.88.ebuild  | 4 ++--
 www-client/chromium/chromium-66.0.3355.0.ebuild   | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)