Skip to content
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.

Sign up for GitHub

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

Jump to bottom

modify json-c default build type, and fix up the assert() errors in t… #547

Merged
merged 3 commits into from
Mar 3, 2020
Merged

modify json-c default build type, and fix up the assert() errors in t… #547

merged 3 commits into from
Mar 3, 2020

Conversation

dota17
Copy link
Member

@dota17 dota17 commented Feb 29, 2020 •
edited
Loading

  1. Solve the problemof testcase failure during cmake release-building.
  2. Modify json-c default cmake build type.
  3. Add the default compilation optimization options.
    See details in Solve some problems about cmake build type (debug/release) #548

@dota17 dota17 mentioned this pull request Feb 29, 2020
Closed
hawicz
hawicz reviewed Mar 1, 2020
View reviewed changes
CMakeLists.txt Outdated Show resolved Hide resolved
hawicz
hawicz reviewed Mar 1, 2020
View reviewed changes
tests/test_deep_copy.c Outdated Show resolved Hide resolved
hawicz
hawicz reviewed Mar 1, 2020
View reviewed changes
CMakeLists.txt Outdated Show resolved Hide resolved
ploxiln
ploxiln reviewed Mar 3, 2020
View reviewed changes
CMakeLists.txt Outdated Show resolved Hide resolved
@hawicz hawicz merged commit a8c9284 into json-c:master Mar 3, 2020
@dota17 dota17 deleted the assert_test branch April 17, 2020 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ploxiln ploxiln ploxiln left review comments

@hawicz hawicz hawicz left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dota17 @ploxiln @hawicz