Skip to content
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

kodi-addon-inputstream-adaptive: update to 21.4.4. #54016

Conversation

thespookman
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, (x86_64)
  • I built this PR locally for these architectures:
    • aarch64
    • aarch64-musl
    • armv6l
    • armv6l-musl
    • armv7l
    • armv7l-musl
    • x86_64-musl

Closes #53872

checksum=353207f5f98bf81ce8d79cec903c28da8c0227d7c7a632692c910d81e59c2dbe
# Bento4 version defined in https://github.com/xbmc/inputstream.adaptive/blob/Piers/depends/common/bento4/bento4.txt
_bento4_version=1.6.0-641-Omega
_bento4_tarball=${_bento4_version}.tar.gz
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is only referenced in a couple places, can those places just use ${_bento4_version}.tar.gz directly?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any objection if I just remove the _bento4_version variable instead? I've realised I no longer reference that anywhere else

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no that's still useful. we usually put those variables up near the primary version (so after the _kodi_release line)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation, done.


# Inputstream Adaptive needs the unextracted tarball
skip_extraction="${_bento4_tarball}"
configure_args=" -DENABLE_INTERNAL_BENTO4=1 -DBENTO4_URL=${XBPS_SRCDISTDIR}/${pkgname}-${version}/${_bento4_tarball} -DVERBOSE=1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should go up in the first block, after build_style

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@thespookman thespookman force-pushed the update-kodi-addon-inputstream-adaptive branch from 01763e5 to 7b40831 Compare January 19, 2025 08:23
@classabbyamp classabbyamp merged commit c3a36c8 into void-linux:master Jan 19, 2025
8 checks passed
@thespookman thespookman deleted the update-kodi-addon-inputstream-adaptive branch January 19, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kodi-addon-inputstream-adaptive is incompatible with the version of kodi in the repository
2 participants