Skip to content

cmake: disable debian info generation on Windows#2525

Merged
BareosBot merged 4 commits intobareos:masterfrom
arogge:fix-win-tests
Feb 2, 2026
Merged

cmake: disable debian info generation on Windows#2525
BareosBot merged 4 commits intobareos:masterfrom
arogge:fix-win-tests

Conversation

@arogge
Copy link
Member

@arogge arogge commented Jan 30, 2026

This disables generating the debian information on Windows builds.
Fixes a build-issue where CMake won't handle symlinks on a cygwin-based Git checkout. As we never need Debian build scripts on Windows, we can just skip it.

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

General
  • Is the PR title usable as CHANGELOG entry?
  • Purpose of the PR is understood
  • Commit descriptions are understandable and well formatted
  • Required backport PRs have been created
  • Correct milestone is set
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR

@arogge arogge requested a review from pstorz January 30, 2026 12:29
@arogge arogge self-assigned this Jan 30, 2026
@pstorz pstorz added this to the 26.0.0 milestone Jan 30, 2026
Copy link
Member

@pstorz pstorz 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!

@arogge arogge force-pushed the fix-win-tests branch 2 times, most recently from ce09b55 to c58a32e Compare January 30, 2026 15:24
Copy link
Contributor

@sebsura sebsura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great change!

Personally, I would have liked it more if we used a GENERATE_DEBIAN_INFO option, and simple set it when building on debian-based builders, but this is nice quick fix.

@BareosBot BareosBot merged commit 9be4600 into bareos:master Feb 2, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants