Skip to content
This repository has been archived by the owner on Apr 15, 2023. It is now read-only.

install binaries in lib/ #8

Merged
merged 3 commits into from Aug 31, 2012
Merged

install binaries in lib/ #8

merged 3 commits into from Aug 31, 2012

Conversation

wsnipex
Copy link
Collaborator

@wsnipex wsnipex commented Aug 30, 2012

pvr addon binaries should go to $prefix/lib/xbmc/addons, like other .so files from normal addons.
addon data still goes to $prefix/share/xbmc/addons

This is needed for packaging, as $prefix/share/xbmc/addons goes in the arch independent package with the standard xbmc packaging scripts and therefore no binaries are allowed.

tested on ubuntu

edit: updated to honor --libdir and --datadir configure options.

@wsnipex
Copy link
Collaborator Author

wsnipex commented Aug 30, 2012

btw, this also needs a change in the xbmc configure script, because it calls the pvr-addon configure with --prefix="${prefix}/share/xbmc/addons" --includedir="${includedir}" --libdir="${prefix}/share/xbmc/addons" --datadir="${prefix}/share/xbmc/addons"

while it (for) now just should be --prefix="${prefix}"

@fiveisalive
Copy link

Yep, +1 from me. I maintain the Fedora/RPM Fusion package for XBMC and such packaging and having the lib / share split is a prerequisite for package acceptance. (Ultimately I plan to package the PVR add-ons as separate binary packages to the main XBMC, so it's good to see that they can be built from a separate source tree).

@opdenkamp
Copy link
Owner

@wsnipex looking good indeed. could you also create a PR for the xbmc tree that passes the correct options. thanks

@fiveisalive that's the idea behind all of this

opdenkamp pushed a commit that referenced this pull request Aug 31, 2012
install binaries in lib/
@opdenkamp opdenkamp merged commit d68f6d2 into opdenkamp:master Aug 31, 2012
@opdenkamp
Copy link
Owner

hmm merge commit wasn't supposed to be included. please rebase pull request branches next time.
i'll fix up the makefile from xbmc when updating the submodule

@wsnipex
Copy link
Collaborator Author

wsnipex commented Aug 31, 2012

uhm yeah, I meant to rebase but obviously forgot it :/
sorry about that!

On 08/31/2012 11:46 PM, Lars Op den Kamp wrote:

hmm merge commit wasn't supposed to be included. please rebase pull
request branches next time.
i'll fix up the makefile from xbmc when updating the submodule


Reply to this email directly or view it on GitHub
#8 (comment).

Jalle19 pushed a commit to Jalle19/xbmc-pvr-addons that referenced this pull request Mar 13, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Apr 17, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Apr 26, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Apr 27, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request May 8, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request May 13, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jun 8, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jun 8, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jul 13, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jul 16, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jul 16, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jul 20, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jul 20, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Aug 2, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Aug 18, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Aug 25, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Sep 9, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Oct 21, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Nov 11, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Nov 11, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Nov 12, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Nov 14, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Nov 29, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Dec 6, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Dec 13, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Dec 19, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Dec 28, 2014
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jan 2, 2015
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Jan 25, 2015
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Feb 17, 2015
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Feb 18, 2015
herrnst pushed a commit to herrnst/xbmc-pvr-addons that referenced this pull request Mar 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants