chore: update executor back to default of not bubbling errors to main flow

[spiffcs]

Description

The latest releases of syft saw a change where err was returned by the executor causing the main control flow to exit on task failure. This PR reverts this change and instead writes the errors to a multi err value which sets prog warnings in the TUI

• Fixes: NO BUG FILED

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• I have tested my code in common scenarios and confirmed there are no regressions
• I have added comments to my code, particularly in hard-to-understand sections

Behavior on latest release:

 ✔ Loaded image                                                                                                                                                                                                                        
 ✔ Parsed image                                                                                                                                                                                 
 ⠧ Cataloging contents             ━━━━━━━━━━━━━━━━━━━━                                                                                                                                                
   └── ⠧ Packages                        [302 packages]
failed to run tasks: 1 error occurred:
        * failed to run task: java-binary-openjdk: failed to resolve files by glob (**/java): unable to search by glob="**/java": unable to get ref for path="/etc/alternatives/java": cycle during symlink resolution
java-binary-ibm: failed to resolve files by glob (**/java): unable to search by glob="**/java": unable to get ref for path="/etc/alternatives/java": cycle during symlink resolution
java-binary-oracle: failed to resolve files by glob (**/java): unable to search by glob="**/java": unable to get ref for path="/etc/alternatives/java": cycle during symlink resolution
java-binary-graalvm: failed to resolve files by glob (**/java): unable to search by glob="**/java": unable to get ref for path="/etc/alternatives/java": cycle during symlink resolution

Behavior on this PR showing warnings in cataloged contents, but still produces an SBOM

 ✔ Loaded image                                                                                                                                                                                                                        
 ✔ Parsed image                                                                                                                                                                                 
 ✘ Cataloged contents                                                                                                                                                                                  
   ├── ✔ Packages                        [296 packages]
   ├── ✔ File digests                    [3,829 files]
   ├── ✔ File metadata                   [3,829 locations]
   └── ✔ Executables                     [998 executables]
NAME                                VERSION                            TYPE
adduser                             3.118ubuntu2                       deb
...

[spiffcs]

I think we want to keep the executor behavior and instead change the task behavior that is responsible for surfacing these errors that should not be causing syft to exit. Let me go a bit deeper in the calls so we don't undo all executor err reporting and find the specific change made in #2998

This PR reverts the changes found here under package_task_factory
https://github.com/anchore/syft/pull/2998/files