Details
- Reviewers
mharbison72 - Group Reviewers
hg-reviewers - Commits
- rHGebee234d952a: errors: set detailed exit code to 100 for some remote errors
Diff Detail
- Repository
- rHG Mercurial
- Branch
- default
- Lint
No Linters Available - Unit
No Unit Test Coverage
Event Timeline
tests/test-clone.t | ||
---|---|---|
621 | I'm surprised this is a considered a remote error. Maybe it's related to the sniffing for HttpError or URLError in the exception handler? (I'm fine with this if you are- maybe there's no way to distinguish from a remote error). | |
tests/test-extdata.t | ||
116 | Probably the same issue as above with thinking OSError is a remote error? |
tests/test-extdata.t | ||
---|---|---|
116 | We still get a URLError here because the path is not found. In that other case in test-clone.t where the user gave us a bad URL, I agree that it would have been better to consider that an InputError (exit code 10). It's less clear-cut where because the user gave us a valid URL but it didn't "respond". Of course, in this case it was a file system path, so it's easier to tell the difference between a URL that doesn't exist and some network error... |
I'm surprised this is a considered a remote error. Maybe it's related to the sniffing for HttpError or URLError in the exception handler? (I'm fine with this if you are- maybe there's no way to distinguish from a remote error).