This project is read-only.
2

Closed

Produce an error if both /settings and /spreadsheet parameters are passed to the command line

description

Hi.
 
I was just testing this and noticed that xmlpreprocess does not throw an exception if both /settings (/s) and /spreadsheet (/x) parameters are passed to the command line.
 
It seems that xmlpreprocess will ignore the /settings file and just use the /spreadsheet file.
 
I would have expected an error be produced and execution stopped.
 
Thank you.
Closed Jun 20, 2012 at 10:01 PM by lorenh
I think I will close this issue since it's been addressed in 2.0.14 as a useful combination of arguments.

comments

lorenh wrote Jun 20, 2012 at 9:57 PM

I don't think I agree that this is a defect, I think this is a valid scenario. In some cases I have wanted to use the values from a spreadsheet, but wanted to overlay some values declared in a settings file. In 2.0.14 this was enabled. The desire for this arose when there was a global spreadsheet that was used for all source branches, and it contained a "local" column with reasonable defaults for all developers, except for a few that needed to change for each branch. Each source branch was able to manage their own override settings xml file that contained just values that needed to be different for them. The two settings were merged by xmlpreprocessor by passing both /x and /s (with /s taking precedence).

wrote Jun 20, 2012 at 10:01 PM

wrote Feb 14, 2013 at 7:04 PM

wrote May 16, 2013 at 10:03 AM