From 5e2be3cb1024baf197366bb4bf2b254f53b87f9a Mon Sep 17 00:00:00 2001
From: matthiasclasen <matthias.clasen@gmail.com>
Date: Thu, 2 Jun 2016 02:50:58 -0400
Subject: [PATCH] builder: Some consistency fixes for output (#64)

* builder: Some consistency fixes for output

Capitalize all error messages.
---
 builder/builder-main.c     | 24 ++++++++++++------------
 builder/builder-manifest.c |  2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/builder/builder-main.c b/builder/builder-main.c
index d9f7c0ad..d1922359 100644
--- a/builder/builder-main.c
+++ b/builder/builder-main.c
@@ -234,7 +234,7 @@ main (int    argc,
 
   if (!g_option_context_parse (context, &argc, &argv, &error))
     {
-      g_printerr ("option parsing failed: %s\n", error->message);
+      g_printerr ("Option parsing failed: %s\n", error->message);
       return 1;
     }
 
@@ -373,20 +373,20 @@ main (int    argc,
                          builder_manifest_get_id (manifest));
       if (!builder_manifest_init_app_dir (manifest, build_context, &error))
         {
-          g_print ("error: %s\n", error->message);
+          g_print ("Error: %s\n", error->message);
           return 1;
         }
 
       if (!builder_cache_commit (cache, body, &error))
         {
-          g_print ("error: %s\n", error->message);
+          g_print ("Error: %s\n", error->message);
           return 1;
         }
     }
 
   if (!builder_manifest_build (manifest, cache, build_context, &error))
     {
-      g_print ("error: %s\n", error->message);
+      g_print ("Error: %s\n", error->message);
       return 1;
     }
 
@@ -394,19 +394,19 @@ main (int    argc,
     {
       if (!builder_manifest_cleanup (manifest, cache, build_context, &error))
         {
-          g_print ("error: %s\n", error->message);
+          g_print ("Error: %s\n", error->message);
           return 1;
         }
 
       if (!builder_manifest_finish (manifest, cache, build_context, &error))
         {
-          g_print ("error: %s\n", error->message);
+          g_print ("Error: %s\n", error->message);
           return 1;
         }
 
       if (!builder_manifest_create_platform (manifest, cache, build_context, &error))
         {
-          g_print ("error: %s\n", error->message);
+          g_print ("Error: %s\n", error->message);
           return 1;
         }
     }
@@ -418,7 +418,7 @@ main (int    argc,
     {
       g_autoptr(GFile) debuginfo_metadata = NULL;
 
-      g_print ("exporting %s to repo\n", builder_manifest_get_id (manifest));
+      g_print ("Exporting %s to repo\n", builder_manifest_get_id (manifest));
 
       if (!do_export (build_context, &error,
                       builder_context_get_build_runtime (build_context),
@@ -444,7 +444,7 @@ main (int    argc,
           g_autofree char *files_arg = NULL;
 
           if (strcmp (name, "metadata.locale") == 0)
-            g_print ("exporting %s.Locale to repo\n", builder_manifest_get_id (manifest));
+            g_print ("Exporting %s.Locale to repo\n", builder_manifest_get_id (manifest));
           else
             continue;
 
@@ -465,7 +465,7 @@ main (int    argc,
       debuginfo_metadata = g_file_get_child (app_dir, "metadata.debuginfo");
       if (g_file_query_exists (debuginfo_metadata, NULL))
         {
-          g_print ("exporting %s.Debug to repo\n", builder_manifest_get_id (manifest));
+          g_print ("Exporting %s.Debug to repo\n", builder_manifest_get_id (manifest));
 
           if (!do_export (build_context, &error, TRUE,
                           "--metadata=metadata.debuginfo",
@@ -482,7 +482,7 @@ main (int    argc,
       if (builder_context_get_build_runtime (build_context) &&
           platform_id != NULL)
         {
-          g_print ("exporting %s to repo\n", platform_id);
+          g_print ("Exporting %s to repo\n", platform_id);
 
           if (!do_export (build_context, &error, TRUE,
                           "--metadata=metadata.platform",
@@ -508,7 +508,7 @@ main (int    argc,
           g_autofree char *files_arg = NULL;
 
           if (strcmp (name, "metadata.platform.locale") == 0)
-            g_print ("exporting %s.Locale to repo\n", platform_id);
+            g_print ("Exporting %s.Locale to repo\n", platform_id);
           else
             continue;
 
diff --git a/builder/builder-manifest.c b/builder/builder-manifest.c
index e65e62db..db552329 100644
--- a/builder/builder-manifest.c
+++ b/builder/builder-manifest.c
@@ -1287,7 +1287,7 @@ rename_icon_cb (BuilderManifest *self,
 
       *found = TRUE;
 
-      g_print ("%s icon %s/%s to %s/%s\n", self->copy_icon ? "copying" : "renaming", rel_dir, source_name, rel_dir, new_name);
+      g_print ("%s icon %s/%s to %s/%s\n", self->copy_icon ? "Copying" : "Renaming", rel_dir, source_name, rel_dir, new_name);
 
       if (self->copy_icon)
         res = linkat (source_parent_fd, source_name, source_parent_fd, new_name, AT_SYMLINK_FOLLOW);