New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
inputstream.adaptive: add bento4 dependancy #5975
Conversation
Given xbmc/inputstream.adaptive@70625e7 - bento4 is now not included in the @SupervisedThinking thanks for picking this up. |
|
Adding the below allows bento4 to use the inputstream.adaptive supplied patches - but now a recursive DEPEND-ancy
The patches do apply cleanly but build of bento4 fails.
|
da00bf6
to
9771056
Compare
@SupervisedThinking - this now builds successfully with bento4 as a dependancy. The ** Ideally bento4 should make sure that the |
That smells like a circular dependency? |
A way out: copy the patches to bento4 and check in inputstream.adaptive pre_build if they are the latest ones. |
o_O. why not move the patches where they belong. to bento4/patches/ ? |
@heitbaum as @stefansaraev said & I mentioned: just copy & paste the patches provided by inputstream.adaptive & add a note in the package file to bump the patchset of bento4 if inputstream is updated. |
Doesn't |
current state
|
@CvH and all others excellent feedback. I know how I want to fix this now. Just need to sit down and do it 👍 |
I have gone with |
Potential improvements:
inputstream.adaptive
happens first (due to the dependancy on bento4 in in TARGET)- but a DEPEND-ancy check should be put in place to ensure the unpackinputstream.adaptive
occurs first in thebento4/package.mk
file - making the patches available to bento4.Thoughts anyone?