Bug 635289

Summary: missing dep on netpbm-progs
Product: [Fedora] Fedora Reporter: Dennis Gilmore <dennis>
Component: fedora-logosAssignee: Tom "spot" Callaway <tcallawa>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 14CC: bruno, jonathan, notting, robatino, tcallawa, vanmeeuwen+fedora
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: fedora-logos-14.0.1-500.fc14 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-10-19 09:13:13 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Dennis Gilmore 2010-09-18 17:47:08 UTC
Description of problem:
/usr/share/anaconda/splashtolss.sh calls pngtopnm however its not installed though dependencies

Version-Release number of selected component (if applicable):


How reproducible:
always

Steps to Reproduce:
1. create a minimial install/chroot with anaconda installed
2. attempt to create an iso
3.
  
Actual results:
buildinstall failed

Expected results:
buildinstall to complete

Additional info:

Comment 1 Dennis Gilmore 2010-09-18 18:25:20 UTC
syslinux-perl is missing also

Comment 2 Brian Lane 2010-09-20 22:11:52 UTC
This is likely a by-product of bug 635330

Comment 3 Chris Lumens 2010-10-13 20:39:18 UTC
/usr/share/anaconda/splashtolss.sh is provided by fedora-logos.

Comment 4 Fedora Update System 2010-10-13 21:08:45 UTC
fedora-logos-14.0.0-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/fedora-logos-14.0.0-3.fc14

Comment 5 Fedora Update System 2010-10-14 13:51:17 UTC
fedora-logos-14.0.0-3.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update fedora-logos'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/fedora-logos-14.0.0-3.fc14

Comment 6 Bill Nottingham 2010-10-14 17:10:04 UTC
I'm a bit confused here. The anaconda code is:

        if [ -f $IMGPATH/usr/share/anaconda/syslinux-vesa-splash.jpg ]; then
                cp $IMGPATH/usr/share/anaconda/syslinux-vesa-splash.jpg $MBD_BOO
TTREE/splash.jpg
                cp $IMGPATH/usr/share/syslinux/vesamenu.c32 $MBD_BOOTTREE/vesame
nu.c32
                sed -i s'/default linux/default vesamenu.c32/g' $MBD_BOOTTREE/sy
slinux.cfg
                sed -i 's/prompt 1/#prompt 1/g' $MBD_BOOTTREE/syslinux.cfg
        elif [ -x $IMGPATH/usr/share/anaconda/splashtolss.sh ]; then
                $IMGPATH/usr/share/anaconda/splashtolss.sh $BOOTDISKDIR/syslinux-splash.png $BOOTDISKDIR/splash.lss
                if [ $? != 0 ]; then
                    echo $0: Failed to create splash.lss
                    exit 1
                fi    
                cp $BOOTDISKDIR/splash.lss $MBD_BOOTTREE/splash.lss
        elif [ -f $IMGPATH/usr/share/anaconda/splash.lss ]; then
                cp $IMGPATH/usr/share/anaconda/splash.lss $MBD_BOOTTREE/splash.lss
        fi

Can't we just have the -logos package ship either syslinux-vesa-splash.jpg or splash.lss, and have anaconda skip the conversion?

Comment 7 Chris Lumens 2010-10-14 18:25:15 UTC
> Can't we just have the -logos package ship either syslinux-vesa-splash.jpg or
> splash.lss, and have anaconda skip the conversion?

It sure seems that way to me.  The commit that introduced this block is 24d01808a35b3f827223602559e3731e854fae45, but it's not really more enlightening.

Comment 8 Tom "spot" Callaway 2010-10-14 18:36:52 UTC
Is there a preference? I'm happy to rework this part.

Comment 9 Bruno Wolff III 2010-10-14 20:25:58 UTC
Pulling in syslinux-perl is a problem. We went to a lot of trouble to make it so perl wasn't required by live spins. This might end up making something go over its target size right before the release.

Comment 10 Andre Robatino 2010-10-15 18:28:01 UTC
Right now, fedora-logos is included in a minimal F14 Final TC1 install, but no *perl* package is. The dependency on syslinux-perl (which indirectly depends on perl) would cause a major size increase.

Comment 11 Fedora Update System 2010-10-15 19:44:00 UTC
fedora-logos-14.0.1-500.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/fedora-logos-14.0.1-500.fc14

Comment 12 Fedora Update System 2010-10-19 09:13:03 UTC
fedora-logos-14.0.1-500.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.