Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 230229 - app-text/noweb: optionally install emacs noweb-mode
Summary: app-text/noweb: optionally install emacs noweb-mode
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: TeX project
URL:
Whiteboard:
Keywords: REVIEWED
Depends on:
Blocks: 184166
  Show dependency tree
 
Reported: 2008-06-30 18:14 UTC by Denys Duchier
Modified: 2008-07-04 17:20 UTC (History)
1 user (show)

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


Attachments
ebuild (noweb-2.11b-r1.ebuild,2.32 KB, text/plain)
2008-06-30 18:16 UTC, Denys Duchier
Details
files/51noweb-gentoo.el (51noweb-gentoo.el,207 bytes, text/plain)
2008-06-30 18:17 UTC, Denys Duchier
Details
ebuild (noweb-2.11b-r1.ebuild,2.32 KB, text/plain)
2008-07-01 16:22 UTC, Denys Duchier
Details
files/51noweb-gentoo.el (51noweb-gentoo.el,212 bytes, text/plain)
2008-07-01 16:23 UTC, Denys Duchier
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Denys Duchier 2008-06-30 18:14:41 UTC
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 Denys Duchier 2008-06-30 18:16:12 UTC
Created attachment 158987 [details]
ebuild
Comment 2 Denys Duchier 2008-06-30 18:17:00 UTC
Created attachment 158989 [details]
files/51noweb-gentoo.el
Comment 3 Ulrich Müller gentoo-dev 2008-07-01 08:43:22 UTC
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 Denys Duchier 2008-07-01 16:22:07 UTC
Created attachment 159133 [details]
ebuild

ebuild with GPL-2 added to LICENSE as suggested
Comment 5 Denys Duchier 2008-07-01 16:23:25 UTC
Created attachment 159135 [details]
files/51noweb-gentoo.el

documentation string for autoload line modified as suggested
Comment 6 Alexis Ballier gentoo-dev 2008-07-03 22:53:28 UTC
(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 Ulrich Müller gentoo-dev 2008-07-04 03:45:08 UTC
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 Ulrich Müller gentoo-dev 2008-07-04 03:49:15 UTC
  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.