Package Base Details: kodi-git

Git Clone URL: (read-only, click to copy)
Submitter: BlackIkeEagle
Maintainer: graysky
Last Packager: graysky
Votes: 82
Popularity: 0.000024
First Submitted: 2014-10-23 06:38
Last Updated: 2021-03-01 12:01

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 7 8 9 10 11 ... Next › Last »

Throne3d commented on 2017-11-15 22:54

I've noticed other packages use the "epoch" variable in the PKGBUILD file to prevent warnings about downgrades. The wiki mentions it – – but gives a warning that "epoch should only be used when absolutely required to do so", and I'm not sure if this is in fact one such situation.

It might be worth looking into?

Edit: also, though you may not care about such a minor thing: in the pkgver function, the echo line seems to have a trailing space (i.e. it's currently written 'echo "$_major.$_minor.$_revision" ', with the space at the end), which shows up rather noticeably when using git diff.

Lenovsky commented on 2017-11-12 00:10

The version format has changed, ignore the warning about downgrading package.

The new format is recommended by ArchWiki [1] and is following: RELEASE.rREVISION where REVISION is a number of revisions since beginning of the history.


Niklas commented on 2017-10-24 11:28

My pkgver uses this for kodi-git

printf "%s.%s.r%s.%s\n" "$(grep 'VERSION_MAJOR' version.txt | sed 's/[^0-9\.]*//g')" "$(grep 'VERSION_MINOR' version.txt | sed 's/[^0-9\.]*//g')" "$(git describe --tags --long | cut -d '-' -f 3)" "$(git rev-parse --short HEAD)"

result: 18.0.r4271.75bddb4e05

Lenovsky commented on 2017-10-23 23:23

First of all, I'd like to thank BlackIkeEagle for all the work. I'll do my best to keep it on track.

Second, I wonder if there's a specific reason why we don't use recommended version format: RELEASE.rREVISION? [1]. Git tags won't work, but we can easily use version.txt [2]. It may look like this:

_major=$(cat version.txt | grep "VERSION_MAJOR" | sed 's/VERSION_MAJOR //')
_minor=$(cat version.txt | grep "VERSION_MINOR" | sed 's/VERSION_MINOR //')
_revision=$(printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)")
echo "$_major.$_minor.$_revision"

And last but not least:
@graysky Thanks for pointing it out - it's already updated.


graysky commented on 2017-10-22 13:36

@Lenovsky - I did it, and it builds and runs. Please see:

graysky commented on 2017-10-22 12:43

You will probably want to update this to sync up with external ffmpeg. See the latest kodi PKGBUILD in [community].

cmake, deps, and makedepends are different

tixetsal commented on 2017-10-17 19:34

Thanks to BlackIkeEagle for all of the hard work, and long live Lenovsky!

a36233 commented on 2017-10-16 16:15

It fails building with:
kodi-git/src/xbmc/xbmc/guilib/FFmpegImage.cpp:254:3: error: ‘AVCodecParameters’ was not declared in this scope
AVCodecParameters* codec_params = m_fctx->streams[0]->codecpar;

kodi-git/src/xbmc/xbmc/guilib/FFmpegImage.cpp:254:22: error: ‘codec_params’ was not declared in this scope
AVCodecParameters* codec_params = m_fctx->streams[0]->codecpar;

kodi-git/src/xbmc/xbmc/guilib/FFmpegImage.cpp:264:7: error: ‘avcodec_parameters_to_context’ was not declared in this scope
if (avcodec_parameters_to_context(m_codec_ctx, codec_params) < 0)

kodi-git/src/xbmc/xbmc/guilib/FFmpegImage.cpp:395:9: error: ‘avcodec_send_frame’ was not declared in this scope
ret = avcodec_send_frame(avctx, frame);

Alad commented on 2017-09-28 11:01

All: if you use this package, please adopt it so others can continue to benefit.

BlackIkeEagle commented on 2017-09-23 06:07

I just disowned this, I don't have enough time to keep up with everything.