Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 293093 - media-tv/xbmc-9999 should pull in all webui styles
Summary: media-tv/xbmc-9999 should pull in all webui styles
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo Xbox project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-13 18:22 UTC by hal
Modified: 2010-05-23 18:32 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to pull in iphone style, and remove previously added "project mayhem" style (not project mayhem 3) (xbmc-9999-web-iphone.patch,371 bytes, patch)
2009-12-07 00:47 UTC, hal
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description hal 2009-11-13 18:22:34 UTC
the xbmc trunk contains different styles for the webui. they are located in:

-> /usr/portage/distfiles/svn-src/xbmc/trunk/web/

these are:
- Project_Mayhem
- Project_Mayhem_III
- xbmciphone

it would be nice to see these interfaces getting installed via the ebuild, too. just like the project_mayhem webui.

they should be placed into:

-> /usr/share/xbmc/web/styles/

Reproducible: Always

Steps to Reproduce:
1. emerge xbmc-9999
2. ls -la /usr/share/xbmc/web/styles/


Actual Results:  
only one webui available

Expected Results:  
all webuis should be available
Comment 1 hal 2009-11-13 20:56:09 UTC
maybe it would be an option to pull in additional webuis via a new useflag. e.g. USE="webui-extra" could be used to accomplish that.
Comment 2 SpanKY gentoo-dev 2009-12-04 08:22:44 UTC
i dont see much point in putting this behind a USE flag.  i also think all the themes but the default suck, but maybe that's just me.

what exactly needs to happen ?  something like:
insinto /usr/share/xbmc/web/styles/
doins -r "${S}"/web/* || die
Comment 3 hal 2009-12-04 08:34:30 UTC
(In reply to comment #2)
> i dont see much point in putting this behind a USE flag.  i also think all the
> themes but the default suck, but maybe that's just me.

well, that was just a proposal from a non ebuild developer. ;) it's up to you to decide whether this would make sense or not.
i think it's fine to have the default style and the iphone style. that said we would have two styles, one for mobile devices and one for devices coming along with a regular display.

> what exactly needs to happen ?  something like:
> insinto /usr/share/xbmc/web/styles/
> doins -r "${S}"/web/* || die

the latest revision shows me the following in the working copy:
" ls -la /usr/portage/distfiles/svn-src/xbmc/trunk/web/
total 24
drwxr-sr-x  6 root portage 4096 Nov 24 00:44 .
drwxr-sr-x 21 root portage 4096 Dec  2 17:27 ..
drwxr-sr-x  6 root portage 4096 Dec  4 08:38 .svn
drwxr-sr-x  4 root portage 4096 Nov 24 00:44 Project_Mayhem
drwxr-sr-x  4 root portage 4096 Nov 24 00:44 Project_Mayhem_III
drwxr-sr-x  4 root portage 4096 Nov 24 00:44 xbmciphone
"

these should put in:

ls -la /usr/share/xbmc/web/styles/
total 12
drwxr-xr-x 3 root root 4096 Nov 23 09:57 .
drwxrwxrwx 3 root root 4096 Dec  4 08:53 ..
drwxr-xr-x 3 root root 4096 Dec  4 08:53 Project_Mayhem


Comment 4 hal 2009-12-04 08:37:06 UTC
> > what exactly needs to happen ?  something like:
> > insinto /usr/share/xbmc/web/styles/
> > doins -r "${S}"/web/* || die
> 
> the latest revision shows me the following in the working copy:
> " ls -la /usr/portage/distfiles/svn-src/xbmc/trunk/web/
> total 24
> drwxr-sr-x  6 root portage 4096 Nov 24 00:44 .
> drwxr-sr-x 21 root portage 4096 Dec  2 17:27 ..
> drwxr-sr-x  6 root portage 4096 Dec  4 08:38 .svn
> drwxr-sr-x  4 root portage 4096 Nov 24 00:44 Project_Mayhem
> drwxr-sr-x  4 root portage 4096 Nov 24 00:44 Project_Mayhem_III
> drwxr-sr-x  4 root portage 4096 Nov 24 00:44 xbmciphone
> "
> 
> these should put in:
> 
> ls -la /usr/share/xbmc/web/styles/
> total 12
> drwxr-xr-x 3 root root 4096 Nov 23 09:57 .
> drwxrwxrwx 3 root root 4096 Dec  4 08:53 ..
> drwxr-xr-x 3 root root 4096 Dec  4 08:53 Project_Mayhem
> 

well not the entire folders should put to the destination dir, but  only the styles folders found inside the webskins folders in trunk.
Comment 5 SpanKY gentoo-dev 2009-12-04 09:32:44 UTC
should be all set now

http://sources.gentoo.org/media-tv/xbmc/xbmc-9999.ebuild?r1=1.41&r2=1.42
Comment 6 hal 2009-12-04 10:08:14 UTC
(In reply to comment #5)
> should be all set now
> 
> http://sources.gentoo.org/media-tv/xbmc/xbmc-9999.ebuild?r1=1.41&r2=1.42
> 

i just tested the cahnges reulting in the followwing:

"ls -la /usr/share/xbmc/web/styles/
total 16
drwxr-xr-x 4 root root 4096 Dec  4 11:04 .
drwxrwxrwx 3 root root 4096 Dec  4 11:04 ..
drwxr-xr-x 3 root root 4096 Dec  4 11:04 Project Mayhem
drwxr-xr-x 3 root root 4096 Dec  4 11:04 Project_Mayhem"
Comment 7 hal 2009-12-04 10:09:04 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > should be all set now
> > 
> > http://sources.gentoo.org/media-tv/xbmc/xbmc-9999.ebuild?r1=1.41&r2=1.42
> > 
> 
> i just tested the cahnges reulting in the followwing:
> 
> "ls -la /usr/share/xbmc/web/styles/
> total 16
> drwxr-xr-x 4 root root 4096 Dec  4 11:04 .
> drwxrwxrwx 3 root root 4096 Dec  4 11:04 ..
> drwxr-xr-x 3 root root 4096 Dec  4 11:04 Project Mayhem
> drwxr-xr-x 3 root root 4096 Dec  4 11:04 Project_Mayhem"
> 

oops, sorry for the typos!
Comment 8 SpanKY gentoo-dev 2009-12-04 10:22:37 UTC
are you saying it doesnt work for you ?
Comment 9 hal 2009-12-04 10:29:47 UTC
(In reply to comment #8)
> are you saying it doesnt work for you ?
> 

exactly. the iphone style is not getting installed, instead i get project mayhem a second time, but without an underscore.
Comment 10 SpanKY gentoo-dev 2009-12-04 22:24:16 UTC
there is no 'styles' directory under the iphone dir, nor 'screen.css'
Comment 11 hal 2009-12-07 00:45:32 UTC
(In reply to comment #10)
> there is no 'styles' directory under the iphone dir, nor 'screen.css'
> 

mh sorry, my fault. 
i appended a patch that pulls in the iphone gui, that's then accessible via "http://IP:8080/iphone/index.html"
the patch also removes the project mayhem style again, that was pulled in with the last change in the ebuild. i have to agree with you that the standard style together with the iphone style is enough.
Comment 12 hal 2009-12-07 00:47:13 UTC
Created attachment 212287 [details, diff]
patch to pull in iphone style, and remove previously added "project mayhem" style (not project mayhem 3)
Comment 13 Oo.et.oO 2010-01-06 04:49:42 UTC
should i be using this ebuild, or is there going to be one for the "stable version again?"

using this ebuild i build fine but i get:

unknown chip id 0x95c5, can't guess.
unknown chip id 0x95c5, can't guess.
unknown chip id 0x95c5, can't guess.
ERROR: Unable to create application. Exiting

i have an ATI radeonHD, vdpau use flag is turned off
Comment 14 Michael Lange 2010-02-19 17:32:27 UTC
with the latest git version i get this

* ERROR: media-tv/xbmc-9999 failed:
 *   (no error message)
 * 
 * Call stack:
 *     ebuild.sh, line  54:  Called src_install
 *   environment, line 3040:  Called die
 * The specific snippet of code:
 *       doins -r "${S}"/web/*/styles/*/ || die;
 * 
 * If you need support, post the output of 'emerge --info =media-tv/xbmc-9999',
 * the complete build log and the output of 'emerge -pqv =media-tv/xbmc-9999'.
 * The complete build log is located at '/var/tmp/portage/media-tv/xbmc-9999/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/media-tv/xbmc-9999/temp/environment'.
 * S: '/var/tmp/portage/media-tv/xbmc-9999/work/xbmc-9999'
 * QA Notice: file does not exist:
 * 
 * 	doins: /var/tmp/portage/media-tv/xbmc-9999/work/xbmc-9999/web/*/styles/*/ does not exist


seems the web-styles are removed
look at http://trac.xbmc.org/changeset/27801/trunk/web
Comment 15 hal 2010-02-19 17:43:24 UTC
(In reply to comment #14)
> with the latest git version i get this
> 
> * ERROR: media-tv/xbmc-9999 failed:
>  *   (no error message)
>  * 
>  * Call stack:
>  *     ebuild.sh, line  54:  Called src_install
>  *   environment, line 3040:  Called die
>  * The specific snippet of code:
>  *       doins -r "${S}"/web/*/styles/*/ || die;
>  * 
>  * If you need support, post the output of 'emerge --info =media-tv/xbmc-9999',
>  * the complete build log and the output of 'emerge -pqv =media-tv/xbmc-9999'.
>  * The complete build log is located at
> '/var/tmp/portage/media-tv/xbmc-9999/temp/build.log'.
>  * The ebuild environment file is located at
> '/var/tmp/portage/media-tv/xbmc-9999/temp/environment'.
>  * S: '/var/tmp/portage/media-tv/xbmc-9999/work/xbmc-9999'
>  * QA Notice: file does not exist:
>  * 
>  *      doins:
> /var/tmp/portage/media-tv/xbmc-9999/work/xbmc-9999/web/*/styles/*/ does not
> exist
> 
> 
> seems the web-styles are removed
> look at http://trac.xbmc.org/changeset/27801/trunk/web
> 

michael,

your are right. this is because team xbmc started to replace the webinterface recently. see http://bugs.gentoo.org/show_bug.cgi?id=305229 for details.

therefore the 9999 ebuild can't install the skins in case one is compiling xbmc from svn source. i guess this bug report should be considered as obsolete. nevertheless we could need an update of the 9999 ebuild to avoid possible failures during installation. also there is a new skin for the json-rpc api available. there's not much to see atm, everything is rather at an experimental stage.