Posts: 11
Joined: Jul 2009
Reputation:
0
wojak
Junior Member
Posts: 11
Hello ace20022
I got this error when I compiled xbmc on linux system:
make[1]: leaving directory `/usr/local/src/xbmc-30-01-2013/lib/libdvd'
CPP xbmc/cores/dvdplayer/DVDAudio.o
CPP xbmc/cores/dvdplayer/DVDClock.o
CPP xbmc/cores/dvdplayer/DVDDemuxSPU.o
CPP xbmc/cores/dvdplayer/DVDFileInfo.o
In file included from DVDFileInfo.cpp:22:
DVDFileInfo.h:42: error: extra qualification ‘CDVDFileInfo::’ on member ‘AddExternalAudioToDetails’
make[1]: *** [DVDFileInfo.o] Error 1
make: *** [xbmc/cores/dvdplayer/DVDPlayer.a] Error 2
Regards.
Posts: 819
Joined: Aug 2012
Reputation:
35
Ace
Team-Kodi Member
Posts: 819
Hi wojak,
thanks for the info, I frequently forget to check linux. I have not pushed the fix to the repo yet, I'll inform you when done so.
Posts: 819
Joined: Aug 2012
Reputation:
35
Ace
Team-Kodi Member
Posts: 819
@wojak: done.
btw I'm working on a branch based on Frodo final which has pieh's language recognition code integrated.
Posts: 1,251
Joined: Dec 2008
Reputation:
15
A quick timeline of this PR:
- 7 months ago: The feature was implemented and a PR created. The following few weeks saw some healthy discussion about how to improve it, which was done.
- 6 months ago: Bump for review... no response from anyone responsible for handling it.
- 6 months + a few weeks ago: Bump for review... again no response.
- 2 weeks ago: Bump for review... no response.
- Today: The author of the commit finally got fed up and closed the PR, and I don't blame him after being ignored for 6 months.
I know the people responsible for the code review process has limited time and it's not a paid job. I love XBMC, and have the utmost respect for all the people who contribute to the project, but how this PR was handled was quite frankly disrespectful to the coder who spent -his- free time on this and had his contribution thrown away. Not a single response during a 6 month time period.
I hope no one responsible feels like this is an unfair "attack", but instead see it for how it was intended: A request for an introspective look at what made this fall through the cracks of the current review process, and a follow up discussion about what can be done to improve it going forward.
Posts: 31,445
Joined: Jan 2011
This has actually triggered an internal discussion on the Team about how we can better handle and communicate with people that have open pull requests. It sounds like a new pull request by Ace is going to be opened, and Spiff is going to look at it. In the future I think we'll have much better communication so people don't feel all alone about their PRs, and hopefully organize/sort through them better, get them to the right people faster, etc.
Posts: 1,251
Joined: Dec 2008
Reputation:
15
Glad to hear it, cause frustrated contributors is the last thing an open source project needs.
Posts: 12
Joined: Jun 2011
Reputation:
0
tomasg
Junior Member
Posts: 12
2014-02-23, 13:29
(This post was last modified: 2014-02-24, 20:12 by tomasg.)
Hello, is there a way to play bluray disc with external audio? Why I would like to have this feature is because sometimes i buy disc without desired language audio track. So I am able to download wanted audio track, but there is probably no way to play disc with it.
Posts: 30
Joined: Nov 2015
Reputation:
0
+1, I'd really love this option also, thanks!
Founder, Producer at
Techspin. Setup: i3-12100 PRO B660-A 8GB DDR4 | Kodi version updated, not sure
| Sony STR-AN1000 7.2 1155W 8K Atmos AVR | Samsung 65" 4K | Optoma HD26 3D + 100" screen.