Show the package name for which the error associated

Author: miry

src/cli.cr

@@ -176,12 +176,18 @@ end
 begin
   Shards.run
 rescue ex : OptionParser::InvalidOption
-  Shards::Log.fatal { ex.message }
+  Shards::Log.fatal(exception: ex) { ex.message }
   exit 1
 rescue ex : Shards::ParseError
   ex.to_s(STDERR)
   exit 1
+rescue ex : Shards::PackageError
+  package = ex.package
+  Shards::Log.error(exception: ex) { "Failed to install `#{package.name}`: #{ex.message}" }
+  Shards::Log.debug { ex.inspect_with_backtrace }
+  exit 1
 rescue ex : Shards::Error
-  Shards::Log.error { ex.message }
+  Shards::Log.error(exception: ex) { ex.message }
+  Shards::Log.debug { ex.inspect_with_backtrace }
   exit 1
 end

src/commands/install.cr

@@ -65,16 +65,19 @@ module Shards
         # packages are returned by the solver in reverse topological order,
         # so transitive dependencies are installed first
         packages.each do |package|
-          # first install the dependency:
-          next unless install(package)
-
-          # then execute the postinstall script
-          # (with access to all transitive dependencies):
-          package.postinstall
-
-          # always install executables because the path resolver never actually
-          # installs dependencies:
-          package.install_executables
+          Log.with_context do
+            Log.context.set package: package.name
+            # first install the dependency:
+            next unless install(package)
+
+            # then execute the postinstall script
+            # (with access to all transitive dependencies):
+            package.postinstall
+
+            # always install executables because the path resolver never actually
+            # installs dependencies:
+            package.install_executables
+          end
         end
       end
 

src/errors.cr

@@ -2,6 +2,14 @@ module Shards
   class Error < ::Exception
   end
 
+  class PackageError < Error
+    getter package
+
+    def initialize(message, @package : Package)
+      super message
+    end
+  end
+
   class Conflict < Error
     getter package
 

src/package.cr

@@ -114,7 +114,7 @@ module Shards
       spec.executables.each do |name|
         exe_name = find_executable_file(Path[install_path], name)
         unless exe_name
-          raise Shards::Error.new("Could not find executable #{name.inspect}")
+          raise Shards::PackageError.new("Could not find executable #{name.inspect}", package: self)
         end
         Log.debug { "Install #{exe_name}" }
         source = File.join(install_path, exe_name)