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

cmake: mgr: exclude .gitignore #19174

Merged
merged 1 commit into from Nov 29, 2017

Conversation

smithfarm
Copy link
Contributor

Fixes RPMLINT warning:

ceph-mgr.x86_64: W: version-control-internal-file /usr/lib64/ceph/mgr/.gitignore
ceph-mgr.x86_64: W: version-control-internal-file /usr/lib64/ceph/mgr/dashboard/static/AdminLTE-2.3.7/.gitignore
You have included file(s) internally used by a version control system in the
package. Move these files out of the package and rebuild it.

Signed-off-by: Nathan Cutler ncutler@suse.com

@smithfarm
Copy link
Contributor Author

CMake Error at src/pybind/CMakeLists.txt:66 (install):
  Syntax error in cmake code at

    /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/CMakeLists.txt:69

  when parsing string

    .*/\.gitignore

  Invalid escape sequence \.

Fixes RPMLINT warning:

ceph-mgr.x86_64: W: version-control-internal-file /usr/lib64/ceph/mgr/.gitignore
ceph-mgr.x86_64: W: version-control-internal-file /usr/lib64/ceph/mgr/dashboard/static/AdminLTE-2.3.7/.gitignore
You have included file(s) internally used by a version control system in the
package. Move these files out of the package and rebuild it.

Note: the backslash has to be doubled up for the regex to make
it through CMake.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
Copy link
Contributor Author

Confirmed that this makes the RPMLINT warning go away.

@tchaikov tchaikov merged commit 34ffafd into ceph:master Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants