[Chapel Merge] Fix more fallouts from #23992

Branch: refs/heads/main
Revision: a5cef8ec079e0fb540e18458beb2163ff24a8e22
Author: vasslitvinov
Link: Fix more fallouts from #23992 by vasslitvinov · Pull Request #23999 · chapel-lang/chapel · GitHub
Log Message:
Fix more fallouts from #23992 (#23999)

This reverts the change to Errors.chpl made in #23992's commit
cfebf149d2. That change switched two asserts from displaying
someLocale.id to displaying someLocale, which threw off a lot of
tests due to problems with resolution order. For example, the test
errhandling/parallel/forall-wrap-first-error started reporting "error:
module-scope variable 'count' may be used before it is initialized".
Using someLocale.id in the assertions avoids these problems.
Correspondingly, this reverts the change to
test/modules/bradc/userInsteadOfStandard/ChapelIO.chpl as no longer

Merging without a review for expediency.

Compare: Comparing 410708b50290b807a73c1d142dc083483330be57...0087ac37d2333738422a9b455452435b36908d57 · chapel-lang/chapel · GitHub

M modules/standard/Errors.chpl
M test/errhandling/errorMessages/taskerrors-remote-1.good
M test/errhandling/errorMessages/taskerrors-remote-2.good
M test/modules/bradc/userInsteadOfStandard/ChapelIO.chpl