9909 2002-10-29 12:26 0000 fluxbox-0.1.12-r2 fails if -nls is present in use variables 2003-02-04 19:42:18 0000 1 1 1 Unclassified Gentoo Linux Ebuilds 1.4_rc1 x86 Linux RESOLVED FIXED P3 minor --- 1 jrollins@charged-web.com mkeadle@gmail.com jrollins@charged-web.com 2002-10-29 12:26:39 0000 The remember patch provided on the fluxbox.org page states it will fail if nls is disabled - check for this in the use variables? jrollins@charged-web.com 2002-10-29 12:44:12 0000 Created an attachment (id=5181) Revised ebuild I think this oughta fix it - worked for me. Someone else want to test? mkeadle@gmail.com 2002-10-30 01:08:45 0000 See, here's the thing. The only difference between -r2 and -r1 is the remember patch. So removing the remeber patch leaves no reason to go to an -r2. Implementing the `use nls &&` works fine, but if that step is skipped you're left with no reason to run -r2. *This* is why it is ~ keyworded, and not put forward as a stable build. I'm aware of the issues with the patch and nls, and honestly I'm still trying to work out the best way to deal with it. I've thought of adding something like: if [ -n "`use nls`" ] then ewarn "You currently have -nls in your USE. This build of Fluxbox will not" ewarn "build if nls is disabled." exit 1 fi mkeadle@gmail.com 2002-10-30 01:10:50 0000 Also.. *PLEASE* when including a text attatchment remember to set the type as 'text/plain'. jrollins@charged-web.com 2002-10-30 10:43:14 0000 Sorry about that, I'm kinda new at this. I understand completely about the whole r2 thing - I had forgotten that i had put ~x86 in my make.conf. Thanks for checking into it - I'm sure whatever solution you come up with will work fine. Sorry for the confusion. kurt.hindenburg@gmail.com 2002-11-01 00:15:53 0000 Yea, I would go with the the test below; that way people have an idea what's up. if [ -n "`use nls`" ] then ewarn "You currently have -nls in your USE. This build of Fluxbox will not" ewarn "build if nls is disabled." exit 1 fi Kinda odd that the remember patch requires nls. I wonder if this a bug/misfeature in the remember patch. USE="nls" emerge fluxbox works fine with 0.1.12-r2 mkeadle@gmail.com 2002-11-04 16:44:51 0000 I've added the `if [ ! "`use nls`" ]` test to the existing -r2 ebuild. It directs the user to run `USE="nls" emerge fluxbox` if the test fails. This build is still ~ keyworded and not released to general population. mkeadle@gmail.com 2002-11-07 15:28:01 0000 I think this package is best left as "unstable", IE ~keyworded permanantly. I'm hearing rumors that Fluxbox 0.1.13 will have AA text and generally be overall neat. This remember patch is nice, but the nls thing coupled with rare crashes just won't make it into stable. I'm setting this to FIXED. I've you've got any issues with the 0.1.12-r2 release, consider running Fluxbox without the remember patch, waiting for 0.1.13, or dealing with the remember patch through other means. Cheers, and thanks for the help guys! 5181 2002-10-29 12:44 0000 Revised ebuild fluxbox-0.1.12-r10.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDAyIEdlbnRvbyBUZWNobm9sb2dpZXMsIEluYy4KIyBEaXN0cmli dXRlZCB1bmRlciB0aGUgdGVybXMgb2YgdGhlIEdOVSBHZW5lcmFsIFB1YmxpYyBMaWNlbnNlLCB2 MgojICRIZWFkZXI6IC9ob21lL2N2c3Jvb3QvZ2VudG9vLXg4Ni94MTEtd20vZmx1eGJveC9mbHV4 Ym94LTAuMS4xMi1yMi5lYnVpbGQsdiAxLjIgMjAwMi8xMC8yNiAwNTowNjozNSBta2VhZGxlIEV4 cCAkCgppbmhlcml0IGNvbW1vbmJveCBmbGFnLW8tbWF0aWMKClM9JHtXT1JLRElSfS8ke1B9CkRF U0NSSVBUSU9OPSJXaW5kb3cgbWFuYWdlciBiYXNlZCBvbiBCbGFja2JveCBhbmQgcHdtIC0tIGhh cyB0YWJzLiBSZW1lbWJlciBwYXRjaCBhcHBsaWVkLiIKU1JDX1VSST0ibWlycm9yOi8vc291cmNl Zm9yZ2UvJHtQTn0vJHtQfS50YXIuZ3oKCW1pcnJvcjovL2dlbnRvby8ke1BOfS1nZW50b28uZGlm Zi5iejIKCWh0dHA6Ly9mbHV4Ym94Lm9yZy9kb3dubG9hZC9wYXRjaGVzL3Vub2ZmaWNpYWwtZmx1 eGJveC0wLjEuMTItcmVtZW1iZXItcGF0Y2guYnoyIgpIT01FUEFHRT0iaHR0cDovL2ZsdXhib3gu c2YubmV0IgoKTElDRU5TRT0iR1BMLTIiClNMT1Q9IjAiCktFWVdPUkRTPSJ+eDg2IgoKbXlkb2M9 IkNoYW5nZUxvZyBDT1BZSU5HIE5FV1MiCm15Y29uZj0iLS1lbmFibGUteGluZXJhbWEiCmZpbHRl ci1mbGFncyAtZm5vLWV4Y2VwdGlvbnMKZXhwb3J0IFdBTlRfQVVUT01BS0VfMV82PTEKZXhwb3J0 IFdBTlRfQVVUT0NPTkZfMl81PTEKCnNyY191bnBhY2soKSB7CgoJdW5wYWNrICR7UH0udGFyLmd6 CgoJY2QgJHtTfQoJdXNlIG5scyAmJiBiemNhdCAke0RJU1RESVJ9L3Vub2ZmaWNpYWwtZmx1eGJv eC0wLjEuMTItcmVtZW1iZXItcGF0Y2guYnoyIHwgcGF0Y2ggLXAxIHx8IGRpZQoJYnpjYXQgJHtE SVNURElSfS8ke1BOfS1nZW50b28uZGlmZi5iejIgfCBwYXRjaCAtcDEgfHwgZGllCgoJc3NlZCAt aSAiczpibGFja2JveC5jYXQ6Zmx1eGJveC5jYXQ6IiBzcmMvbWFpbi5jYwp9CgpzcmNfY29tcGls ZSgpIHsKCgljb21tb25ib3hfc3JjX2NvbXBpbGUKCgljZCBkYXRhCgltYWtlIFwKCQlwa2dkYXRh ZGlyPS91c3Ivc2hhcmUvY29tbW9uYm94IGluaXQKfQoKCnNyY19pbnN0YWxsKCkgewoKCWNvbW1v bmJveF9zcmNfaW5zdGFsbAoJY2QgZGF0YQoJaW5zaW50byAvdXNyL3NoYXJlL2NvbW1vbmJveAoJ ZG9pbnMgaW5pdAoJaW5zaW50byAvdXNyL3NoYXJlL2NvbW1vbmJveC9mbHV4Ym94Cglkb2lucyBr ZXlzCn0K