Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 18925 - mplayer 0.90 skins.
Summary: mplayer 0.90 skins.
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Alastair Tse (RETIRED)
URL: http://www.mplayer.hu
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-04-07 16:37 UTC by Terje Kvernes
Modified: 2003-04-07 22:13 UTC (History)
0 users

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


Attachments
the ebuild as an attachment. (mplayer-skins-0.2.ebuild,1.51 KB, text/plain)
2003-04-07 16:38 UTC, Terje Kvernes
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Terje Kvernes 2003-04-07 16:37:16 UTC
these skins should work fine with 0.90-pre as well.  it's basically a cleanup
and a bump of mplayer-skins-0.1-r1.  one version of the ebuild is as follows:

# Copyright 1999-2003 Gentoo Technologies, Inc.
# Distributed under the terms of the GNU General Public License v2
# $Header:
/home/cvsroot/gentoo-x86/x11-themes/mplayer-skins/mplayer-skins-0.1-r1.ebuild,v
1.2 2003/03/27 11:46:31 seemant Exp $

S=${WORKDIR}
DESCRIPTION="Collection of mplayer themes"
HOMEPAGE="http://www.mplayerhq.hu/"
THEME_URI="http://www.mplayerhq.hu/MPlayer/Skin"
SKINS="AlienMind-1.0.tar.bz2
Blue-1.0.tar.bz2
Blue-small-1.0.tar.bz2
BlueHeart-1.4.tar.bz2
Canary-1.0.tar.bz2
CornerMP-1.0.tar.bz2
CornerMP-aqua-1.0.tar.bz2
Cyrus-1.0.tar.bz2
MidnightLove-1.5.tar.bz2
Orange-1.0.tar.bz2
QPlayer-1.0.tar.bz2
WindowsMediaPlayer6-1.2.tar.bz2
avifile-1.5.tar.bz2
default-1.7.tar.bz2
disappearer-1.0.tar.bz2
gnome-1.1.tar.bz2
hayraphon-1.0.tar.bz2
hwswskin-1.0.tar.bz2
krystal-1.0.tar.bz2
mentalic-1.1.tar.bz2
neutron-1.4.tar.bz2
phony-1.0.tar.bz2
plastic-1.1.1.tar.bz2
proton-1.1.tar.bz2
slim-1.0.tar.bz2
softgrip-1.0.tar.bz2
trium-1.0.tar.bz2
xanim-1.5.tar.bz2
xine-lcd-1.0.tar.bz2
"
SLOT="0"
LICENSE="freedist"
KEYWORDS="x86 ppc sparc alpha"

DEPEND="net-misc/wget"
RDEPEND="media-video/mplayer
     sys-apps/bzip2"

for skin in $SKINS; do
       SRC_URI="${THEME_URI}/$skin $SRC_URI";
done

src_install () {
    dodir /usr/share/mplayer/Skin
    cp -dR * ${D}/usr/share/mplayer/Skin/
    chown -R root.root ${D}/usr/share/mplayer/Skin/
    chmod -R o-w ${D}/usr/share/mplayer/Skin/
    chmod -R a+rX ${D}/usr/share/mplayer/Skin/
}


this has the very nagging issue of needing to actually list all those pesky
skins.  one could use lynx -dump, cut and grep to actually be very dynamic and
all, but I'm not sure how that would work with digests?

anyhow, the ugly version is to generate SRC_URI as follows:

for skin in $( lynx -dump http://www.mplayerhq.hu/MPlayer/Skin/\
  | grep '. http' | grep bz2 | grep -v md5 | cut -f6 -d'/' ); do
   SRC_URI="${THEME_URI}/$skin $SRC_URI;
done

  we would of course have to list lynx as a requirement for mplayer-skins.  :-)
Comment 1 Terje Kvernes 2003-04-07 16:38:43 UTC
Created attachment 10342 [details]
the ebuild as an attachment.

I suppose you might want it as an attachment as well?  :-)
Comment 2 Alastair Tse (RETIRED) gentoo-dev 2003-04-07 18:44:08 UTC
ok, i committed a new version of the ebuild with more skins installed. but i didn't put in any of your suggestions with SRC_URI because:

a) portage does not handle bash scripting of SRC_URI very well.
b) wget'ing would be a very bad idea because of maintainability issues. who knows when suddenly a skin is in that directory that breaks the ebuild? and also digesting issues, etc.

even though dynamically fetching the skin names would protect us from version bumps, but this is a big price to pay. and as with (a), portage doesn't deal well with scripted SRC_URI.

oh, and Canary wasn't added because it was just plain not working.
Comment 3 Terje Kvernes 2003-04-07 22:13:03 UTC
well, you know best.  I've seen the SRC_URI build trick been done before though, so maybe some broader policies should be sanctioned?  and thanks for digging out the Canary skin, my apologies for submitting the build without testing it enough.  :-/

sigh.  at least there is room for improvement.