Re: [CEDET-devel] bug in ede-set-project-variables

[Eric M. L. <eri...@gm...>]

On 03/06/2012 09:17 AM, Widger, Niels wrote:
> I've been trying to use the :local-variables slot in ede-project to
> automatically set compile-command based on the current project.  This
> slot is defined to be a list such as
>
> ((VARIABLE VALUE) (VARIABLE VALUE)...)
>
> however the current definition of ede-set-project-variables in the BZR
> repository winds up setting each VARIABLE to (VALUE) instead of VALUE:
>
> http://cedet.bzr.sourceforge.net/bzr/cedet/code/trunk/annotate/head%3A/ede/ede.el#L1422
>
> I believe that line 1422 should be changed from
>
>         (set (car v) (cdr v)))))
>
> to
>
>         (set (car v) (car (cdr v))))))

Thanks for discovering that!  It also points out another area I have no 
tests, so I will see if I can find a nice way to add tests along with 
your patch.

Thanks!
Eric