Skip to content

Conversation

@JosiahWI
Copy link
Contributor

Specifying a version range ensures that our build is compatible with NEW policies in future CMake versions, and therefore won't unexpectedly break when we increase our mininum version. It is fine to use a CMake version newer than the upper end of our version range, but we should keep the range updated as newer versions become available. We should use a latest CMake on at least one CI run.

Specifying a version range ensures that our build is compatible with NEW
policies in future CMake versions, and therefore won't unexpectedly
break when we increase our mininum version. It is fine to use a CMake
version newer than the upper end of our version range, but we should
keep the range updated as newer versions become available. We should use
a latest CMake on at least one CI run.
@JosiahWI JosiahWI requested a review from cmcfarlen August 22, 2023 13:52
@JosiahWI JosiahWI self-assigned this Aug 22, 2023
@JosiahWI JosiahWI added the CMake work related to CMakes scripts or issues label Aug 22, 2023
@JosiahWI JosiahWI added this to the 10.0.0 milestone Aug 22, 2023
@JosiahWI JosiahWI requested a review from bneradt August 22, 2023 13:53
Copy link
Contributor

@bneradt bneradt 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.

@bneradt
Copy link
Contributor

bneradt commented Aug 22, 2023

[approve ci]

1 similar comment
@ezelkow1
Copy link
Member

[approve ci]

@bneradt bneradt merged commit ca7b39a into apache:master Aug 28, 2023
@JosiahWI JosiahWI deleted the feat/cmake-version-range branch September 15, 2023 12:45
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
Specifying a version range ensures that our build is compatible with NEW
policies in future CMake versions, and therefore won't unexpectedly
break when we increase our mininum version. It is fine to use a CMake
version newer than the upper end of our version range, but we should
keep the range updated as newer versions become available. We should use
a latest CMake on at least one CI run.
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* commit '01409a61bab49c81cf422cdb29a94dcd07bf3c37': (31 commits)
  cmake: add more executables (apache#10284)
  Fix a crash due to bad disks (apache#10287)
  ts_lua_transform: input_reader may be used uninitialized (apache#10297)
  CID 1508887 fix parent_select plugin stat Time of check. (apache#10260)
  Make remap API compatibility checks consistent. (apache#10253)
  remove deprecated 'NATIVE' AIO mode (apache#10269)
  Fix connect failure count (apache#10294)
  Add version range up to CMake 3.27 (apache#10259)
  untangle fds_limit global (apache#10288)
  replace net_config_poll_timeout with EThread::default_wait_interval_ms (apache#10289)
  suppress coverity false positive on race condition (apache#10278)
  initialize variable (apache#10277)
  Check the inet_pton return value for IP reputations (apache#10267)
  Fix missing virtual destructor. (apache#10293)
  tools/jtest: use @SWOC_LIBS@ (apache#10292)
  Fix m4 quoting on configure error messages. (apache#10258)
  build_h3_tools: explicitly chmod the destination directory (apache#10274)
  Fix CID 1508979: unchecked return value (apache#10281)
  Fix CID 1508970: Uninitialized scalar variable (apache#10282)
  Fix CID 1508992, 1508976, 1508972: uninitialized scalar variables (apache#10275)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake work related to CMakes scripts or issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants