Bug 128637
Summary: | texinfo: install-info & bz2 files | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Steve Grubb <linux_4ever> | ||||
Component: | texinfo | Assignee: | Miloslav Trmač <mitr> | ||||
Status: | CLOSED RAWHIDE | QA Contact: | Ben Levenson <benl> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | ||||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | 4.8-9 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2006-01-16 05:22:12 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: | |||||||
Attachments: |
|
Description
Steve Grubb
2004-07-27 16:36:33 UTC
Created attachment 102287 [details]
patch currently being tested
In case you are considering option #1, I have attached a patch that uses full pathnames to the compressor/decompressors. Additionally, gunzip can handle .Z uncompress files. So gunzip was substituted for uncompress. I am currently testing in this mode and do not see anymore problems with install-info. I commented out the zlib patch at line 51 of the spec file and added the attached patch. Even if you may disagree with this approach, it is wiser to use full paths for the info utility helpers. The man utility uses full paths, so should info. I think they way forward is to fix the patch. Is this actually causing problems right now? We don't ship any .bz2 info files as far as I can see. It may be true that you do not ship bz2 info files, but that does not precude others from doing it. Info files that are bzip2'd are smaller. For example, tar.info.bz2 is 90007 while tar.info.gz is 115960. It really does add up when you have hundreds of them. If you do not like ripping out the zlib code, please use the fullpath part for the info utility. I think it was the first hunk of the patch. It might even be better to turn filesys.c into filesys.c.in and use configure/autoconf to insert the full path like 'man' does. I have changed the zlib patch to restore popen () use; gzipped files are handled with zlib, bzip2ed by piping from bzip2. Thanks for your report. |