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.

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

A fix for v1.6 under Borland C++ #1

Closed
wants to merge 1 commit into from

Conversation

ctruta
Copy link
Member

@ctruta ctruta commented Nov 14, 2014

The macros passed in the command line to Borland make are ignored if
similarly-named macros are already defined in makefiles. This behavior
is different from POSIX make and other make programs.

Surround the macro definitions with ifndef guards.

The macros passed in the command line to Borland make are ignored if
similarly-named macros are already defined in makefiles. This behavior
is different from POSIX make and other make programs.

Surround the macro definitions with ifndef guards.
@ctruta ctruta changed the title Work around the non-standard behavior of Borland make. A fix for v1.6 under Borland C++ Nov 14, 2014
@glennrp
Copy link
Contributor

glennrp commented Nov 16, 2014

The libpng16 and libpng17 branches have been updated with this change.

@glennrp glennrp closed this Nov 16, 2014
glennrp pushed a commit that referenced this pull request Aug 30, 2016
@ctruta ctruta deleted the libpng16-cos branch June 18, 2018 00:40
GerHobbelt pushed a commit to GerHobbelt/libpng that referenced this pull request Mar 20, 2021
mathieumallet referenced this pull request in YOU-i-Labs/libpng Aug 10, 2021
link ZLIB::zlib and (optional) libm privately and use check_library_exists()
BlazesRus pushed a commit to BlazesRus/libpng that referenced this pull request May 6, 2022
GerHobbelt pushed a commit to GerHobbelt/libpng that referenced this pull request Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants