Bug 230229 - app-text/noweb: optionally install emacs noweb-mode
Bug#: 230229 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: tex@gentoo.org Reported By: denys.duchier@univ-orleans.fr
Component: Ebuilds
URL: 
Summary: app-text/noweb: optionally install emacs noweb-mode
Keywords:  REVIEWED
Status Whiteboard: 
Opened: 2008-06-30 18:14 0000
Description:   Opened: 2008-06-30 18:14 0000
the app-text/noweb package provides noweb-mode.el: it should be appropriately
installed when the "emacs" use flag is set.  This proposed enhancement provides
this improvement: it was patterned after app-text/txt2tags.

Reproducible: Always

------- Comment #1 From Denys Duchier 2008-06-30 18:16:12 0000 -------
Created an attachment (id=158987) [details]
ebuild

------- Comment #2 From Denys Duchier 2008-06-30 18:17:00 0000 -------
Created an attachment (id=158989) [details]
files/51noweb-gentoo.el

------- Comment #3 From Ulrich Müller 2008-07-01 08:43:22 0000 -------
Ebuild looks good to me. Maybe add GPL-2 to LICENSE, since it's in the header
of noweb-mode.el?

Emacs site-init file: Since noweb-mode is a minor mode, the "autoload" line
should read:
(autoload  'noweb-mode "noweb-mode" "Minor meta mode for editing noweb files."
t)

------- Comment #4 From Denys Duchier 2008-07-01 16:22:07 0000 -------
Created an attachment (id=159133) [details]
ebuild

ebuild with GPL-2 added to LICENSE as suggested

------- Comment #5 From Denys Duchier 2008-07-01 16:23:25 0000 -------
Created an attachment (id=159135) [details]
files/51noweb-gentoo.el

documentation string for autoload line modified as suggested

------- Comment #6 From Alexis Ballier 2008-07-03 22:53:28 0000 -------
(In reply to comment #3)
> Ebuild looks good to me. Maybe add GPL-2 to LICENSE, since it's in the header
> of noweb-mode.el?
> 
> Emacs site-init file: Since noweb-mode is a minor mode, the "autoload" line
> should read:
> (autoload  'noweb-mode "noweb-mode" "Minor meta mode for editing noweb files."
> t)


please feel free to apply if you're happy with it and it just adds emacs
support ;)
I don't use emacs myself and have no clue how these modes work so I'd be rather
useless there :/

------- Comment #7 From Ulrich Müller 2008-07-04 03:45:08 0000 -------
I'll also add some quotes around $(tc-getCC) because I get the following build
failure:

>>> Compiling source in /var/tmp/portage/app-text/noweb-2.11b/work/noweb-2.11b/src ...
make: invalid option -- 'O'
Usage: make [options] [target] ...
Options:
  -b, -m                      Ignored for compatibility.
  -B, --always-make           Unconditionally make all targets.
  -C DIRECTORY, --directory=DIRECTORY
                              Change to DIRECTORY before doing anything.
  -d                          Print lots of debugging information.
  --debug[=FLAGS]             Print various types of debugging information.
  -e, --environment-overrides
                              Environment variables override makefiles.
  -f FILE, --file=FILE, --makefile=FILE
                              Read FILE as a makefile.
  -h, --help                  Print this message and exit.
  -i, --ignore-errors         Ignore errors from commands.
  -I DIRECTORY, --include-dir=DIRECTORY
                              Search DIRECTORY for included makefiles.
  -j [N], --jobs[=N]          Allow N jobs at once; infinite jobs with no arg.
  -k, --keep-going            Keep going when some targets can't be made.
  -l [N], --load-average[=N], --max-load[=N]
                              Don't start multiple jobs unless load is below N.
  -L, --check-symlink-times   Use the latest mtime between symlinks and target.
  -n, --just-print, --dry-run, --recon
                              Don't actually run any commands; just print them.
  -o FILE, --old-file=FILE, --assume-old=FILE
                              Consider FILE to be very old and don't remake it.
  -p, --print-data-base       Print make's internal database.
  -q, --question              Run no commands; exit status says if up to date.
  -r, --no-builtin-rules      Disable the built-in implicit rules.
  -R, --no-builtin-variables  Disable the built-in variable settings.
  -s, --silent, --quiet       Don't echo commands.
  -S, --no-keep-going, --stop
                              Turns off -k.
  -t, --touch                 Touch targets instead of remaking them.
  -v, --version               Print the version number of make and exit.
  -w, --print-directory       Print the current directory.
  --no-print-directory        Turn off -w, even if it was turned on implicitly.
  -W FILE, --what-if=FILE, --new-file=FILE, --assume-new=FILE
                              Consider FILE to be infinitely new.
  --warn-undefined-variables  Warn when an undefined variable is referenced.

This program built for i686-pc-linux-gnu
Report bugs to <bug-make@gnu.org>
 * 
 * ERROR: app-text/noweb-2.11b failed.
 * Call stack:
 *               ebuild.sh, line   49:  Called src_compile
 *             environment, line 2293:  Called die
 * The specific snippet of code:
 *       emake ICONC="icont" CC=$(tc-getCC) CFLAGS="${CFLAGS}" LIBSRC="icon" )
|| die;
 *  The die message:
 *   (no error message)

------- Comment #8 From Ulrich Müller 2008-07-04 03:49:15 0000 -------
  04 Jul 2008; Ulrich Mueller <ulm@gentoo.org> +files/50noweb-gentoo.el,
  noweb-2.11b.ebuild:
  Optionally install Emacs noweb-mode. Ebuild and site-init file provided by
  Denys Duchier <denys.duchier@univ-orleans.fr>, bug 230229.