FS#61227 - [tribler] vlc is a hard dependency
Attached to Project:
Community Packages
Opened by cochise césar (cochise) - Tuesday, 01 January 2019, 02:20 GMT
Last edited by Filipe Laíns (FFY00) - Tuesday, 29 January 2019, 22:54 GMT
Opened by cochise césar (cochise) - Tuesday, 01 January 2019, 02:20 GMT
Last edited by Filipe Laíns (FFY00) - Tuesday, 29 January 2019, 22:54 GMT
|
Details
Description:
The package says vlc is a opt-depend, but he program don't start with it installed, so it should be a hard-dep Additional info: * package version(s) 7.2.0_exp1-3 Steps to reproduce: Install tribler without vlc installed. |
This task depends upon
Closed by Filipe Laíns (FFY00)
Tuesday, 29 January 2019, 22:54 GMT
Reason for closing: Fixed
Additional comments about closing: tribler 7.2.0_rc1-3
Tuesday, 29 January 2019, 22:54 GMT
Reason for closing: Fixed
Additional comments about closing: tribler 7.2.0_rc1-3
Error in sys.excepthook:
Traceback (most recent call last):
File "/usr/share/tribler/TriblerGUI/tribler_window.py", line 66, in on_exception
self.delete_tray_icon()
File "/usr/share/tribler/TriblerGUI/tribler_window.py", line 283, in delete_tray_icon
if self.tray_icon:
AttributeError: 'TriblerWindow' object has no attribute 'tray_icon'
Original exception was:
Traceback (most recent call last):
File "run_tribler.py", line 114, in <module>
window = TriblerWindow()
File "/usr/share/tribler/TriblerGUI/tribler_window.py", line 164, in __init__
self.video_player_page.initialize_player()
File "/usr/share/tribler/TriblerGUI/widgets/videoplayerpage.py", line 49, in initialize_player
self.instance = vlc.Instance()
File "/usr/share/tribler/TriblerGUI/vlc.py", line 1952, in __new__
return libvlc_new(len(args), args)
File "/usr/share/tribler/TriblerGUI/vlc.py", line 4408, in libvlc_new
ctypes.c_void_p, ctypes.c_int, ListPOINTER(ctypes.c_char_p))
File "/usr/share/tribler/TriblerGUI/vlc.py", line 331, in _Cfunction
raise NameError('no function %r' % (name,))
NameError: no function 'libvlc_new'
Aside: upstream has a truly horrible shellscript wrapper that redirects all terminal output to some random file in /tmp and spams the console with pushd messages (shellscripts never, ever need to return to their previous working directory as the shellscript will not leak out to the user's terminal, and moreover, when using "exec", succeeding lines like the one that calls popd will never even be read or executed by the shell -- the whole point of exec is to discard the shell).
We should replace this wrapper with a much simpler one that doesn't attempt to impede the user from locating terminal output for debugging.