e2fsprogs: return error when failed to populate fs
cover e2fsprogs: return error when failed to populate fs
Testinfo:
$ bitbake core-image-minimal core-image-sato core-image-sato-sdk
It worked well.
// Robert
The following changes since commit ef196434620522affc11b5b1b867386b5d14a4c3:
alsa-tools: fix build when x11 and gtk+ not available (2014-03-19 14:41:55 +0000)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib rbt/e2fsprogs
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/e2fsprogs
Robert Yang (1):
e2fsprogs: return error when failed to populate fs
...c-return-error-when-failed-to-populate-fs.patch | 37 ++++++++++++++++++++++ .../recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bb | 1 + 2 files changed, 38 insertions(+) create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/misc-mke2fs.c-return-error-when-failed-to-populate-fs.patch
Signature · 1 line
1.8.3.1
PATCH 1/1 e2fsprogs: return error when failed to populate fs
We need return retval when "mke2fs -d" failed, otherwise the "$?" would be 0 which is misleading.
[YOCTO #6011]
Signed-off-by Robert Yang <[email protected]>
--- ...c-return-error-when-failed-to-populate-fs.patch | 37 ++++++++++++++++++++++ .../recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bb | 1 + 2 files changed, 38 insertions(+) create mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/misc-mke2fs.c-return-error-when-failed-to-populate-fs.patch
meta/recipes-devtools/e2fsprogs/e2fsprogs/misc-mke2fs.c-return-error-when-failed-to-populate-fs.patch new file +37 -0
@@ -0,0 +1,37 @@ +From e4ffd6fe975888d306b5f700ba54af18b59f6759 Mon Sep 17 00:00:00 2001 +From: Robert Yang <[email protected]> +Date: Thu, 20 Mar 2014 11:10:21 +0800 +Subject: [PATCH] misc/mke2fs.c: return error when failed to populate filesystem + +We need return retval when "mke2fs -d" failed, otherwise the "$?" would +be 0 which is misleading. + +Upstream-Status Submitted + +Signed-off-by: Robert Yang <[email protected]> +--- + misc/mke2fs.c | 7 ++++--- + 1 file changed, 4 insertions(+), 3 deletions(-) + +diff --git a/misc/mke2fs.c b/misc/mke2fs.c +index a63f0b7..846190e 100644 +--- a/misc/mke2fs.c ++++ b/misc/mke2fs.c +@@ -2745,10 +2745,11 @@ no_journal: + current_fs = fs; + root = EXT2_ROOT_INO; + retval = populate_fs(root, root_dir); +- if (retval) ++ if (retval) { + fprintf(stderr, "%s", +- _("\nError while populating file system")); +- else if (!quiet) ++ _("\nError while populating file system\n")); ++ return retval; ++ } else if (!quiet) + printf("%s", _("done\n")); + } + +-- +1.8.3.1 +
meta/recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bb +1 -0
@@ -17,6 +17,7 @@ SRC_URI += "file://acinclude.m4 \ file://0010-debugfs-use-the-functions-in-misc-create_inode.c.patch \ file://0011-mke2fs.8.in-update-the-manual-for-the-d-option.patch \ file://0001-e2fsprogs-fix-cross-compilation-problem.patch \ + file://misc-mke2fs.c-return-error-when-failed-to-populate-fs.patch \ " SRC_URI[md5sum] = "3f8e41e63b432ba114b33f58674563f7"
Signature · 1 line
1.8.3.1