You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think the '--accept-eula' configuration option is misleading.
There is already an easyconfig parameter of a very similar name, 'accept_eula', and of different data types, string vs bool.
Using the name '--accept-eula-for' instead would be more self-explanatory. The use case would be: '--accept-eula-for=AOCC,Intel-OneAPI'. I noticed that Kenneth already used such a name in PR #3535 but renamed it to --accept-eula.
To me, the configuration option naming, '--accept-eula', implies boolean type. The first time I saw the option, I thought I need to specify True to accept it. I did it and noticed that it didn't work. Checking Github and the documentation, I figured out that I need to specify the package for which I accept EULA.
On top of that, one should specify for which packages he accepts EULA. In most cases, one doesn't care and thus wants to specify that he accepts them all but in the current state cannot. Therefore, I propose to add a wildcard '*' which means that he accepts EULA for all packages he installs.
Best regards,
Robert
The text was updated successfully, but these errors were encountered:
Hi guys,
I think the '--accept-eula' configuration option is misleading.
There is already an easyconfig parameter of a very similar name, 'accept_eula', and of different data types, string vs bool.
Using the name '--accept-eula-for' instead would be more self-explanatory. The use case would be: '--accept-eula-for=AOCC,Intel-OneAPI'. I noticed that Kenneth already used such a name in PR #3535 but renamed it to --accept-eula.
To me, the configuration option naming, '--accept-eula', implies boolean type. The first time I saw the option, I thought I need to specify True to accept it. I did it and noticed that it didn't work. Checking Github and the documentation, I figured out that I need to specify the package for which I accept EULA.
On top of that, one should specify for which packages he accepts EULA. In most cases, one doesn't care and thus wants to specify that he accepts them all but in the current state cannot. Therefore, I propose to add a wildcard '*' which means that he accepts EULA for all packages he installs.
Best regards,
Robert
The text was updated successfully, but these errors were encountered: