-
Notifications
You must be signed in to change notification settings - Fork 644
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Behaviour when working with packages and setting options. #1448
Comments
I am in agreement that this makes sense, but perhaps the semantics at this time should just be concatenation of arguments? Otherwise, the overriding behavior might be somewhat difficult to precisely specify, given the current underspecified nature of the semantics of combining arguments. For example, |
I wonder if this should only apply to certain arguments (--log is a good example) and any invalid ones should report an error rather than being combined. On 31 Jul 2014, at 09:05, David Christiansen notifications@github.com wrote:
|
Yes and yes. When implementing this feature the behaviour behind the union of option sets should be decided upon first. When presented with two like options the behaviour should be either to:
As to how this behaviour should be assigned to each flag is left for the implemented. |
Going through some old issues and noticed that Issue #499 might actually be a symptom of this. |
Currently when building packages from the command line, all options not included in the
ipkg
file are ignored. For example, given the command below:$> idris --log 3 --build foobar.ipkg
The logging level is not set to 3. This default behaviour is counter intuitive in the cases that someone wishes to quickly add options when developing packages. For example: setting logging levels, setting verbosity output, setting warning flags et cetera.
I think the default behaviour should be that the options used when building a package should be the union of those specified in an
ipkg
file, and those specified on the command line. Furthermore, in the event that two options clash, I think that the options from the command line should have precedence.This should be flagged as low hanging fruit/be a mini project.
The text was updated successfully, but these errors were encountered: