154353 2006-11-07 05:24 0000 Add jpeg2k use flag to media-libs/netpbm 2006-11-07 12:40:13 0000 1 1 1 Unclassified Gentoo Linux Applications unspecified All Linux RESOLVED FIXED P2 normal --- 1 betelgeuse@gentoo.org graphics@gentoo.org betelgeuse@gentoo.org 2006-11-07 05:24:35 0000 Like the other use flags, jasper is not mandatory so I added a jpeg2k use flag for it. betelgeuse@gentoo.org 2006-11-07 05:41:34 0000 Created an attachment (id=101391) Patch against netpbm-10.36.ebuild lu_zero@gentoo.org 2006-11-07 06:18:35 0000 Looks fine, please commit =) vapier@gentoo.org 2006-11-07 10:31:58 0000 that isnt how it works ... netpbm will use the bundled jasper version of you set that value to NONE and that certainly is not correct betelgeuse@gentoo.org 2006-11-07 12:04:00 0000 (In reply to comment #3) > that isnt how it works ... netpbm will use the bundled jasper version of you > set that value to NONE and that certainly is not correct > Please read converter/other/jpeg2000/Makefile again. Relevant code: ifneq ($(JASPERHDR_DIR),NONE) ifneq ($(JASPERLIB_USE),NONE) BINARIES = pamtojpeg2k jpeg2ktopam endif endif betelgeuse@gentoo.org 2006-11-07 12:40:13 0000 (In reply to comment #3) > that isnt how it works ... netpbm will use the bundled jasper version of you > set that value to NONE and that certainly is not correct > I verified this with rm -r libjasper and ebuild netpbm-10.36.ebuild compile worked just fine. install had problems because of the SUBDIRS variable. I could of course send a patch upstream that disables the libjasper SUBDIR when jasper support is off and then we can just rm the internal copy. Any way this bug can be marked fixed.w 101391 2006-11-07 05:41 0000 Patch against netpbm-10.36.ebuild netpbm-10.36.patch text/plain LS0tIC9yb290L25ldHBibS0xMC4zNi4wLmVidWlsZAkyMDA2LTExLTA3IDE1OjM3OjQwLjAwMDAw MDAwMCArMDIwMAorKysgbmV0cGJtLTEwLjM2LjAuZWJ1aWxkCTIwMDYtMTEtMDcgMTU6Mzg6MTku MDAwMDAwMDAwICswMjAwCkBAIC0xMywxNiArMTMsMTYgQEAKIExJQ0VOU0U9IkdQTC0yIgogU0xP VD0iMCIKIEtFWVdPUkRTPSJ+YWxwaGEgfmFtZDY0IH5hcm0gfmhwcGEgfmlhNjQgfm1pcHMgfnBw YyB+cHBjNjQgfnMzOTAgfnNoIH5zcGFyYyB+eDg2IH54ODYtZmJzZCIKLUlVU0U9InN2Z2EganBl ZyB0aWZmIHBuZyB4bWwgemxpYiIKK0lVU0U9InN2Z2EganBlZyBqcGVnMmsgdGlmZiBwbmcgeG1s IHpsaWIiCiAKIERFUEVORD0ianBlZz8gKCA+PW1lZGlhLWxpYnMvanBlZy02YiApCisJanBlZzJr PyAoIG1lZGlhLWxpYnMvamFzcGVyICkKIAl0aWZmPyAoID49bWVkaWEtbGlicy90aWZmLTMuNS41 ICkKIAlwbmc/ICggPj1tZWRpYS1saWJzL2xpYnBuZy0xLjIuMSApCiAJeG1sPyAoIGRldi1saWJz L2xpYnhtbDIgKQogCXpsaWI/ICggc3lzLWxpYnMvemxpYiApCiAJc3ZnYT8gKCBtZWRpYS1saWJz L3N2Z2FsaWIgKQogCW1lZGlhLWxpYnMvamJpZ2tpdAotCW1lZGlhLWxpYnMvamFzcGVyCiAJbWVk aWEtbGlicy91cnQiCiAKIG5ldHBibV9saWJ0eXBlKCkgewpAQCAtNDUsNiArNDUsMTAgQEAKIAl1 c2UgJDEgJiYgZWNobyAtbCR7MjotJDF9IHx8IGVjaG8gTk9ORQogfQogCituZXRwYm1famFzcGVy KCkgeworCXVzZSAhanBlZzJrICYmIGVjaG8gTk9ORQorfQorCiBzcmNfdW5wYWNrKCkgewogCXVu cGFjayAke0F9CiAJY2QgIiR7U30iCkBAIC04Myw3ICs4Nyw3IEBACiAJSkJJR0xJQiA9IC1samJp ZwogCUpCSUdIRFJfRElSID0KIAlKQVNQRVJMSUIgPSAtbGphc3BlcgotCUpBU1BFUkhEUl9ESVIg PQorCUpBU1BFUkhEUl9ESVIgPSAkKG5ldHBibV9qYXNwZXIpCiAJVVJUTElCID0gLWxybGUKIAlV UlRIRFJfRElSID0KIAlFT0YK