Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename "include/assert.h" #24098

Merged
merged 1 commit into from Sep 18, 2018

Conversation

badone
Copy link
Contributor

@badone badone commented Sep 14, 2018

This conflicts with the system assert.h so rename and change includes to
reflect the new name.

Fixes: http://tracker.ceph.com/issues/35682

Signed-off-by: Brad Hubbard bhubbard@redhat.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

This conflicts with the system assert.h so rename and change includes to
reflect the new name.

Fixes: http://tracker.ceph.com/issues/35682

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
@batrick
Copy link
Member

batrick commented Sep 17, 2018

See also: #23131

@tchaikov
Copy link
Contributor

it conflicts with #24020 . dropping it temporarily.

@liewegas liewegas merged commit 5009d9e into ceph:master Sep 18, 2018
liewegas added a commit that referenced this pull request Sep 18, 2018
* refs/pull/24098/head:
	Rename "include/assert.h"

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants