-
Notifications
You must be signed in to change notification settings - Fork 664
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
scaleflux: enable sfx-nvme.c build without json-c dependencies checking #2639
Open
ikegami-t
wants to merge
11
commits into
linux-nvme:master
Choose a base branch
from
ikegami-t:sfx-json-dep
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ikegami-t
force-pushed
the
sfx-json-dep
branch
4 times, most recently
from
December 21, 2024 12:14
8e00484
to
1bfbc0b
Compare
To fix the cross compiler build fails and some indentation errors also. Signed-off-by: Tokunori Ikegami <[email protected]>
Only build json print codes with CONFIG_JSONC build option instead. Signed-off-by: Tokunori Ikegami <[email protected]>
…king Only build json print codes with CONFIG_JSONC build option instead. Signed-off-by: Tokunori Ikegami <[email protected]>
ssstc plugin does not use json so no need to check the build dependency. Signed-off-by: Tokunori Ikegami <[email protected]>
Only build json print codes with CONFIG_JSONC build option instead. Signed-off-by: Tokunori Ikegami <[email protected]>
This is to prevent the build warning without json. Signed-off-by: Tokunori Ikegami <[email protected]>
ikegami-t
force-pushed
the
sfx-json-dep
branch
from
December 21, 2024 12:18
1bfbc0b
to
213bdcd
Compare
Only remaining a plugin for solidigm to enable on muon build without json but it seems as difficult to do this since basically the plugin based on to use json for example to parse json configuration file. So at this moment all plugins excluding |
Use empty json functions instead without json. Signed-off-by: Tokunori Ikegami <[email protected]>
Use empty json functions instead without json. Signed-off-by: Tokunori Ikegami <[email protected]>
ikegami-t
force-pushed
the
sfx-json-dep
branch
from
December 21, 2024 13:15
f1bf376
to
0250ca0
Compare
Use empty json functions instead without json. Signed-off-by: Tokunori Ikegami <[email protected]>
Use empty json functions instead without json. Signed-off-by: Tokunori Ikegami <[email protected]>
Since some json macros can be used without json as common. Signed-off-by: Tokunori Ikegami <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Only build json print codes with CONFIG_JSONC build option instead.