Issue654

Title make validation optional
Priority feature Status resolved
Superseder Nosy List jendrik, malte
Assigned To jendrik Keywords
Optional summary

Created on 2016-05-03.22:02:37 by jendrik, last changed by jendrik.

Messages
msg5360 (view) Author: jendrik Date: 2016-05-12.15:13:54
Merged and pushed.

I'm currently preparing lab 2.0 (which removes many deprecated things). In lab 2.0 --validate 
is always prepended to the driver_options. In lab 1.x adding --validate is now recommended in 
the documentation.
msg5359 (view) Author: malte Date: 2016-05-12.12:59:35
No, go ahead! :-)
msg5358 (view) Author: jendrik Date: 2016-05-12.12:42:23
Florian and Silvan see no need for the extra option. Of course, you can always 
use your BDFL veto :-)
msg5357 (view) Author: malte Date: 2016-05-12.12:28:21
Perhaps ask the others what they prefer.
msg5356 (view) Author: jendrik Date: 2016-05-12.12:26:38
I removed that functionality to simplify the code and driver options. Since the 
call to validate is printed, users can always copy the call and add the options 
they need. If you think this is too cumbersome, I can add a --validate-options 
argument.
msg5353 (view) Author: malte Date: 2016-05-12.11:19:40
I see that it's no longer possible to pass options to the validator. Is this
what we want? I often want to pass -v.
msg5345 (view) Author: jendrik Date: 2016-05-12.10:11:30
I made a pull request at
https://bitbucket.org/jendrikseipp/downward/pull-requests/50.

Can you please have a look, Malte?
msg5308 (view) Author: jendrik Date: 2016-05-03.22:02:37
We want to add a "--validate" parameter to the driver script and only invoke VAL 
when the parameter is set. The "--debug" parameter should imply "--validate". lab 
should use the "--validate" parameter by default.
History
Date User Action Args
2016-05-12 15:13:54jendriksetstatus: reviewing -> resolved
messages: + msg5360
2016-05-12 12:59:35maltesetmessages: + msg5359
2016-05-12 12:42:23jendriksetmessages: + msg5358
2016-05-12 12:28:21maltesetmessages: + msg5357
2016-05-12 12:26:38jendriksetmessages: + msg5356
2016-05-12 11:19:40maltesetmessages: + msg5353
2016-05-12 10:11:30jendriksetstatus: unread -> reviewing
messages: + msg5345
2016-05-11 22:03:42jendriksetassignedto: jendrik
2016-05-03 22:02:37jendrikcreate