sbin/hammer: Don't include libhammer.h unconditionally
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Fri, 12 Aug 2016 13:29:10 +0000 (22:29 +0900)
committerTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Fri, 12 Aug 2016 15:14:01 +0000 (00:14 +0900)
Though /sbin/hammer links to libhammer, only two commands use it.

Move SNAPSHOTS_BASE from cmd_cleanup.c to hammer.h.
This macro is defined in both sbin/hammer and libhammer.
It apparently doesn't need to ask an external library for snapshot location.

 # grep libhammer sbin/hammer sbin/newfs_hammer -rIl
 sbin/hammer/cmd_info.c
 sbin/hammer/cmd_snapshot.c
 sbin/hammer/hammer.h

sbin/hammer/cmd_cleanup.c
sbin/hammer/cmd_info.c
sbin/hammer/cmd_snapshot.c
sbin/hammer/hammer.h

index c404ce2..0b420d9 100644 (file)
@@ -99,12 +99,6 @@ static int cleanup_dedup(const char *path, const char *snapshots_path,
 
 static void runcmd(int *resp, const char *ctl, ...) __printflike(2, 3);
 
-/*
- * WARNING: Do not make the SNAPSHOTS_BASE "/var/snapshots" because
- * it will interfere with the older HAMMER VERS < 3 snapshots directory
- * for the /var PFS.
- */
-#define SNAPSHOTS_BASE "/var/hammer"   /* HAMMER VERS >= 3 */
 #define WS     " \t\r\n"
 
 struct didpfs *FirstPFS;
index c17aab0..f703b7b 100644 (file)
@@ -34,6 +34,7 @@
  */
 
 #include <libutil.h>
+#include <libhammer.h>
 
 #include "hammer.h"
 
index 050bd4f..3674adb 100644 (file)
@@ -34,6 +34,8 @@
  * $DragonFly: src/sbin/hammer/cmd_snapshot.c,v 1.7 2008/07/10 18:47:22 mneumann Exp $
  */
 
+#include <libhammer.h>
+
 #include "hammer.h"
 
 #define DEFAULT_SNAPSHOT_NAME "snap-%Y%m%d-%H%M"
index 727364a..11a9599 100644 (file)
 #include <dirent.h>
 #include <uuid.h>
 
-#include <libhammer.h>
-
 #include "hammer_util.h"
 
+/*
+ * WARNING: Do not make the SNAPSHOTS_BASE "/var/snapshots" because
+ * it will interfere with the older HAMMER VERS < 3 snapshots directory
+ * for the /var PFS.
+ */
+#define SNAPSHOTS_BASE "/var/hammer"   /* HAMMER VERS >= 3 */
+
 extern int RecurseOpt;
 extern int VerboseOpt;
 extern int QuietOpt;