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

Build failures on Fedora 36 with KeepassXC develop branch #7633

Closed
apollo13 opened this issue Mar 24, 2022 · 7 comments
Closed

Build failures on Fedora 36 with KeepassXC develop branch #7633

apollo13 opened this issue Mar 24, 2022 · 7 comments
Labels

Comments

@apollo13
Copy link

Overview

After installing minizip-devel on Fedora 36 (minizip-3.0.2-6.fc36.src.rpm from upstream https://github.com/nmoinvaz/minizip) I get the following build failures

Steps to Reproduce

  1. compile on fedora 36 :)

Expected Behavior

Compile fine…

Actual Behavior

[  9%] Building CXX object src/keeshare/CMakeFiles/keeshare.dir/ShareExport.cpp.o
/home/florian/sources/keepassxc/src/keeshare/ShareExport.cpp: In function ‘bool {anonymous}::writeZipFile(void*, const QString&, const QByteArray&)’:
/home/florian/sources/keepassxc/src/keeshare/ShareExport.cpp:111:9: error: ‘zipOpenNewFileInZip64’ was not declared in this scope; did you mean ‘zipOpenNewFileInZip_64’?
  111 |         zipOpenNewFileInZip64(zf,
      |         ^~~~~~~~~~~~~~~~~~~~~
      |         zipOpenNewFileInZip_64

also Z_BEST_COMPRESSION is undefined. Where from should I obtain minizip?

@apollo13 apollo13 added the bug label Mar 24, 2022
@apollo13
Copy link
Author

Apparently the build only works against https://github.com/madler/zlib/tree/master/contrib/minizip and not the "fork" mentioned on https://www.winimage.com/zLibDll/minizip.html

@apollo13
Copy link
Author

I can build on fedora using minizip-compat-devel -- I'll leave this open for the maintainers to decide which library is the wanted one. Sorry for the noise.

@apollo13
Copy link
Author

/cc @Germano0 so he doesn't have to repeat this work when fixing the spec file for fedora :)

@droidmonkey
Copy link
Member

droidmonkey commented Mar 24, 2022

The compat one is desired, will update the wiki. Minizip-ng is another ballgame. Really Fedora packaging dropped the ball here by not adding the -ng

@Germano0
Copy link
Contributor

Thank you, now I am able to build KXC

@droidmonkey
Copy link
Member

I updated the wiki

@Adadov
Copy link

Adadov commented Mar 25, 2022

I just made patches for spec file and ShareExport allowing 2.7.0 release to build on mock for Fedora 35 & 36.
You can find them here: https://gist.github.com/Adadov/5222947aa68cc94c4a7cfc86e47dcec6
Runs on Fedora 35, wasn't able to test run on Fedora 36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants