Control: clone 909561 -1
Control: retitle -1 simple-cdd: support variables in configuration files
Control: tags 909561 pending

On 2019-05-29, Vagrant Cascadian wrote:
> On 2018-09-25, Raphaƫl Hertzog wrote:
>> I'm putting this as important because it's a regression compared to
>> before the python rewrite and because the sample configuration files shipped
>> in the package make you believe that you can rely on this variable.
>>
>> I'm referring to those lines for example in
>> /usr/share/doc/simple-cdd/examples/simple-cdd.conf.detailed.gz:
>>
>>     #debian_cd_dir=$simple_cdd_dir/debian-cd
>>     #simple_cdd_temp="$simple_cdd_dir/tmp/"
>>     # export SPLASHPNG="$simple_cdd_dir/yourimage.png"
>>     #hd_img="$simple_cdd_dir/qemu-test.hd.img"
>>
>> I just checked by putting "env >/tmp/foo" in the configuration file that I
>> passed to the --conf parameter and I confirm that "simple_cdd_dir" is not
>> set in the environment.
>
>> You can decide to set it or not, I don't really care but it should be either
>> documented and supported, or unsupported and with no misleading references
>> to it in the configuration file.

I've added a commit upstream which removes references to variables in
the example configuration files and profiles, since the fix may take
longer.


> I think this extends to setting any variables within variables any of
> simple-cdd's configuration files.
>
> I definitely consider it a bug.

Cloning this bug to track fixing the actual issue.


live well,
  vagrant

Attachment: signature.asc
Description: PGP signature

Reply via email to