Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 716354 - media-libs/virglrenderer-0.6.0 and 0.7.0 build requires python2 but the ebuild doesn't say it does
Summary: media-libs/virglrenderer-0.6.0 and 0.7.0 build requires python2 but the ebuil...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-05 17:20 UTC by gen2dev
Modified: 2020-04-19 20:44 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 gen2dev 2020-04-05 17:20:00 UTC
Emerging media-libs/virglrenderer-0.6.0 on a gentoo machine which does not have python 2.7 installed doesn't pull in python 2.7 and fails because it requires it.
   ...
   make[3]: Entering directory '/var/tmp/portage/media-libs/virglrenderer-0.6.0/work/virglrenderer-0.6.0/src/gallium/auxiliary'
   /bin/mkdir -p util
   python2 ./util/u_format_table.py ./util/u_format.csv > util/u_format_table.c
   python2: no python-exec wrapped executable found in /usr/lib/python-exec.
   make[3]: *** [Makefile:901: util/u_format_table.c] Error 127

virglrenderer-0.6.0.ebuild doesn't say anything about needing python2 (or any python), so portage didn't provide it. virglrenderer-0.7.0.ebuild, which is still masked, too.

Also on gentoo looking for an executable "python2" to decide if python 2 is available fails, because python-exec always creates /usr/bin/python2 whether or not python 2 is actually available.

By the way, 0.7.0 mentions python3 in it's configure.ac but 1) it still prefers python2, and 2) when I hacked it to prefer python3 it failed I guess because the scripts are not python3-ready yet. Fair enough for a masked package.
  Traceback (most recent call last):
  File "./util/u_format_table.py", line 168, in <module>
    main()
  File "./util/u_format_table.py", line 164, in main
    write_format_table(formats)
  File "./util/u_format_table.py", line 132, in write_format_table
    print("   %s,\t/* is_array */" % (bool_map(format.is_array()),))
  File "/var/tmp/portage/media-libs/virglrenderer-0.7.0/work/virglrenderer-0.7.0/src/gallium/auxiliary/util/u_format_parse.py", line 164, in is_array
    return self.array_element() != None
  File "/var/tmp/portage/media-libs/virglrenderer-0.7.0/work/virglrenderer-0.7.0/src/gallium/auxiliary/util/u_format_parse.py", line 73, in __eq__
    return self.type == other.type and self.norm == other.norm and self.pure == other.pure and self.size == other.size
  AttributeError: 'NoneType' object has no attribute 'type'
  make[3]: *** [Makefile:905: util/u_format_table.c] Error 1
Comment 1 Larry the Git Cow gentoo-dev 2020-04-19 20:44:49 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=73004619cc8ba46a7ee42b1496dbf0f064ccc87e

commit 73004619cc8ba46a7ee42b1496dbf0f064ccc87e
Author:     Matthias Maier <tamiko@gentoo.org>
AuthorDate: 2020-04-19 20:01:23 +0000
Commit:     Matthias Maier <tamiko@gentoo.org>
CommitDate: 2020-04-19 20:44:35 +0000

    media-libs/virglrenderer: version bump to 0.8.2
    
     - contrary to the suggestion in https://bugs.gentoo.org/693246, do not
       add a USE=gbm flag. The current code cannot be compiled with
       mesa[gbm] and upstream restricted support for gbm allocation to
       minigbm in current master
    
     - port to meson build system
    
     - disable testsuite -
    
     - verified that the build system doesn't require python directly any more
    
    Closes: https://bugs.gentoo.org/693246
    Closes: https://bugs.gentoo.org/716354
    Closes: https://bugs.gentoo.org/631740
    
    Package-Manager: Portage-2.3.99, Repoman-2.3.22
    Signed-off-by: Matthias Maier <tamiko@gentoo.org>

 media-libs/virglrenderer/Manifest                  |  1 +
 .../virglrenderer/virglrenderer-0.8.2.ebuild       | 46 ++++++++++++++++++++++
 2 files changed, 47 insertions(+)