Subject: bug#37527: [PATCH] Install C source code for for debugging help

From: Eli Zaretskii <[email-masked]>
Date: Tue, 21 Jan 2020 19:02:56 +0200

> Cc: Michael Albinus <[email-masked]>, [email-masked],
>  [email-masked]
> From: Paul Eggert <[email-masked]>
> Date: Tue, 21 Jan 2020 01:13:53 -0800
> 
> --- a/src/emacs.c
> +++ b/src/emacs.c
> @@ -522,6 +522,7 @@ init_cmdargs (int argc, char **argv, int skip_args, char const *original_pwd)
>  	      if (!NILP (etc_exists))
>  		{
>                    Vinstallation_directory = Ffile_name_as_directory (dir);
> +		  Vsource_directory = Vinstallation_directory;
>  		  break;
>  		}
>  	    }
> @@ -547,6 +548,7 @@ init_cmdargs (int argc, char **argv, int skip_args, char const *original_pwd)
>  		{
>  		  tem = Fexpand_file_name (build_string (".."), dir);
>                    Vinstallation_directory = Ffile_name_as_directory (tem);
> +		  Vsource_directory = Vinstallation_directory;
>  		  break;
>  		}

Bother.  Should we really step on source-directory like that?  What if
the original source tree still exists?  Maybe it would be safer to
introduce a new variable, say, install-source-directory?