zlib icon indicating copy to clipboard operation
zlib copied to clipboard

CMake Update

Open Mizux opened this issue 7 years ago • 10 comments

  • Update project() to use VERSION (need CMake >= 3.0)
  • ~~Explicit default option for assembly~~ (removed in v1.2.13)
  • Use of target_include_directories() instead of include_directories()
  • Add ZLIB::ZLIB target alias (to follow FindZLIB.cmake convention)
  • Use GNUInstallDirs Module
  • Add Export Targets, Config and ConfigVersion cmake file
  • Use CTest Module for enable/disable test using BUILD_TESTING (note: include(CTest) call enable_testing())

Mizux avatar Jan 29 '18 12:01 Mizux

I'd really like a lot of these changes, is there any problem with these? Perhaps if they were broken into separate pull requests it would be easier to swallow?

johnb003 avatar Jun 20 '18 01:06 johnb003

As far as I know, I would say @madler doesn't work (i.e. maintain) anymore this repo so it will be never merge.

  1. When you lose interest in a program, your last duty to it is to hand it off to a competent successor. The Cathedral and the Bazaar

ps: https://github.com/madler/zlib/pulls?page=1&q=is%3Apr+is%3Aopen+cmake&utf8=%E2%9C%93 29 Open...

Mizux avatar Jun 20 '18 06:06 Mizux

@Mizux: Can you rebase your PR?

Neustradamus avatar Oct 26 '22 04:10 Neustradamus

@madler: I think it is time to look about this?

Neustradamus avatar Oct 26 '22 04:10 Neustradamus

@Neustradamus on my way to rebase it... (will also fix ortools patch ^v^ )

Mizux avatar Oct 26 '22 06:10 Mizux

@Neustradamus just rebased to master any feedback welcome...

FYI: upstream macos job are broken see: https://github.com/madler/zlib/pull/337/commits/19d7d6b32ee3af4ed4bdec415f42f518c93b5025

Mizux avatar Oct 26 '22 07:10 Mizux

@Mizux: Thanks for your quick rebase... :)

Have you seen this PR for example?

  • https://github.com/madler/zlib/pull/347

There are a lot of:

CMake PRs:

  • https://github.com/madler/zlib/pulls?q=is%3Apr+is%3Aopen+cmake

CMake Issues:

  • https://github.com/madler/zlib/issues?q=is%3Aissue+is%3Aopen+cmake

cc: @puneetmatharu.

Neustradamus avatar Oct 26 '22 11:10 Neustradamus

yes too much, no [cmake] label to flag them.

  • #347 It contains 27 commits and few unrelated stuffs should be split, rebase on master and see what's remain.

Will be better to have a cmake zlib committee, try to submit all PR which already have consensus among cmake users, close duplicates then focus on remaining one to have everything merged or rejected...

Mizux avatar Oct 26 '22 11:10 Mizux

@Mizux: Can you rebase your PR?

Neustradamus avatar Jul 30 '23 14:07 Neustradamus

There are too many cmake issues and pull requests for me to go through and disposition them, and I wouldn't know what's good or bad anyway. I very much like @Mizux 's suggestion to have a group render on, coordinate, and most importantly verify proposed cmake changes. My main concern are proposed changes that work in one case, but messes up a bunch of others. Ideally this group would be able to cover a good set of platforms.

madler avatar Aug 18 '23 02:08 madler