[Simple-cdd-devel] simple-cdd bug...

[Vagrant Cascadian]

On Thu, Dec 06, 2012 at 05:28:42PM -0300, Gustavo Ortiz Uriburu wrote:
> I've found a bug in build-simple-cdd script:

Thanks for the patches!

In the future, please submit a separate patch for each issue, and preferrably 
using the debian bug tracking system:

  http://www.debian.org/Bugs/Reporting


Unfortunately, I'm not sure these issues are severe enough to make it into
wheezy. I've committed at least two parts of your submitted patch to bzr.


> the KERNEL_PARAMS export option was malformed,

I disagree with that change, "export KERNEL_PARAMS" does not require also 
passing the value.


> the keyboard option passed to the kernel wasn't working.

Committed:

http://anonscm.debian.org/loggerhead/simple-cdd/simple-cdd-devel/revision/524

 
> I've added an option to generate a DVD.

Committed:

http://anonscm.debian.org/loggerhead/simple-cdd/simple-cdd-devel/revision/523


The patch also included an extra "sudo" call, to presumably to quiet 
"debconf-set-selections --checkonly" from issuing warnings about the 
inaccessible passwords database. This should be fixed in 
debconf-set-selections:

  http://bugs.debian.org/587380

There is no real reason to run this as root, and I don't want to assume sudo is
available for the user, it's just an ugly warning message that can be ignored. 

I've attempted to work around that bug and suppress the warning messages, but 
due to debconf's unusual use of file descriptors, this is tricky, and I've not 
been successfull.


live well,
  vagrant