Fix bad include (#10213)

Summary:
include "include/rocksdb/blah.h" is messing up some internal
builds vs. include "rocksdb/blah." This fixes the bad case and adds a
check for future instances.

Pull Request resolved: https://github.com/facebook/rocksdb/pull/10213

Test Plan: back-port to 7.4 release candidate and watch internal build

Reviewed By: hx235

Differential Revision: D37296202

Pulled By: pdillinger

fbshipit-source-id: d7cc6b2c57d858dff0444f19320d83c8b4f9b185
main
Peter Dillinger 3 years ago committed by Facebook GitHub Bot
parent ccb4f047ae
commit 501543573a
  1. 8
      build_tools/check-sources.sh
  2. 2
      db/blob/blob_source.h

@ -17,12 +17,18 @@ if [ "$?" != "1" ]; then
BAD=1 BAD=1
fi fi
git grep -n '<rocksdb/' -- ':!build_tools/check-sources.sh' git grep -n 'include <rocksdb/' -- ':!build_tools/check-sources.sh'
if [ "$?" != "1" ]; then if [ "$?" != "1" ]; then
echo '^^^^^ Use double-quotes as in #include "rocksdb/something.h"' echo '^^^^^ Use double-quotes as in #include "rocksdb/something.h"'
BAD=1 BAD=1
fi fi
git grep -n 'include "include/rocksdb/' -- ':!build_tools/check-sources.sh'
if [ "$?" != "1" ]; then
echo '^^^^^ Use #include "rocksdb/something.h" instead of #include "include/rocksdb/something.h"'
BAD=1
fi
git grep -n 'using namespace' -- ':!build_tools' ':!docs' \ git grep -n 'using namespace' -- ':!build_tools' ':!docs' \
':!third-party/folly/folly/lang/Align.h' \ ':!third-party/folly/folly/lang/Align.h' \
':!third-party/gtest-1.8.1/fused-src/gtest/gtest.h' ':!third-party/gtest-1.8.1/fused-src/gtest/gtest.h'

@ -10,7 +10,7 @@
#include "cache/cache_helpers.h" #include "cache/cache_helpers.h"
#include "cache/cache_key.h" #include "cache/cache_key.h"
#include "db/blob/blob_file_cache.h" #include "db/blob/blob_file_cache.h"
#include "include/rocksdb/cache.h" #include "rocksdb/cache.h"
#include "rocksdb/rocksdb_namespace.h" #include "rocksdb/rocksdb_namespace.h"
#include "table/block_based/cachable_entry.h" #include "table/block_based/cachable_entry.h"

Loading…
Cancel
Save