Bug 158688 – app-misc/sl ebuild cleanup (respect LDFLAGS and compiler, use linguas_ja instead of cjk)

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Diego Elio Pettenò (RETIRED) 2006-12-20 15:16:03 UTC]

Just stumbled across this randomly, and seen that it uses the cjk useflag just to add the Japanese manpage; as that is not always good (people like me might have cjk enabled but don't ask for proper Japanese support), I've looked at the ebuild, so I'm attaching a cleanup diff.

Changes:
 - cjk useflag is replaced by linguas_ja (that translates to have "ja" in the LINGUAS variable);
 - you can drop the !ppc-macos dependency on nkf, if Fabian cannot make nkf to build on main tree, he can fix it for prefix or use package.use.mask;
 - virtual/libc is a moot dependency, just drop it;
 - using make to compile a single .c file in an executable is an overkill, especially since the Makefile does not add anything extra to the compile line; the current emake call also does not respect user-defined LDFLAGS or a crosscompiler, instead the use of gcc directly allows a simpler src_unpack and respect all the variables;
 - added a debug useflag to add -DDEBUG to the CFLAGS;
 - quoted all the variables that are user-defined.

HTH,
Diego

[Comment 1 Diego Elio Pettenò (RETIRED) 2006-12-20 15:16:29 UTC]

Created attachment 104470 [details, diff]
Ebuild patch

[Comment 2 Mamoru KOMACHI (RETIRED) 2006-12-30 04:35:36 UTC]

Absolutely. Thanks Diego, I applied the patch to existing ebuild, except
that it doesn't require nkf anymore and uses iconv to convert its manpage.
(if source encoding is fixed we usually don't need to use nkf, unless
source documents contain invalid characters)