[mythtv-commits] Ticket #7090: Updates for the rpm packaging scripts/spec

[MythTV]

#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 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...

 > 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. :)

 > 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).

 > * 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.

 > * 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?)

 > 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...

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7090#comment:1>
MythTV <http://www.mythtv.org/>
MythTV