Skip to content

Conversation

behlendorf
Copy link
Contributor

Motivation and Context

We don't want any extra incorrect noise in the configure output.

Description

This is purely a cosmetic fix which removes a stray "no" from the configure output.

How Has This Been Tested?

Before the change:

checking whether inode_owner_or_capable() exists... yes
checking whether super_block uses const struct xattr_handler... yes
no
checking whether xattr_handler->get() wants dentry and inode and flags... no
checking whether xattr_handler->set() wants dentry, inode, and mnt_idmap... no

after

checking whether inode_owner_or_capable() exists... yes
checking whether super_block uses const struct xattr_handler... yes
checking whether xattr_handler->get() wants dentry and inode and flags... no
checking whether xattr_handler->set() wants dentry, inode, and mnt_idmap... no

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

This is purely a cosmetic fix which removes a stray "no" from
the configure output.

Signed-off-by: Brian Behlendorf <[email protected]>
@behlendorf behlendorf added the Status: Code Review Needed Ready for review and testing label Dec 13, 2024
@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Dec 14, 2024
@behlendorf behlendorf merged commit 22259fb into openzfs:master Dec 14, 2024
23 of 24 checks passed
behlendorf added a commit to behlendorf/zfs that referenced this pull request Dec 16, 2024
This is purely a cosmetic fix which removes a stray "no" from
the configure output.

Reviewed-by: Tino Reichardt <[email protected]>
Reviewed-by:  Alexander Motin <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#16867
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Jan 26, 2025
This is purely a cosmetic fix which removes a stray "no" from
the configure output.

Reviewed-by: Tino Reichardt <[email protected]>
Reviewed-by:  Alexander Motin <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#16867
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Accepted Ready to integrate (reviewed, tested)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants