Bug 1450271
Summary: | Missing dependencies on command-line tools for converter plugin | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Yaakov Selkowitz <yselkowi> |
Component: | qmmp | Assignee: | Karel Volný <kvolny> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | kvolny |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-06-08 14:00:14 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 1449467 | ||
Bug Blocks: |
Description
Yaakov Selkowitz
2017-05-12 05:57:21 UTC
good catch, thanks, I've never used that plugin so I had no clue it calls commandline tools (I'm out of my clones to assign one to code reviews :-)) I've chosen the weak dependencies, I'm not in favor of pulling in everything forcefully 'just in case' - and while at it, suggesting the plugin-pack might be a good idea too I don't think this is worth update in stable branches (but it could sneak along another fix if it comes); I have prepared the change locally and I'll add it in Rawhide when updating to next qmmp version (which I believe to be around the corner) (In reply to Karel Volný from comment #1) > good catch, thanks, I've never used that plugin so I had no clue it calls > commandline tools (I'm out of my clones to assign one to code reviews :-)) Me neither, I only discovered it when I was looking for packages that might benefit from LAME support. > I've chosen the weak dependencies, I'm not in favor of pulling in everything > forcefully 'just in case' > > - and while at it, suggesting the plugin-pack might be a good idea too Makes sense to me. > I don't think this is worth update in stable branches (but it could sneak > along another fix if it comes); I have prepared the change locally and I'll > add it in Rawhide when updating to next qmmp version (which I believe to be > around the corner) It's certainly not urgent, thanks. https://src.fedoraproject.org/cgit/rpms/qmmp.git/commit/?id=c30d695a1ae6d03efb111815b66be4244a41df80 |