-
Notifications
You must be signed in to change notification settings - Fork 12
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
Unify the configurarion-name
and overriden-properties
into one configurations.json
file.
#52
Comments
denis-shienkov
added a commit
that referenced
this issue
Jul 30, 2021
We need to use the `configurations.json` file which describes the all build configurations with its overridden properties. In this case we don't need in the `overridden-properties.json` file, and in the `qbs.overriddenProperties` command. Instead, we use the `qbs.editConfiguration` command, and the `configurations.json` file which creates automatically at the first extension startup. Also the user can modify this file as needs to add an other custom build configurations with its overridden properties. Task-number: #52
Right now it is implemented with the following [
{
"name": "release",
"display-name": "Release",
"description": "Enable optimizations.",
"overridden-properties": {}
},
{
"name": "debug",
"display-name": "Debug",
"description": "Disable optimizations.",
"overridden-properties": {
"projects.foo.hasSpecialFeature": false
}
},
{
"name": "profiling",
"display-name": "Profiling",
"description": "Enable profiling.",
"overridden-properties": {}
}
] where the :
|
denis-shienkov
added a commit
that referenced
this issue
Aug 1, 2021
UPD: File |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now we use the separate
overriden-properties.json
file to store the overriden properties, and the separate buildconfigurations
button on the status bar.In this case it is impossible to assign the overriden-properties for the specific build configuration.
The solution is to get rid of the
overriden-properties.json
file in vafor to theconfigurations.json
file. In this case thisconfigurations.json
file may contains all build configurations and its overriden properties e.g. in a such format:where the
release
,debug
,profiling
(or something custom) - are the build configurations maps, and the underlying dict is the map containing overriden-properties for each configuration (it iskey -> value
pairs).Also we need to do the additional changes:
configurations.json
file if it not exists yet, and then to open this file in the editor.configurations.json
file to select the desired configuration.configuration.json
file.The text was updated successfully, but these errors were encountered: