205147 2008-01-10 08:17 0000 sys-apps/man-1.6f COMPRESS support breaks with app-arch/lzma 2009-01-08 02:01:31 0000 1 1 1 Unclassified Gentoo Linux Core system 2007.0 All Linux RESOLVED FIXED P2 normal --- 1 astralstorm@gen2.org base-system@gentoo.org cuciferus@gmail.com dany_it@yahoo.com darkside@gentoo.org gengor@gentoo.org jakub@gentoo.org jer@gentoo.org kerframil@gmail.com luke-jr+gentoobugs@utopios.org pacho@condmat1.ciencias.uniovi.es tomas.creemers@gmail.com astralstorm@gen2.org 2008-01-10 08:17:22 0000 sys-apps/man-1.6f has broken COMPRESS option in its man.conf It is also missing app-arch/lzma dependency. Another missing option is .lzma decompression. Correct COMPRESS line would be: COMPRESS /usr/bin/lzma e -si -so (see caveat below) Decompression should be: .lzma /usr/bin/lzma d -so Caveat: Fixing COMPRESS line uncovers a bug in man - any space in COMPRESS command flags it as unsafe. (I worked it around using a script) astralstorm@gen2.org 2008-01-10 08:28:35 0000 Only happens if you have app-arch/lzma installed at the time of the installation. vapier@gentoo.org 2008-01-10 08:47:12 0000 app-arch/lzma is not supported. use app-arch/lzma-utils if you want proper decompression support. astralstorm@gen2.org 2008-01-10 08:54:35 0000 If it's not supported (and breaks man), it should be masked. Please mask app-arch/lzma then. Also, please fix .lzma decompression line. jakub@gentoo.org 2008-01-10 09:03:44 0000 (In reply to comment #2) > app-arch/lzma is not supported. use app-arch/lzma-utils if you want proper > decompression support. vapier, you missed the point. sys-apps/man *default* to (misconfigure) lzma when you compile it w/ lzma installed. automagic, broken. vapier@gentoo.org 2008-01-10 09:06:53 0000 what i meant is that it isnt supported by man the lzma line is not broken if you use lzma-utils, thus there is nothing to fix no, i didnt miss the point ... man will add compression lines if binaries exist on your system ... the only thing that changes is the config line. whether i always force the .lzma line to "-d -c" or let the man configure script check for it the end result is the same: do not use lzma compression with man and the lzma package jakub@gentoo.org 2008-01-10 09:11:02 0000 vapier, this is plain broken automagic junk. You should get exact same package no matter what you have installed on your system, unless there's a USE flag to alter the default configuration. To reproduce and break your man out of the box: 1/ emerge app-arch/lzma 2/ emerge sys-apps/man 3/ read /etc/man.conf If you are refusing to fix this in sys-apps/man, then make it block app-arch/lzma. jakub@gentoo.org 2008-01-10 09:22:42 0000 (In reply to comment #5) > the lzma line is not broken if you use lzma-utils, thus there is nothing to fix Plus it's broken w/ lzma-utils as well 1/ emerge app-arch/lzma-utils 2/ emerge/upgrade sys-apps/man - it now defaults to lzma compression 3/ unmerge/depclean or whatnot app-arch/lzma-utils because you don't need it any more 4/ you have broken your man without any way whatsoever to detect why it happened Unless man always defaults to stuff that's guaranteed to exist on every system our there (.gz/.bz2) it can't be automagic. vapier@gentoo.org 2008-01-10 09:24:15 0000 i was thinking of the trailing suffix definitions for decompression bindings, not that COMPRESS is used for cat/cache maintenance ive added a blocker against app-arch/lzma jakub@gentoo.org 2008-01-10 09:45:54 0000 Well thanks, however as noted in Comment #7, lzma is not a sane preferred default for this... Oneliner patch attached, making the blocker unneeded as well. jakub@gentoo.org 2008-01-10 09:46:22 0000 Created an attachment (id=140597) man-1.6-bzip2-default-compress.patch vapier@gentoo.org 2008-01-10 09:50:22 0000 that argument is inaccurate. it applies to every compression in man, not just lzma-utils. jakub@gentoo.org 2008-01-10 09:56:23 0000 (In reply to comment #11) > that argument is inaccurate. it applies to every compression in man, not just > lzma-utils. Well, unless you want to add a bunch of use flags for every compression supported minus gz/bz2 instead of a trivial oneliner (that defaults to bzip2 which is *guaranteed* to exist on every Gentoo system and so man will always default to it), I don't see what you are after here. The current blocker doesn't fix the problem mentioned in Comment #7. jakub@gentoo.org 2008-01-15 07:09:28 0000 *** Bug 205903 has been marked as a duplicate of this bug. *** kerframil@gmail.com 2008-02-06 12:17:27 0000 I would concur that, if a compression method is employable by man via a package that isn't provided by `emerge system` - and we _want_ to support said method - then it perhaps ought to be subject to a USE flag to avoid the problem described in comment 7. That man is deciding during the build process which compression method it will use as a default is problematic. For man to decide to use lzma of its own accord thus makes lzma-utils a functional dependency beyond the knowledge of the package manager. On the other hand, the user can edit /etc/man.conf at any time (also beyond the knowledge of the package manager) but I think it's reasonable to assume that most users would not bother editing man.conf. I sure as heck don't. There can be no question that lzma is good at what it does. However, the main argument against using it is that the overheads during compression are significantly greater than bzip2, both in terms of CPU and memory useage. On the other hand, the memory requirements of lzma during decompression are similar to bzip2 and it's apparently faster at decompression. All in all, I think bzip2 should be a static default and I support the patch attached to this bug. Let's look at this in practical terms. If the user wants to switch to lzma (or any other compression method) then they can still do so, and they will know what they're doing. After all, they _chose_ to edit man.conf and they should be _aware_ of the requirements entailed by their changes. On the other hand, users who couldn't care less which compression method that man uses but want to employ lzma for other reasons (including myself) might well be surprised to encounter the problem that's the topic of bug 205903. We can't expect all users to be so judicious that they are going to scrutinise the new man.conf so closely as to notice 7zma sneaking in as a default compression method. I consider myself to be an advanced user and could easily envisage this catching me off guard. In fact, it's a bit of a pain - I actively don't want lzma as a compression method in man but want to continue to have lzma-utils installed on my system. Thus, I will need to remember to ignore that particular change upon each occasion that man is emerged in the future. So, please, let's just make bzip2 a default and let users who want to customise man.conf do so of their own accord. jakub@gentoo.org 2008-02-25 09:05:04 0000 Well, reopen. With coreutils not DEPENDing (but not RDEPENDing) on lzma-utils, everyone will be affected by this. Please make something that is *guaranteed* to exist on every system out there a default compressions, lzma-utils is not one of those. wwong@princeton.edu 2008-03-23 22:04:04 0000 In addition to app-arch/lzma, it also breaks with the lzma implementation in app-arch/p7zip (which provides /usr/bin/lzma). I am not an expert, just a user, but I think perhaps the same treatment for app-arch/lzma should apply to app-arch/p7zip if the lzma COMPRESS support is to be used. luke-jr+gentoobugs@utopios.org 2008-05-17 14:41:32 0000 I've just been hit with this too. IMO, USE=lzma makes the most sense. vapier@gentoo.org 2008-11-02 12:32:12 0000 *** Bug 245036 has been marked as a duplicate of this bug. *** darkside@gentoo.org 2008-12-07 22:21:28 0000 (In reply to comment #15) > Well, reopen. With coreutils not DEPENDing (but not RDEPENDing) on lzma-utils, > everyone will be affected by this. Please make something that is *guaranteed* > to exist on every system out there a default compressions, lzma-utils is not > one of those. > @base-system: The ability to view man pages is broken on binary hosts unless we can get something to rdepend on lzma-utils. In this case, I build in a chroot and that has lzma-utils installed because other packages DEPEND on it then man generates a man.conf that uses /usr/bin/lzma. lzma-utils does not get installed on the install host using emerge -K. Please consider adding lzma-utils to RDEPEND (the most proper, IMO) or maybe it is time to add lzma-utils to the system set? darkside@gentoo.org 2008-12-27 06:26:57 0000 Created an attachment (id=176484) compress patch if COMPRESS is set in the env as an abs path, then man will use that to do the compression. darkside@gentoo.org 2008-12-27 06:28:57 0000 Created an attachment (id=176485) man-1.6f-r2.ebuild.diff If USE=lzma is set then COMPRESS is set to /usr/bin/lzma. Initial testing seems to work as desired. darkside@gentoo.org 2009-01-08 02:01:31 0000 Cardoe reviewed my attached patches and approved. The new USE flag 'lzma' is now on 1.6f-r3. Please contact me if there are any issues. Also sent patch to email on the HOMEPAGE. 140597 2008-01-10 09:46 0000 man-1.6-bzip2-default-compress.patch man-1.6-bzip2-default-compress.patch text/plain LS0tIGNvbmZpZ3VyZS5vcmlnCTIwMDgtMDEtMTAgMTA6MzU6MzIuMDAwMDAwMDAwICswMTAwCisr KyBjb25maWd1cmUJMjAwOC0wMS0xMCAxMDozNzo0NS4wMDAwMDAwMDAgKzAxMDAKQEAgLTEwMTMs NyArMTAxMyw3IEBACiAgIHRoZW4KICAgICBET19DT01QUkVTU0lPTj10cnVlCiAgICAgY29tcHJl c3M9Ci0gICAgZm9yIGkgaW4gbHptYSBiemlwMiBnemlwIGJ6aXAgdHppcCBwYWNrIGNvbXByZXNz IGZyZWV6ZSB5YWJiYQorICAgIGZvciBpIGluIGJ6aXAyIGd6aXAgYnppcCBsem1hIHR6aXAgcGFj ayBjb21wcmVzcyBmcmVlemUgeWFiYmEKICAgICBkbwogICAgICAgZXZhbCBGJGk9bWlzc2luZwog ICAgICAgZm9yIGogaW4gJERFRlBBVEgK 176484 2008-12-27 06:26 0000 compress patch man-1.6f-compress.patch text/plain LS0tIGNvbmZpZ3VyZS5vcmlnCTIwMDgtMTItMjYgMjM6NTk6NDAuNzc5MDQyOTA2IC0wNjAwCisr KyBjb25maWd1cmUJMjAwOC0xMi0yNyAwMDoxMzoxNS4yNjUyNTYyMTUgLTA2MDAKQEAgLTEwMTIs MjAgKzEwMTIsMjUgQEAKICAgaWYgdGVzdCAiJGFucyIgPSAidHJ1ZSIKICAgdGhlbgogICAgIERP X0NPTVBSRVNTSU9OPXRydWUKLSAgICBjb21wcmVzcz0KLSAgICBmb3IgaSBpbiBsem1hIGJ6aXAy IGd6aXAgYnppcCB0emlwIHBhY2sgY29tcHJlc3MgZnJlZXplIHlhYmJhCi0gICAgZG8KLSAgICAg IGV2YWwgRiRpPW1pc3NpbmcKLSAgICAgIGZvciBqIGluICRERUZQQVRICi0gICAgICBkbwotICAg ICAgICBpZiB0ZXN0IC1mICRqLyRpCi0gICAgICAgIHRoZW4KLQkgIGV2YWwgRiRpPSRqLyRpCi0J ICBpZiBbIHgkY29tcHJlc3MgPSB4IF07IHRoZW4gY29tcHJlc3M9JGovJGk7IGZpCi0JICBicmVh awotICAgICAgICBmaQotICAgICAgZG9uZQotICAgIGRvbmUKKwlpZiBbIHgkQ09NUFJFU1MgIT0g eCBdOyB0aGVuCisJCWNvbXByZXNzPSRDT01QUkVTUworCQllY2hvICRjb21wcmVzcworCWVsc2UK KwkgICAgY29tcHJlc3M9CisJICAgIGZvciBpIGluIGx6bWEgYnppcDIgZ3ppcCBiemlwIHR6aXAg cGFjayBjb21wcmVzcyBmcmVlemUgeWFiYmEKKwkgICAgZG8KKwkgICAgICBldmFsIEYkaT1taXNz aW5nCisJICAgICAgZm9yIGogaW4gJERFRlBBVEgKKwkgICAgICBkbworCSAgICAgICAgaWYgdGVz dCAtZiAkai8kaQorCSAgICAgICAgdGhlbgorCQkgIGV2YWwgRiRpPSRqLyRpCisJCSAgaWYgWyB4 JGNvbXByZXNzID0geCBdOyB0aGVuIGNvbXByZXNzPSRqLyRpOyBmaQorCQkgIGJyZWFrCisJICAg ICAgICBmaQorCSAgICAgIGRvbmUKKwkgICAgZG9uZQorCWZpCiAKICAgICBpZiBbIHgkZGVmYXVs dCA9IHggXTsgdGhlbgogICAgICAgZWNobyAiIgo= 176485 2008-12-27 06:28 0000 man-1.6f-r2.ebuild.diff man-1.6f-r2.ebuild.diff text/plain SW5kZXg6IG1hbi0xLjZmLXIyLmVidWlsZAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvdmFyL2N2c3Jv b3QvZ2VudG9vLXg4Ni9zeXMtYXBwcy9tYW4vbWFuLTEuNmYtcjIuZWJ1aWxkLHYKcmV0cmlldmlu ZyByZXZpc2lvbiAxLjgKZGlmZiAtdSAtcjEuOCBtYW4tMS42Zi1yMi5lYnVpbGQKLS0tIG1hbi0x LjZmLXIyLmVidWlsZAkyIE9jdCAyMDA4IDE2OjUyOjIxIC0wMDAwCTEuOAorKysgbWFuLTEuNmYt cjIuZWJ1aWxkCTI3IERlYyAyMDA4IDA2OjI4OjM0IC0wMDAwCkBAIC0xMSwxMiArMTEsMTMgQEAK IExJQ0VOU0U9IkdQTC0yIgogU0xPVD0iMCIKIEtFWVdPUkRTPSJhbHBoYSBhbWQ2NCBhcm0gaHBw YSBpYTY0IG02OGsgfm1pcHMgcHBjIHBwYzY0IHMzOTAgc2ggc3BhcmMgfnNwYXJjLWZic2QgeDg2 IH54ODYtZmJzZCIKLUlVU0U9Im5scyIKK0lVU0U9Imx6bWEgbmxzIgogCiBERVBFTkQ9Im5scz8g KCBzeXMtZGV2ZWwvZ2V0dGV4dCApIgogUkRFUEVORD0iPj1zeXMtYXBwcy9ncm9mZi0xLjE5LjIt cjEKIAkhc3lzLWFwcHMvbWFuLWRiCi0JIWFwcC1hcmNoL2x6bWEiCisJIWFwcC1hcmNoL2x6bWEK Kwlsem1hPyAoIGFwcC1hcmNoL2x6bWEtdXRpbHMgKSIKIFBST1ZJREU9InZpcnR1YWwvbWFuIgog CiBwa2dfc2V0dXAoKSB7CkBAIC0zOSw2ICs0MCw3IEBACiAJZXBhdGNoICIke0ZJTEVTRElSfSIv bWFuLTEuNmQtZmJzZC5wYXRjaAogCWVwYXRjaCAiJHtGSUxFU0RJUn0iL21hbi0xLjZlLWhlYWRl cnMucGF0Y2gKIAllcGF0Y2ggIiR7RklMRVNESVJ9Ii9tYW4tMS42Zi1zby1zZWFyY2gucGF0Y2gK KwllcGF0Y2ggIiR7RklMRVNESVJ9Ii9tYW4tMS42Zi1jb21wcmVzcy5wYXRjaAogCiAJc3RyaXAt bGluZ3VhcyAkKGV2YWwgJChncmVwIF5MQU5HVUFHRVM9IGNvbmZpZ3VyZSkgOyBlY2hvICR7TEFO R1VBR0VTLy8sLyB9KQogfQpAQCAtNTgsNiArNjAsMTIgQEAKIAllbHNlCiAJCW15bGFuZz0ibm9u ZSIKIAlmaQorCWlmIHVzZSBsem1hOyB0aGVuCisJCW15Y29tcHJlc3M9L3Vzci9iaW4vbHptYQor CWVsc2UKKwkJbXljb21wcmVzcz0vYmluL2J6aXAyCisJZmkKKwlDT01QUkVTUz0kbXljb21wcmVz cyBcCiAJLi9jb25maWd1cmUgXAogCQktY29uZmRpcj0vZXRjIFwKIAkJK3NnaWQgK2ZocyBcCg==