xpra icon
Bug tracker and wiki

Opened 10 months ago

Closed 9 months ago

#587 closed defect (worksforme)

compile error in enc_x265

Reported by: stefops Owned by: stefops
Priority: major Milestone: 0.14
Component: encodings Version: 0.13.x
Keywords: Cc:

Description (last modified by totaam)

Error compiling Cython file:
------------------------------------------------------------
...
    assert colorspace in COLORSPACES, "invalid colorspace: %s (must be one of %s)" % (colorspace, COLORSPACES.keys())
    #ratings: quality, speed, setup cost, cpu cost, gpu cost, latency, max_w, max_h, max_pixels
    #we can handle high quality and any speed
    #setup cost is moderate (about 10ms)
    return video_codec_spec(encoding=encoding, output_colorspaces=COLORSPACES[colorspace],
                      Encoder, codec_type=get_type(),
                            ^
------------------------------------------------------------

xpra/codecs/enc_x265/encoder.pyx:242:29: Non-keyword arg following keyword arg
Traceback (most recent call last):
  File "setup.py", line 1620, in <module>
    setup_options["ext_modules"] = cythonize(ext_modules, gdb_debug=False)
  File "/usr/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 785, in cythonize
    cythonize_one(*args[1:])
  File "/usr/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 902, in cythonize_one
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: xpra/codecs/enc_x265/encoder.pyx
==> ERROR: A failure occurred in build().
    Aborting...

should it be

codec_class=Encoder,

Change History (2)

comment:1 Changed 10 months ago by totaam

  • Description modified (diff)
  • Owner changed from antoine to stefops
  • Summary changed from compile error to compile error in enc_x265

Yes, you're right I think.

It should be fixed in r6606. Let me know and I'll backport the fix.

Note: no-one should be using enc_x265 at this point because it is ridiculously slow, which is why it doesn't really get tested very often, and not even compile-tested since it isn't packaged for any of the major distros AFAIK.

comment:2 Changed 9 months ago by totaam

  • Resolution set to worksforme
  • Status changed from new to closed

Not heard back - closing.

Note: See TracTickets for help on using tickets.