[Cbe-oss-dev] [patch 9/9] petitboot: Merge shared headers into petitboot.h

Geoff Levand geoffrey.levand at am.sony.com
Fri Nov 7 11:21:34 EST 2008


From: Yuji Mano <yuji.mano at am.sony.com>

Merge the library shared headers into a single header petitboot.h.

Also fixes these warnings:

  warning: redundant redeclaration of 'mountpoint_for_device'
  warning: redundant redeclaration of 'resolve_path'

Signed-off-by: Yuji Mano <yuji.mano at am.sony.com>
Signed-off-by: Geoff Levand <geoffrey.levand at am.sony.com>
---
This change is a step to convert petitboot into a collection of applications
linking to a core petitboot library.

 devices.c               |    3 -
 lib/Makefile.am         |    6 --
 lib/kboot-parser.c      |    2 
 lib/message.h           |   30 ----------
 lib/native-parser.c     |    7 +-
 lib/parser.c            |    3 -
 lib/parser.h            |   46 ---------------
 lib/paths.c             |    2 
 lib/paths.h             |   53 ------------------
 lib/petitboot-paths.h   |   24 --------
 lib/petitboot.h         |  141 ++++++++++++++++++++++++++++++++++++++++++++++++
 lib/yaboot-parser.c     |    9 +--
 petitboot-gui.c         |    2 
 petitboot-udev-helper.c |    4 -
 test/parser-test.c      |    3 -
 15 files changed, 156 insertions(+), 179 deletions(-)

--- a/devices.c
+++ b/devices.c
@@ -9,9 +9,8 @@
 #include <asm/byteorder.h>
 
 #include <libtwin/twin_png.h>
+#include <petitboot.h>
 #include "petitboot-gui.h"
-#include "petitboot-paths.h"
-#include "message.h"
 
 #define PBOOT_DEFAULT_ICON	"tux.png"
 
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -25,11 +25,7 @@ AM_CFLAGS = \
 	-Wredundant-decls \
 	-Winline
 
-pkginclude_HEADERS = \
-	message.h \
-	parser.h \
-	paths.h \
-	petitboot-paths.h
+include_HEADERS = petitboot.h
 
 petitboot_headers = \
 	params.h \
--- a/lib/kboot-parser.c
+++ b/lib/kboot-parser.c
@@ -10,7 +10,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 
-#include "parser.h"
+#include "petitboot.h"
 #include "params.h"
 
 #define buf_size 1024
--- a/lib/message.h
+++ /dev/null
@@ -1,30 +0,0 @@
-
-#ifndef _MESSAGE_H
-#define _MESSAGE_H
-
-enum device_action {
-	DEV_ACTION_ADD_DEVICE = 0,
-	DEV_ACTION_ADD_OPTION = 1,
-	DEV_ACTION_REMOVE_DEVICE = 2,
-	DEV_ACTION_REMOVE_OPTION = 3
-};
-
-struct device {
-	char *id;
-	char *name;
-	char *description;
-	char *icon_file;
-};
-
-struct boot_option {
-	char *id;
-	char *name;
-	char *description;
-	char *icon_file;
-	char *boot_image_file;
-	char *initrd_file;
-	char *boot_args;
-};
-
-
-#endif /* _MESSAGE_H */
--- a/lib/native-parser.c
+++ b/lib/native-parser.c
@@ -1,12 +1,11 @@
 
-#include "parser.h"
-#include "params.h"
-#include "paths.h"
-
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 
+#include "petitboot.h"
+#include "params.h"
+
 const char *conf_filename = "/boot/petitboot.conf";
 
 static struct boot_option *cur_opt;
--- a/lib/parser.c
+++ b/lib/parser.c
@@ -1,9 +1,8 @@
 
-#include <petitboot-paths.h>
 #include <stdlib.h>
 #include <string.h>
 
-#include "parser.h"
+#include "petitboot.h"
 
 extern struct parser native_parser;
 extern struct parser yaboot_parser;
--- a/lib/parser.h
+++ /dev/null
@@ -1,46 +0,0 @@
-
-#ifndef _PARSERS_H
-#define _PARSERS_H
-
-#include <stdarg.h>
-#include "message.h"
-
-struct parser {
-	char *name;
-	int priority;
-	int (*parse)(const char *device);
-	struct parser *next;
-};
-
-enum generic_icon_type {
-	ICON_TYPE_DISK,
-	ICON_TYPE_USB,
-	ICON_TYPE_OPTICAL,
-	ICON_TYPE_NETWORK,
-	ICON_TYPE_UNKNOWN
-};
-
-#define streq(a,b) (!strcasecmp((a),(b)))
-
-/* general functions provided by parsers.c */
-void iterate_parsers(const char *devpath, const char *mountpoint);
-
-void free_device(struct device *dev);
-void free_boot_option(struct boot_option *opt);
-
-const char *generic_icon_file(enum generic_icon_type type);
-
-/* functions provided by udev-helper or the test wrapper */
-void pb_log(const char *fmt, ...);
-
-int mount_device(const char *dev_path);
-
-char *resolve_path(const char *path, const char *current_dev);
-const char *mountpoint_for_device(const char *dev_path);
-
-enum generic_icon_type guess_device_type(void);
-
-int add_device(const struct device *dev);
-int add_boot_option(const struct boot_option *opt);
-
-#endif /* _PARSERS_H */
--- a/lib/paths.c
+++ b/lib/paths.c
@@ -4,7 +4,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 
-#include "paths.h"
+#include "petitboot.h"
 
 static char *mount_base;
 
--- a/lib/paths.h
+++ /dev/null
@@ -1,53 +0,0 @@
-#ifndef PATHS_H
-#define PATHS_H
-
-/**
- * Given a string (eg /dev/sda1, sda1 or UUID=B8E53381CA9EA0E3), parse the
- * device path (eg /dev/sda1). Any device descriptions read from config files
- * should be parsed into the path first.
- *
- * The cur_dev is provided for some remapping situations. If NULL is provided,
- * no remapping will be done.
- *
- * Returns a newly-allocated string.
- */
-char *parse_device_path(const char *dev_str, const char *current_device);
-
-/**
- * Get the mountpoint for a device.
- */
-const char *mountpoint_for_device(const char *dev);
-
-/**
- * Resolve a path given in a config file, to a path in the local filesystem.
- * Paths may be of the form:
- *  device:path (eg /dev/sda:/boot/vmlinux)
- *
- * or just a path:
- *  /boot/vmlinux
- * - in this case, the current mountpoint is used.
- *
- * Returns a newly-allocated string containing a full path to the file in path
- */
-char *resolve_path(const char *path, const char *current_device);
-
-
-/**
- * Set the base directory for newly-created mountpoints
- */
-void set_mount_base(const char *path);
-
-/**
- * Utility function for joining two paths. Adds a / between a and b if
- * required.
- *
- * Returns a newly-allocated string.
- */
-char *join_paths(const char *a, const char *b);
-
-/**
- * encode a disk label (or uuid) for use in a symlink.
- */
-char *encode_label(const char *label);
-
-#endif /* PATHS_H */
--- a/lib/petitboot-paths.h
+++ /dev/null
@@ -1,24 +0,0 @@
-#ifndef _PATHS_H
-#define _PATHS_H
-
-#ifndef PREFIX
-#define PREFIX "/usr"
-#endif
-
-#ifndef PKG_SHARE_DIR
-#define PKG_SHARE_DIR PREFIX "/share/petitboot"
-#endif
-
-#ifndef TMP_DIR
-#define TMP_DIR "/var/tmp/mnt/"
-#endif
-
-#define PBOOT_DEVICE_SOCKET "/var/tmp/petitboot-dev"
-#define MOUNT_BIN "/bin/mount"
-#define UMOUNT_BIN "/bin/umount"
-#define BOOT_GAMEOS_BIN "/usr/bin/ps3-boot-game-os"
-
-/* at present, all default artwork strings are const. */
-#define artwork_pathname(s) (PKG_SHARE_DIR "/artwork/" s)
-
-#endif /* _PATHS_H */
--- /dev/null
+++ b/lib/petitboot.h
@@ -0,0 +1,141 @@
+
+#ifndef _PETITBOOT_LIB_H
+#define _PETITBOOT_LIB_H
+
+/* default system paths */
+
+#ifndef PREFIX
+#define PREFIX "/usr"
+#endif
+
+#ifndef PKG_SHARE_DIR
+#define PKG_SHARE_DIR PREFIX "/share/petitboot"
+#endif
+
+#ifndef TMP_DIR
+#define TMP_DIR "/var/tmp/mnt/"
+#endif
+
+#define PBOOT_DEVICE_SOCKET "/var/tmp/petitboot-dev"
+#define MOUNT_BIN "/bin/mount"
+#define UMOUNT_BIN "/bin/umount"
+#define BOOT_GAMEOS_BIN "/usr/bin/ps3-boot-game-os"
+
+/* at present, all default artwork strings are const. */
+#define artwork_pathname(s) (PKG_SHARE_DIR "/artwork/" s)
+
+/* message interface */
+
+enum device_action {
+	DEV_ACTION_ADD_DEVICE = 0,
+	DEV_ACTION_ADD_OPTION = 1,
+	DEV_ACTION_REMOVE_DEVICE = 2,
+	DEV_ACTION_REMOVE_OPTION = 3
+};
+
+struct device {
+	char *id;
+	char *name;
+	char *description;
+	char *icon_file;
+};
+
+struct boot_option {
+	char *id;
+	char *name;
+	char *description;
+	char *icon_file;
+	char *boot_image_file;
+	char *initrd_file;
+	char *boot_args;
+};
+
+/* parser interface */
+
+struct parser {
+	char *name;
+	int priority;
+	int (*parse)(const char *device);
+	struct parser *next;
+};
+
+enum generic_icon_type {
+	ICON_TYPE_DISK,
+	ICON_TYPE_USB,
+	ICON_TYPE_OPTICAL,
+	ICON_TYPE_NETWORK,
+	ICON_TYPE_UNKNOWN
+};
+
+#define streq(a,b) (!strcasecmp((a),(b)))
+
+/* general functions provided by parsers.c */
+void iterate_parsers(const char *devpath, const char *mountpoint);
+
+void free_device(struct device *dev);
+void free_boot_option(struct boot_option *opt);
+
+const char *generic_icon_file(enum generic_icon_type type);
+
+/* functions provided by udev-helper or the test wrapper */
+void pb_log(const char *fmt, ...);
+
+int mount_device(const char *dev_path);
+
+enum generic_icon_type guess_device_type(void);
+
+int add_device(const struct device *dev);
+int add_boot_option(const struct boot_option *opt);
+
+/* path routines */
+
+/**
+ * Given a string (eg /dev/sda1, sda1 or UUID=B8E53381CA9EA0E3), parse the
+ * device path (eg /dev/sda1). Any device descriptions read from config files
+ * should be parsed into the path first.
+ *
+ * The cur_dev is provided for some remapping situations. If NULL is provided,
+ * no remapping will be done.
+ *
+ * Returns a newly-allocated string.
+ */
+char *parse_device_path(const char *dev_str, const char *current_device);
+
+/**
+ * Get the mountpoint for a device.
+ */
+const char *mountpoint_for_device(const char *dev);
+
+/**
+ * Resolve a path given in a config file, to a path in the local filesystem.
+ * Paths may be of the form:
+ *  device:path (eg /dev/sda:/boot/vmlinux)
+ *
+ * or just a path:
+ *  /boot/vmlinux
+ * - in this case, the current mountpoint is used.
+ *
+ * Returns a newly-allocated string containing a full path to the file in path
+ */
+char *resolve_path(const char *path, const char *current_device);
+
+
+/**
+ * Set the base directory for newly-created mountpoints
+ */
+void set_mount_base(const char *path);
+
+/**
+ * Utility function for joining two paths. Adds a / between a and b if
+ * required.
+ *
+ * Returns a newly-allocated string.
+ */
+char *join_paths(const char *a, const char *b);
+
+/**
+ * encode a disk label (or uuid) for use in a symlink.
+ */
+char *encode_label(const char *label);
+
+#endif /* _PETITBOOT_LIB_H */
--- a/lib/yaboot-parser.c
+++ b/lib/yaboot-parser.c
@@ -1,9 +1,4 @@
 
-#include "parser.h"
-#include "params.h"
-#include "paths.h"
-#include "yaboot-cfg.h"
-
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
@@ -14,6 +9,10 @@
 #include <ctype.h>
 #include <sys/param.h>
 
+#include "petitboot.h"
+#include "params.h"
+#include "yaboot-cfg.h"
+
 static struct device *dev;
 static char *devpath;
 static char *defimage;
--- a/petitboot-gui.c
+++ b/petitboot-gui.c
@@ -21,8 +21,8 @@
 #include <libtwin/twin_png.h>
 #include <libtwin/twin_jpeg.h>
 
+#include <petitboot.h>
 #include "petitboot-gui.h"
-#include "petitboot-paths.h"
 
 #ifdef _USE_X11
 #include <libtwin/twin_x11.h>
--- a/petitboot-udev-helper.c
+++ b/petitboot-udev-helper.c
@@ -19,9 +19,7 @@
 #include <linux/cdrom.h>
 #include <sys/ioctl.h>
 
-#include "parser.h"
-#include "paths.h"
-#include "petitboot-paths.h"
+#include <petitboot.h>
 
 /* Define below to operate without the frontend */
 #undef USE_FAKE_SOCKET
--- a/test/parser-test.c
+++ b/test/parser-test.c
@@ -6,8 +6,7 @@
 #include <unistd.h>
 #include <string.h>
 
-#include "parser.h"
-#include "paths.h"
+#include <petitboot.h>
 
 void pb_log(const char *fmt, ...)
 {

-- 






More information about the cbe-oss-dev mailing list