Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 544136 - split perl files out into sub package
Summary: split perl files out into sub package
Keywords:
Status: CLOSED CANTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: syslinux
Version: 14
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Peter Jones
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 559644
Blocks: FedoraMini F14Blocker, F14FinalBlocker
TreeView+ depends on / blocked
 
Reported: 2009-12-04 00:33 UTC by Peter Robinson
Modified: 2010-08-23 16:00 UTC (History)
4 users (show)

Fixed In Version: syslinux-4.02-3
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-08-23 16:00:06 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
split perl scripts into sub package (1.25 KB, application/octet-stream)
2010-01-14 23:18 UTC, Peter Robinson 		no flags Details
updates spec file for syslinux 4.02 (1.81 KB, patch)
2010-08-14 10:18 UTC, Peter Robinson 		no flags Details | Diff
Show Obsolete (1) View All

Description Peter Robinson 2009-12-04 00:33:39 UTC 
Up until version 3.75-3 the perl dependency was suppressed and anaconda/liveinst still worked. It would be great to split the perl utils out into a sub package so Fedora Mini related spins can include the liveinst feature without needing to pull in 40 odd meg of perl.
Comment 1 Peter Robinson 2010-01-14 23:17:58 UTC 
Looking at this a little closer it seems that there are two perl scripts used by anaconda.

First looks like its used to generate a random password from a python script. The second is for making isohybrid images.

[perobinson@neo anaconda-13.21]$ grep -R md5pass *
booty/x86.py:            args.append("--md5pass=%s" %(self.password))
kickstart.py:            if self.md5pass:
kickstart.py:                anaconda.id.bootloader.setPassword(self.md5pass)
[perobinson@neo anaconda-13.21]$ grep -R isohybrid *
anaconda.spec:- Use isohybrid to make boot.iso a hybrid image (katzj)
anaconda.spec.in:- Use isohybrid to make boot.iso a hybrid image (katzj)
scripts/mk-images.x86:        if [ -x /usr/bin/isohybrid ]; then
scripts/mk-images.x86:            isohybrid $TOPDESTPATH/images/$BOOTISO || echo "Unable to make hybrid boot.iso"
Comment 2 Peter Robinson 2010-01-14 23:18:49 UTC 
Created attachment 384485 [details]
split perl scripts into sub package
Comment 3 Bug Zapper 2010-03-15 13:26:54 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 13 development cycle.
Changing version to '13'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Comment 4 Mads Kiilerich 2010-03-30 18:39:40 UTC 
This would also mitigate the problem discussed in http://lists.fedoraproject.org/pipermail/livecd/2010-March/005706.html : that livecd-creator has hardcoded that syslinux must be installed into the image, even though its only use is that some of its files is copied to the isolinux folder.

I do however still think that the patch on http://lists.fedoraproject.org/pipermail/livecd/2010-March/005714.html would be a better solution to that problem.
Comment 5 Peter Robinson 2010-06-03 07:44:46 UTC 
With isohybrid now ported to C [1] there's no perl dep in anaconda so there's no reason not to split this out so as to be able to remove perl from most live CDs.

[1] https://fedorahosted.org/fedora-engineering-services/ticket/15
Comment 6 Matthias Clasen 2010-06-19 15:09:00 UTC 
This would be really nice. 

syslinux is the only thing pulling perl on the livecd now...
Comment 7 Bug Zapper 2010-07-30 10:48:02 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 14 development cycle.
Changing version to '14'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Comment 8 Peter Robinson 2010-08-14 10:18:05 UTC 
Created attachment 438791 [details]
updates spec file for syslinux 4.02

Updated patch for the spec file for the 4.02 release that has the new C based isohybrid
Comment 9 Peter Robinson 2010-08-14 10:19:05 UTC 
pjones: any chance of getting this in soon so we can remove the perl dep from the live images?
Comment 10 Matthias Clasen 2010-08-20 23:51:30 UTC 
So, syslinux 4.02 is out now, but it our package still has a perl dependency.
Can we please drop it ?
Comment 11 Kevin Fenzi 2010-08-21 00:49:56 UTC 
Yes please. This would be very nice to drop.
Comment 12 Peter Robinson 2010-08-23 16:00:06 UTC 
Fixed in: https://admin.fedoraproject.org/updates/syslinux-4.02-3.fc14

Please verify and add karma
Note You need to log in before you can comment on or make changes to this bug.