[mythtv-commits] Ticket #7090: Updates for the rpm packaging scripts/spec
MythTV
mythtv at cvs.mythtv.org
Fri Sep 18 00:15:15 UTC 2009
#7090: Updates for the rpm packaging scripts/spec
----------------------------------------+-----------------------------------
Reporter: jtwyford at gmail.com | Owner: mdean
Type: enhancement | Status: new
Priority: minor | Milestone: unknown
Component: Contributed Scripts & Apps | Version: head
Severity: medium | Resolution:
Mlocked: 0 |
----------------------------------------+-----------------------------------
Comment(by jtwyford at gmail.com):
Replying to [comment:1 jarod]:
> Replying to [ticket:7090 jtwyford at gmail.com]:
> > I've been working on updates to both build_myth.sh and mythtv.spec for
some time to keep up with trunk and generally streamline things.
>
> Um... I fail to understand what needed changing... I've been building
weekly svn trunk packages in RPM Fusion...
Maybe I haven't been looking in the right place...
>
> > Primarily: -themes is now built as part of mythtv.spec and is not a
package of all its own. Thus build_myth.sh has been streamlined too.
However, installing via the script still makes a rather boneheaded
assumption that is flat out wrong most of the time.
>
> Do Not Want. Very BAD for packaging. The themes should be a noarch
package of its own. After release, the themes typically don't need to be
updated, but the code does. Including the themes in the base package means
that we'd be pushing out a new themes bundle every time we pushed an svn
fixes update package, which is a huge waste of bandwidth. No thanks. :)
It's still in its own noarch mythtv-themes subpackage, just getting built
via mythtv.spec, not via mythtv-themes.spec.
> > Also:[[BR]]
> > * mythbackend now runs as the no-privilege user 'mythtv' that gets
added during install instead of root.[[BR]]
>
> Doesn't work cleanly. On startup, the backend starts before a user has
logged in. The only user with read/write privs on tuner devices is root.
Chowning devices in the initscript is not particularly cool either
(haven't looked to see if you're doing this yet).
All I've done differently is add mythtv to the video group and the one-
liner to the init script.
>
> > * I've also dropped the old themes from the distribution and base-
themes temporarily from the spec.[[BR]]
>
> Dropping themes is not to be done lightly. A user that was using a theme
you dropped gets hosed if they update to a newer package that doesn't
include the theme they were using anymore.
/hastily does some readding
> > * Gone are all references to arts.[[BR]]
>
> This part we definitely want if arts isn't a mythtv-supported audio
option anymore...
>
> > * Various and sundry updates to the spec to get closer to passing
rpmlint.
>
> Cool.
>
> > Some issues still exist that I have reservations about, more or less
in order of priority:[[BR]]
> > 1. themes is huge (a 50MiB rpm on x86_64!), but which theme should we
include by default so as to recreate base-themes?[[BR]]
>
> The size is reason numero uno most of the themes have no business being
built in the main package, per above.
>
> > 2. I'm not entirely sure this spec will build on systems other than my
own. mock's failing at mythpainter_vdpau.cpp
([http://koji.fedoraproject.org/koji/buildinfo?buildID=132524 libvdpau] is
not yet in updates-testing/updates, so no BuildRequire / Require)[[BR]]
>
> I'll ping Nicholas Chauvet about building it for F11 too.
>
> > 3. Does the mythtv user/group still need to exist/be created?
>
> Probably not.
>
> > Why not some other already-defined non-privileged user?[[BR]]
>
> Such as? (And what for?)
That was supposed to be if instead of why... Mainly, one less network-
facing process running as root.
> > 5. rpmlint is complaining about a lot of 'unused-direct-shlib-
dependency'ies in /usr/lib64/*myth* (which are bugs of their own)[[BR]]
>
> Hrm. Yuk. There are also some selinux issues remaining, which might be
related... But meh. Only so many hours in the day...
>
> > 6. I've had to chmod some mythvideo/mythweb files in the spec to
properly set/remove their execute bits. Can we get those fixed in the
repo?
>
> Probably.
>
> > Current rpmlint output and `svn diff` from head are attached
>
> Will look 'em over in a bit. Kids need to be put to bed first...
Thanks for the pointers. Updates to the spec when I get back from dinner.
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/7090#comment:2>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list