Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 652234 - sys-devel/clang-6.* should depend on dev-python/pyyaml: run-clang-tidy.py: ImportError: No module named yaml
Summary: sys-devel/clang-6.* should depend on dev-python/pyyaml: run-clang-tidy.py: Im...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: LLVM support project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-02 19:11 UTC by LE GARREC Vincent
Modified: 2018-04-04 13:56 UTC (History)
2 users (show)

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 LE GARREC Vincent 2018-04-02 19:11:45 UTC
When I run run-clang-tidy.py without pyyaml installed, I have

Traceback (most recent call last):
  File "/usr/lib/llvm/6/share/clang/run-clang-tidy.py", line 51, in <module>
    import yaml
ImportError: No module named yaml

Should clang's ebuild must depend on pyyaml just for this script ?
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-04-03 21:42:02 UTC
To be honest, I don't think it's really a good idea to add a big dep on pyyaml just for one optional script in share. I see two options here:

a. a postinst message telling you to install pyyaml if you want to use it,

b. splitting this script into a separate package with pyyaml dep.

Both options suck in their own way. But I think it's not really worth the effort to make yet another package for one script.
Comment 2 Larry the Git Cow gentoo-dev 2018-04-04 13:56:10 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=64c144a8d2c2fce9015477b4a1f3d16ddc629baf

commit 64c144a8d2c2fce9015477b4a1f3d16ddc629baf
Author:     Michał Górny <mgorny@gentoo.org>
AuthorDate: 2018-04-04 12:27:49 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2018-04-04 13:56:03 +0000

    sys-devel/clang: Inform about utility scripts in postinst
    
    Closes: https://bugs.gentoo.org/652234

 sys-devel/clang/clang-5.0.1.ebuild    | 5 +++++
 sys-devel/clang/clang-6.0.0-r1.ebuild | 7 +++++++
 sys-devel/clang/clang-6.0.9999.ebuild | 7 +++++++
 sys-devel/clang/clang-9999.ebuild     | 7 +++++++
 4 files changed, 26 insertions(+)