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

Fix compilation of ModelicaMatIO.c if HAVE_ZLIB=0 is defined #3128

Merged
merged 1 commit into from Oct 10, 2019
Merged

Fix compilation of ModelicaMatIO.c if HAVE_ZLIB=0 is defined #3128

merged 1 commit into from Oct 10, 2019

Conversation

beutlich
Copy link
Member

@beutlich beutlich commented Oct 2, 2019

Backport of #3085 for maint/3.2.3.

@beutlich beutlich added bug Critical/severe issue L: C-Sources Issue addresses Modelica/Resources/C-Sources labels Oct 2, 2019
@beutlich beutlich added this to the MSL3.2.3+maint milestone Oct 2, 2019
@beutlich beutlich self-assigned this Oct 2, 2019
HansOlsson
HansOlsson approved these changes Oct 7, 2019
View reviewed changes
Copy link
Contributor

@HansOlsson HansOlsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, and the normal style of #if

@dietmarw dietmarw removed their request for review October 7, 2019 10:13
@beutlich beutlich requested review from sjoelund and removed request for sjoelund October 9, 2019 14:44
@beutlich beutlich merged commit 889163b into modelica:maint/3.2.3 Oct 10, 2019
@beutlich beutlich deleted the fix-compilation branch October 10, 2019 06:39
@beutlich beutlich changed the title maint/3.2.3: Fix compilation of ModelicaMatIO.c if HAVE_ZLIB=0 is defined Fix compilation of ModelicaMatIO.c if HAVE_ZLIB=0 is defined Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjoelund sjoelund sjoelund approved these changes

@HansOlsson HansOlsson HansOlsson approved these changes

Assignees

@beutlich beutlich

Labels
bug Critical/severe issue L: C-Sources Issue addresses Modelica/Resources/C-Sources
Projects
None yet
Milestone
MSL3.2.3+maint
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@beutlich @sjoelund @HansOlsson