Revert "Ignore error when removing temp directory in coreml_preprocess"

[metascroy]

Reverts #12615

I want to better understand what is going on

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/12652

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 5 Pending, 1 Unrelated Failure

As of commit 042abb4 with merge base 9f2b8cd ():

NEW FAILURE - The following job has failed:

• pull / test-eval_llama-mmlu-linux / linux-job (gh)
  RuntimeError: Command docker exec -t ffe5a318c4865db8f504710c643b38d5742309cca7cb0bd0d0b7f7faf479e500 /exec failed with exit code 1

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

• pull / unittest-editable / linux / linux-job (gh) (trunk failure)
  backends/xnnpack/test/ops/test_to_copy.py::TestChannelsLastTaggedReshapePass::test_quantized_to_copy

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[github-actions]

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

[cccclai]

Yeah make sense to understand better before moving forward