From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Wed, 10 Dec 2014 19:05:32 +0100 Subject: [Buildroot] [PATCH v2 3/3] fs/rawimg: new rootfs target In-Reply-To: <1418141008-18739-4-git-send-email-kaszak@gmail.com> References: <1418141008-18739-1-git-send-email-kaszak@gmail.com> <1418141008-18739-4-git-send-email-kaszak@gmail.com> Message-ID: <20141210180532.GC3926@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Karoly, All, On 2014-12-09 17:03 +0100, Karoly Kasza spake thusly: > Rootfs target to call the new mkrawimg package. We (Peter, Thomas P., Samuel and I) discussed your patch during the Patchwork cleanup session yesterday. We came up to the following conclusion. We believe this is trying to cover a very specific use-case, and should better be done in a post-image script. Basically, we believe that Buildroot should not try to be too smart about the filesystem generation. We offer a set of options to build basic filesystem images for basic use-cases, as a mean to get kick-started more easily and for simple systems. However, more complex filesystem layouts ought to be outside the scope of what Buildroot should try to automate, given the unlimited combinations that may exist. We are however willing to add host packages that provide the tools required for generating various filesystems or manipulating various partition types (some of them are already present in the "Host utilities" sub-menu) and delegate to the user to provide one or more post-image scripts that can use those tools and assemble the required filesystem and/or disk images from the rootfs.tar tarball. Another point in the conclusion was that we should provide some sample post-image scripts that users can have a look at to see how to write their own. As an aside, I add a personal note that was not discussed yesterday: we should document in the manual all of the above. As a consequence, we've marked your two patches as Rejected on Patchwork: - [2/3] package/mkrawimg: new package - [3/3] fs/rawimg: new rootfs target Thank you for your numerous contributions so far, they are much appreciated! :-) Regards, Yann E. MORIN. > Signed-off-by: Karoly Kasza > --- > > Changes v1 -> v2: > - Removed script into new mkrawimg package. > > fs/Config.in | 1 + > fs/rawimg/Config.in | 15 +++++++++++++++ > fs/rawimg/rawimg.mk | 13 +++++++++++++ > 3 files changed, 29 insertions(+) > create mode 100644 fs/rawimg/Config.in > create mode 100644 fs/rawimg/rawimg.mk > > diff --git a/fs/Config.in b/fs/Config.in > index 5853113..36961db 100644 > --- a/fs/Config.in > +++ b/fs/Config.in > @@ -8,6 +8,7 @@ source "fs/initramfs/Config.in" > source "fs/iso9660/Config.in" > source "fs/jffs2/Config.in" > source "fs/romfs/Config.in" > +source "fs/rawimg/Config.in" > source "fs/squashfs/Config.in" > source "fs/tar/Config.in" > source "fs/ubifs/Config.in" > diff --git a/fs/rawimg/Config.in b/fs/rawimg/Config.in > new file mode 100644 > index 0000000..68b518e > --- /dev/null > +++ b/fs/rawimg/Config.in > @@ -0,0 +1,15 @@ > +config BR2_TARGET_ROOTFS_RAWIMG > + bool "raw disk image" > + depends on BR2_i386 || BR2_x86_64 > + depends on BR2_LINUX_KERNEL > + select BR2_PACKAGE_HOST_MKRAWIMG > + select BR2_LINUX_KERNEL_INSTALL_TARGET > + select BR2_TARGET_ROOTFS_EXT2 > + help > + Build a bootable raw disk image, that can be booted in a VM (e.g. QEMU), > + or directly written to disk. Creates a valid partition table with MBR, > + and one primary ext* partition. > + > +comment "raw disk image needs a Linux kernel to be built" > + depends on BR2_i386 || BR2_x86_64 > + depends on !BR2_LINUX_KERNEL > diff --git a/fs/rawimg/rawimg.mk b/fs/rawimg/rawimg.mk > new file mode 100644 > index 0000000..c0a720d > --- /dev/null > +++ b/fs/rawimg/rawimg.mk > @@ -0,0 +1,13 @@ > +################################################################################ > +# > +# Build the raw disk image > +# > +################################################################################ > + > +ROOTFS_RAWIMG_DEPENDENCIES = rootfs-ext2 host-mkrawimg > + > +define ROOTFS_RAWIMG_CMD > + PATH=$(BR_PATH) mkrawimg $(BINARIES_DIR) $@ > +endef > + > +$(eval $(call ROOTFS_TARGET,rawimg)) > -- > 1.7.10.4 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'