summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStewart Smith <stewart@linux.vnet.ibm.com>2015-12-08 12:06:17 +1100
committerStephen Cprek <smcprek@us.ibm.com>2016-02-19 15:31:33 -0600
commit9dcb7a898beab758354b431552af1171af70a1e9 (patch)
tree3452e23664177881864169a48a4218a83fd533f9
parentad7a994f5d8b37ffe56e7f296dd0348da2a45686 (diff)
downloadtalos-hostboot-9dcb7a898beab758354b431552af1171af70a1e9.tar.gz
talos-hostboot-9dcb7a898beab758354b431552af1171af70a1e9.zip
linker/gensyms: use stderr for displaying error messages
Since genlist is used in makefiles and stdout redirected, if there is an error, you will not see it in make output at all, instead having things fail in weird ways. Use fprintf for stderr rather than to stdout to avoid Change-Id: I947a72bb1ecc33cbecf51f426a5b76664a6dc03b Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com> Signed-off-by: Patrick Williams <iawillia@us.ibm.com> Forwardport: yes Github: See open-power/hostboot#39 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/23180 Tested-by: Jenkins Server Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/23185
-rw-r--r--src/build/linker/gensyms.C14
1 files changed, 7 insertions, 7 deletions
diff --git a/src/build/linker/gensyms.C b/src/build/linker/gensyms.C
index ccd7225f0..d445deb69 100644
--- a/src/build/linker/gensyms.C
+++ b/src/build/linker/gensyms.C
@@ -136,13 +136,13 @@ int main(int argc, char** argv)
int base_fd = open(g_imageName.c_str(), O_RDONLY);
if (-1 == base_fd)
{
- printf("Failed to open image file: %s.\n", g_imageName.c_str());
+ fprintf(stderr, "Failed to open image file: %s.\n", g_imageName.c_str());
exit(-1);
}
struct stat base_stat;
if (0 != fstat(base_fd, &base_stat))
{
- printf("Failed to stat image file: %s.\n", g_imageName.c_str());
+ fprintf(stderr, "Failed to stat image file: %s.\n", g_imageName.c_str());
exit(-1);
}
g_imageFileSize = base_stat.st_size;
@@ -156,13 +156,13 @@ int main(int argc, char** argv)
int ext_fd = open(g_extImageName.c_str(), O_RDONLY);
if (-1 == ext_fd)
{
- printf("Failed to open image file: %s.\n", g_extImageName.c_str());
+ fprintf(stderr, "Failed to open image file: %s.\n", g_extImageName.c_str());
exit(-1);
}
struct stat ext_stat;
if (0 != fstat(ext_fd, &ext_stat))
{
- printf("Failed to stat image file: %s.\n", g_extImageName.c_str());
+ fprintf(stderr, "Failed to stat image file: %s.\n", g_extImageName.c_str());
exit(-1);
}
g_extImageFileSize = ext_stat.st_size;
@@ -175,7 +175,7 @@ int main(int argc, char** argv)
g_crossPrefix = getenv("CROSS_PREFIX");
if (NULL == g_crossPrefix)
{
- printf("Environment variable CROSS_PREFIX not set.\n");
+ fprintf(stderr, "Environment variable CROSS_PREFIX not set.\n");
exit(-1);
}
g_crossPrefix = strdup(g_crossPrefix);
@@ -220,7 +220,7 @@ int main(int argc, char** argv)
void print_usage()
{
- printf("gensyms <image> [<extimage> <extoffset>]\n");
+ fprintf(stderr, "gensyms <image> [<extimage> <extoffset>]\n");
exit(-1);
}
@@ -241,7 +241,7 @@ void parse_modinfo_file(const string& i_image)
FILE* modinfo_file = fopen(modinfo_name.c_str(), "r");
if (NULL == modinfo_file)
{
- printf("Unable to open modinfo file.\n");
+ fprintf(stderr, "Unable to open modinfo file.\n");
exit(-1);
}
OpenPOWER on IntegriCloud