Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 757537 - sys-devel/llvm should bdepend on dev-python/sphinx
Summary: sys-devel/llvm should bdepend on dev-python/sphinx
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: 2020-11-29 03:18 UTC by Alexey
Modified: 2020-11-29 08:59 UTC (History)
2 users (show)

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


Attachments
build.log (build.log,105.11 KB, text/x-log)
2020-11-29 03:18 UTC, Alexey
Details
emerge --info (file_757537.txt,20.36 KB, text/plain)
2020-11-29 03:18 UTC, Alexey
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey 2020-11-29 03:18:28 UTC
Created attachment 675661 [details]
build.log

At least, for version 12. Maybe earlier.

-- Targeting X86                                                                                                                                                                                                                                                                  
-- Registering Bye as a pass plugin (static build: OFF)                                                                                                                                                                                                                           
-- Sphinx enabled.                                                                                                                                                                                                                                                                
CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:164 (message):                                                                                                                                                                                        
  Failed to locate sphinx-build executable (missing: SPHINX_EXECUTABLE)                                                                                                                                                                                                           
Call Stack (most recent call first):                                                                                                                                                                                                                                              
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:445 (_FPHSA_FAILURE_MESSAGE)                                                                                                                                                                                       
  cmake/modules/FindSphinx.cmake:19 (find_package_handle_standard_args)                                                                                                                                                                                                           
  cmake/modules/AddSphinxTarget.cmake:5 (find_package)                                                                                                                                                                                                                            
  docs/CMakeLists.txt:106 (include)
Comment 1 Alexey 2020-11-29 03:18:54 UTC
Created attachment 675664 [details]
emerge --info
Comment 2 Ionen Wolkens gentoo-dev 2020-11-29 04:39:24 UTC
That's only needed for live ebuilds (that set LLVM_MANPAGES=build) given the others provide pregenerated man pages.

Issue looks like llvm.org_set_globals() does BDEPEND+="dev-python/sphinx" but then the ebuilds overwrites that later on with BDEPEND="..." and won't pull it.
Comment 3 Ionen Wolkens gentoo-dev 2020-11-29 04:56:20 UTC
(In reply to Ionen Wolkens from comment #2)
> then the ebuilds overwrites that later on
That function also makes changes to IUSE/RESTRICT but then they get overwritten in a similar fashion, although adding doc/test would be duplicates as it is right now.
Comment 4 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-11-29 08:20:53 UTC
Thanks for the analysis.  I'll fix that.
Comment 5 Larry the Git Cow gentoo-dev 2020-11-29 08:59:31 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=567a28e63c8db5205cff3c4b6f357c430d597e25

commit 567a28e63c8db5205cff3c4b6f357c430d597e25
Author:     Michał Górny <mgorny@gentoo.org>
AuthorDate: 2020-11-29 08:50:16 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2020-11-29 08:50:16 +0000

    sys-devel/llvm: Delay llvm.org_set_globals after vars
    
    Closes: https://bugs.gentoo.org/757537
    Signed-off-by: Michał Górny <mgorny@gentoo.org>

 sys-devel/llvm/llvm-11.0.0.9999.ebuild | 7 ++++---
 sys-devel/llvm/llvm-12.0.0.9999.ebuild | 7 ++++---
 2 files changed, 8 insertions(+), 6 deletions(-)