Skip to content

Stop installing diskquota build-info file#1623

Open
tuhaihe wants to merge 1 commit intoapache:mainfrom
tuhaihe:remove-diskquota-info-fix
Open

Stop installing diskquota build-info file#1623
tuhaihe wants to merge 1 commit intoapache:mainfrom
tuhaihe:remove-diskquota-info-fix

Conversation

@tuhaihe
Copy link
Member

@tuhaihe tuhaihe commented Mar 16, 2026

Remove generation and installation of diskquota-build-info from diskquota to avoid writing extra files into the install prefix root (e.g. $GPHOME or /usr/local/cloudberry-db).

Drop the unused CMake helper cmake/BuildInfo.cmake and remove cmake/Git.cmake and its invocation now that no build-info is produced.

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


Remove generation and installation of diskquota-build-info from
diskquota to avoid writing extra files into the install prefix root
(e.g. $GPHOME or /usr/local/cloudberry-db).

Drop the unused CMake helper cmake/BuildInfo.cmake and remove
cmake/Git.cmake and its invocation now that no build-info is
produced.
@tuhaihe tuhaihe force-pushed the remove-diskquota-info-fix branch from a909eb6 to 4b8f56c Compare March 17, 2026 03:06
Copy link
Contributor

@leborchuk leborchuk left a comment

Choose a reason for hiding this comment

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

Yes, additional buildinfo does not needed. We create debian/rpm installation binary files. They contain all information about who and where builded binaries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants