path: root/boot/grub
diff options
authorGravatar Thomas Petazzoni <thomas.petazzoni@free-electrons.com>2013-12-28 15:56:34 +0100
committerGravatar Thomas Petazzoni <thomas.petazzoni@free-electrons.com>2013-12-29 12:17:46 +0100
commit4e0257bb90b399228f9e0008c35fa9d26ae09e70 (patch)
tree75853d8a46524f26ec6cefd584abfdd53c091eb4 /boot/grub
parentcbbbe16714562a4ec4d90b85cd913cb66120b158 (diff)
grub: replace string option for filesystem selection by booleans
In 1cece2813bf635ae29e9a8287322439b9c5992f7 (grub: add option to configure the list of supported filesystems), we introduced the BR2_TARGET_GRUB_FS_SUPPORT option which allows to provide a space-separated list of filesystems that Grub should support. However, it turns out that this not very practical, because the iso9660 filesystem logic in Buildroot should force the ISO9660 support to be enabled in Grub, which is not easy to do with a string option. Therefore, this patch changes this option from a string option to a list of boolean option, one per filesystem supported. A few useful details: - Since Grub legacy is dead, the list of filesystem, and therefore the number of options, will not grow. - We have only added options for filesystems that are likely to be used in an embedded Linux context. Filesystems such as VSTAfs, Minix, UFS2 or FFS2 are not supported. - There is no need to add some Config.in.legacy support for the previous option, since it was added after Buildroot 2013.11, and was therefore never part of an official Buildroot release. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Diffstat (limited to 'boot/grub')
2 files changed, 49 insertions, 19 deletions
diff --git a/boot/grub/Config.in b/boot/grub/Config.in
index c3a729f7df..3ae8448161 100644
--- a/boot/grub/Config.in
+++ b/boot/grub/Config.in
@@ -45,19 +45,47 @@ config BR2_TARGET_GRUB_SPLASH
A splashimage is a 14-color indexed .xpm picture which
is displayed as background for the grub menu.
- string "Filesystem to support"
- default "ext2fs fat"
- help
- Space separated list of filesystems to support. Possible
- values are ext2fs, fat, ffs, ufs2, minix, reiserfs, vstafs,
- jfs, xfs and iso9660.
bool "diskless support"
enable diskless support
+menu "filesystem drivers"
+ bool "ext2"
+ default y
+ help
+ Enable support for the ext2 filesystem in Grub
+ bool "FAT"
+ default y
+ help
+ Enable support for the FAT filesystem in Grub.
+config BR2_TARGET_GRUB_FS_ISO9660
+ bool "ISO9660"
+ help
+ Enable support for the ISO9660 filesystem in Grub.
+ bool "IBM JFS"
+ help
+ Enable support for the JFS filesystem in Grub.
+ bool "ReiserFS"
+ help
+ Enable support for the ReiserFS filesystem in Grub.
+ bool "SGI XFS"
+ help
+ Enable support for the XFS filesystem in Grub.
menu "network drivers"
config BR2_TARGET_GRUB_3c595
diff --git a/boot/grub/grub.mk b/boot/grub/grub.mk
index eb066af76c..43121ff7a4 100644
--- a/boot/grub/grub.mk
+++ b/boot/grub/grub.mk
@@ -46,18 +46,20 @@ GRUB_CONFIG-$(BR2_TARGET_GRUB_undi) += --enable-undi
GRUB_CONFIG-$(BR2_TARGET_GRUB_via_rhine) += --enable-via-rhine
GRUB_CONFIG-$(BR2_TARGET_GRUB_w89c840) += --enable-w89c840
-GRUB_POSSIBLE_FILESYSTEMS = ext2fs fat ffs ufs2 minix \
- reiserfs vstafs jfs xfs iso9660
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_EXT2),--enable-ext2fs,--disable-ext2fs)
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_FAT),--enable-fat,--disable-fat)
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_ISO9660),--enable-iso9660,--disable-iso9660)
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_JFS),--enable-jfs,--disable-jfs)
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_REISERFS),--enable-reiserfs,--disable-reiserfs)
+GRUB_CONFIG-y += $(if $(BR2_TARGET_GRUB_FS_XFS),--enable-xfs,--disable-xfs)
+GRUB_CONFIG-y += --disable-ffs --disable-ufs2 --disable-minix --disable-vstafs
-# Calculate the list of stage 1.5 files to install. They are prefixed
-# by the filesystem name, except for ext2fs, where the stage 1.5 is
-# prefixed by e2fs.
- $(if $(filter $(fs),$(GRUB_SELECTED_FILESYSTEMS)),--enable-$(fs),--disable-$(fs)))
+GRUB_STAGE_1_5_TO_INSTALL += $(if $(BR2_TARGET_GRUB_FS_ISO9660),iso9660)
# Apply the patches from the Debian patch