[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [RFR] templates://gcl/{in.gcl.templates}



Christian Perrier wrote:
> Your review should be sent as an answer to this mail.

> + Please see the README.Debian file for a brief description of these terms.
> + Choosing this option will determine which image will be used by default
> + when executing 'gcl@EXT@'.

(What values can 'gcl@EXT@' expand to?  I only see /usr/bin/gcl in
the file lists.) 

>  Template: gcl@EXT@/default_gcl_prof
[...]
> + Profiling support may be enforced by filling the
> + GCL_PROF environment variable with any non-empty string for profiling
> + support. More optimized builds are possible by setting an empty string for the more optimized builds, e.g.
>   GCL_PROF=t gcl@EXT@.  If profiling is enabled, this will be reported in
>   the initial startup banner.

"Filling" is bad, "profiling support" and "optimized builds" are
each repeated, and "are possible by setting" doesn't work.  This
order also implies that the example with $GCL_PROF set goes with the
optimised builds.  And as someone on this list pointed out, there's
nothing indefinite about the empty string.

    Set the GCL_PROF environment variable to the empty string for more
    optimized builds, or any non-empty string for profiling support; e.g.
    GCL_PROF=t gcl@EXT@. If profiling is enabled, this will be reported in
    the initial startup banner.

>   GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
>   implemented in C, and complying mostly with the standard set 
> + forth in the book "Common Lisp, the Language I". It attempts
>   to strike a useful middle ground in performance and portability
>   from its design around C.

That's "first edition", not "volume one".  The reference to
"striking" a middle ground "from" a design seems a little
confused...

    GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
    implemented in C, and complying mostly with the standard set 
    forth in the book "Common Lisp, the Language (1st edition)". It
    attempts to take a useful middle ground in performance and portability
    by way of its design around C.

That still seems vague (only "attempts"?), but it'll do.
-- 
JBR	with qualifications in linguistics, experience as a Debian
	sysadmin, and probably no clue about this particular package
--- ../gcl.old/debian/in.gcl.templates	2007-11-25 13:46:32.000000000 +0000
+++ debian/in.gcl.templates	2007-12-05 12:27:39.000000000 +0000
@@ -1,26 +1,29 @@
 Template: gcl@EXT@/default_gcl_ansi
 Type: boolean
-_Description: Use the work-in-progress ansi build by default?
+_Description: Use the work-in-progress ANSI build by default?
  GCL is in the process of providing an ANSI compliant image in addition to
- its traditional CLtL1 image still in production use.  Please see the
- README.Debian file for a brief description of these terms.  Setting this
- variable will determine which image you will use by default on executing
- 'gcl@EXT@'.  You can locally override this choice by setting the GCL_ANSI
+ its traditional CLtL1 image still in production use.
+ .
+ Please see the README.Debian file for a brief description of these terms.
+ Choosing this option will determine which image will be used by default
+ when executing 'gcl@EXT@'.
+ .
+ This setting may be overridden by setting the GCL_ANSI
  environment variable to any non-empty string for the ANSI build, and to
- the empty string for the CLtL1 build, e.g. GCL_ANSI=t gcl@EXT@.  The
- flavor of the build in force will be reported in the initial startup
+ the empty string for the CLtL1 build, e.g. GCL_ANSI=t gcl@EXT@. The
+ currently enforced build flavor will be reported in the initial startup
  banner.
 
 Template: gcl@EXT@/default_gcl_prof
 Type: boolean
 _Description: Use the profiling build by default?
- GCL now has optional support for profiling via gprof.  Please see the
- documentation
- for si::gprof-start and si::gprof-quit for details. As this build is slower
- than builds without gprof support, it is not recommended for final production use.
- You can locally override the default choice made here
- by setting the
- GCL_PROF environment variable to any non-empty string for profiling
- support, and to the empty string for the more optimized builds, e.g.
- GCL_PROF=t gcl@EXT@.  If profiling is enabled, this will be reported in
+ GCL has optional support for profiling via gprof.
+ .
+ Please see the documentation for si::gprof-start and si::gprof-quit
+ for details. As this build is slower than builds without gprof
+ support, it is not recommended for final production use.
+ .
+ Set the GCL_PROF environment variable to the empty string for more
+ optimized builds, or any non-empty string for profiling support; e.g.
+ GCL_PROF=t gcl@EXT@. If profiling is enabled, this will be reported in
  the initial startup banner.
--- ../gcl.old/debian/control	2007-11-25 13:46:32.000000000 +0000
+++ debian/control	2007-12-05 12:23:00.000000000 +0000
@@ -9,14 +9,14 @@
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends}, ${gcc}, debconf (>= 1.2.0), emacs22 | emacsen
 Suggests: gcl-doc
-Description: GNU Common Lisp compiler
+Description: GNU Common Lisp compiler - binaries
  GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
  implemented in C, and complying mostly with the standard set 
- forth in the book "Common Lisp, the Language I".  It attempts
- to strike a useful middle ground in performance and portability
- from its design around C.
+ forth in the book "Common Lisp, the Language (1st edition)". It
+ attempts to take a useful middle ground in performance and portability
+ by way of its design around C.
  .
- This package contains the Lisp system itself.  Documentation 
+ This package contains the Lisp system itself. The documentation 
  is provided in the gcl-doc package.
 
 Package: gcl-doc
@@ -24,13 +24,13 @@
 Architecture: all
 Conflicts: gclinfo
 Replaces: gclinfo
-Description: Documentation for GNU Common Lisp
+Description: GNU Common Lisp compiler - documentation
  GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
  implemented in C, and complying mostly with the standard set 
- forth in the book "Common Lisp, the Language I".  It attempts
- to strike a useful middle ground in performance and portability
- from its design around C.
+ forth in the book "Common Lisp, the Language (1st edition)". It
+ attempts to take a useful middle ground in performance and portability
+ by way of its design around C.
  .
- This package contains Documentation in info format of both the
- system internals, as well as the graphical interface currently
+ This package contains the GCL documentation in info format for the
+ system internals as well as the graphical interface currently
  implemented in Tcl/Tk.
Template: gcl@EXT@/default_gcl_ansi
Type: boolean
_Description: Use the work-in-progress ANSI build by default?
 GCL is in the process of providing an ANSI compliant image in addition to
 its traditional CLtL1 image still in production use.
 .
 Please see the README.Debian file for a brief description of these terms.
 Choosing this option will determine which image will be used by default
 when executing 'gcl@EXT@'.
 .
 This setting may be overridden by setting the GCL_ANSI
 environment variable to any non-empty string for the ANSI build, and to
 the empty string for the CLtL1 build, e.g. GCL_ANSI=t gcl@EXT@. The
 currently enforced build flavor will be reported in the initial startup
 banner.

Template: gcl@EXT@/default_gcl_prof
Type: boolean
_Description: Use the profiling build by default?
 GCL has optional support for profiling via gprof.
 .
 Please see the documentation for si::gprof-start and si::gprof-quit
 for details. As this build is slower than builds without gprof
 support, it is not recommended for final production use.
 .
 Set the GCL_PROF environment variable to the empty string for more
 optimized builds, or any non-empty string for profiling support; e.g.
 GCL_PROF=t gcl@EXT@. If profiling is enabled, this will be reported in
 the initial startup banner.
Source: gcl
Section: interpreters
Priority: optional
Maintainer: Camm Maguire <camm@enhanced.com>
Build-Depends: debhelper (>= 4.1.16), libreadline5-dev | libreadline-dev, m4, tk8.4-dev, texlive-latex-base, texinfo, libgmp3-dev, autotools-dev, libxmu-dev, libxaw7-dev, po-debconf
Standards-Version: 3.7.2

Package: gcl
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}, ${gcc}, debconf (>= 1.2.0), emacs22 | emacsen
Suggests: gcl-doc
Description: GNU Common Lisp compiler - binaries
 GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
 implemented in C, and complying mostly with the standard set 
 forth in the book "Common Lisp, the Language (1st edition)". It
 attempts to take a useful middle ground in performance and portability
 by way of its design around C.
 .
 This package contains the Lisp system itself. The documentation 
 is provided in the gcl-doc package.

Package: gcl-doc
Section: doc
Architecture: all
Conflicts: gclinfo
Replaces: gclinfo
Description: GNU Common Lisp compiler - documentation
 GNU Common Lisp (GCL) is a Common Lisp compiler and interpreter
 implemented in C, and complying mostly with the standard set 
 forth in the book "Common Lisp, the Language (1st edition)". It
 attempts to take a useful middle ground in performance and portability
 by way of its design around C.
 .
 This package contains the GCL documentation in info format for the
 system internals as well as the graphical interface currently
 implemented in Tcl/Tk.

Reply to: