Won't compile on OpenMandriva ROME due to zlib conflict

Open Beginner friendly
#829 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
65/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Tech stack
cpp
Domain
build-system

Research direction

Start with file.cpp and inspect the zlib include near the top, comparing it with the include shown in the issue. Move the non-Windows zlib include as described, then build FCEUX on OpenMandriva ROME and confirm the zlib conflict no longer prevents compilation.

Written by the indexing model from the issue text.

Description

Describe the bug
When building file.cpp on OpenMandriva you get the error:
/usr/include/zlib.h:34:4: error: Include zlib-ng.h for zlib-ng API or zlib.h for zlib-compat API
The fix is to move

#ifndef WIN32
#include <zlib.h>
#endif

to the top of file.cpp

  • OS and Version: OpenMandriva ROME
Dominant language
C++
Stars
1.6k
Forks
314
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from TASEmulators/fceux

All issues in TASEmulators/fceux

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.