Re: [CEDET-devel] autoconf-new-program bug ?

[Eric M. L. <er...@si...>]

Hi Pierre,

  I think you are right.  I guess I never used it interactively. :)
Please feel free to fix that bug.

Thanks!
Eric

On Fri, 2009-07-24 at 14:55 +0200, Pierre Lorenzon wrote:
> Hi 
> 
> First I must precise that I always use the last cvs version of
> cedet. Actually the one on my system at the moment was
>   downloaded this morning ! This stands also for my previous
> message. 
> 
> Here is the beginning of the code of the `autoconf-new-program'
> function :
> 
> >>> -- Code
> 
> (defun autoconf-new-program (rootdir program testfile)
>   "Initialize a new configure.in in ROOTDIR for PROGRAM using TESTFILE.
> ROOTDIR is the root directory of a given autoconf controlled project.
> PROGRAM is the program to be configured.
> TESTFILE is the file used with AC_INIT.
> configure the initial configure script using `autoconf-new-automake-string'"
>   (interactive "dRoot Dir: \nsProgram: \nsTest File: ")
>   (if (bufferp rootdir)
>       (set-buffer rootdir)
>     (if (not (string-match "\\(/\\|\\\\)$" rootdir))
> 	(setq rootdir (concat rootdir "/")))
> 
> >>> -- End Code
> 
> Is this "dRoot dir ... " correct ? When reading the emacs-lisp
> manual I find : 
> 
> >>> -- Info
> 
> `d'
>      The position of point, as an integer (*note Point::).  No I/O.
> 
> >>> -- End Info
> 
> whereas 			>-- Info
> 
> `D'
>      A directory name.  The default is the current default directory of
>      the current buffer, `default-directory' (*note File Name
>      Expansion::).  Existing, Completion, Default, Prompt.
> 
> 
> >>> -- End Info
> 
> Shouldn't this `d' be a `D' ? When trying to start this command
> interactively, I receive an error saying that the argument is
> not a string ! Indeed it is an integer and completely matching
> the code but not what we want !
> 
> May I fix that ?
> 
> Regards
> 
> Pierre
> 
> 
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> Cedet-devel mailing list
> Ced...@li...
> https://lists.sourceforge.net/lists/listinfo/cedet-devel