Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 50823 - ng-1.5_beta1.ebuild (New Package)
Summary: ng-1.5_beta1.ebuild (New Package)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Mamoru KOMACHI (RETIRED)
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2004-05-11 23:00 UTC by Atsushi Kobayashi
Modified: 2004-05-15 12:01 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Japanised Micro gnu emacs (with direct canna support) (ng-1.5_beta1.ebuild,1.12 KB, text/plain)
2004-05-11 23:03 UTC, Atsushi Kobayashi
Details
fix uses ncurses patch (ng-1.5beta1-ncurses.patch,553 bytes, patch)
2004-05-11 23:04 UTC, Atsushi Kobayashi
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Atsushi Kobayashi 2004-05-11 23:00:35 UTC
Please find attached ng-1.5_beta1.ebuild and a patch.

Ng is the Japanized Micro Gnu emacs. based on Mg (MicroGnuEmacs)
(with direct canna support.)

http://tt.sakura.ne.jp/~amura/ng/

I suggest app-editors/ng


Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Atsushi Kobayashi 2004-05-11 23:03:40 UTC
Created attachment 31236 [details]
Japanised Micro gnu emacs (with direct canna support)
Comment 2 Atsushi Kobayashi 2004-05-11 23:04:56 UTC
Created attachment 31237 [details, diff]
fix uses ncurses patch
Comment 3 Mamoru KOMACHI (RETIRED) gentoo-dev 2004-05-15 12:01:34 UTC
Hi, I commited the ebuild to CVS. Your ebuild looks quite tidy
and plain, but there are several flaws in your ebuild.

* You need to inherit eutils if you use epatch. epatch can be
used without it but you are strongly advised to inherit it.
* LICENSE is not GPL but GNU EMACS GENERAL PUBLIC LICENSE.
I committed it as Emacs. (If you are going to submit another
package that needs some license files other than one from
/usr/portage/licenses, it is a good idea to write so)
* sys-apps/sed is not usually necessary for runtime, so it should
be removed from RDEPEND.
* myconf="`use_enable canna`" won't work. It will be expanded as
--enable-canna and --disable-canna depending on what USE flag
you have, but some configure scripts are broken so that 
--disable-canna means --enable-canna (e.g. im-ja's configure
script is such). I changed that part to

    if use canna; then
        myconf="--enable-canna"
    fi

(no --enable-canna nor --disable-canna means you don't want to
enable canna support). You need to actually check each USE flag
if you add one.
* "into" and "dodir" aren't necessarry. Portage defaults to
"into /usr" (see man 5 ebuild) and insinto automatically creates
target directory.

Anyhow, thanks for submitting the ebuild. Keep in touch with
Bugzilla! ;)