On Tue, Oct 08, 2024 at 12:43:20AM -0700, Karthik Nayak wrote:
> shejialuo <shejialuo@xxxxxxxxx> writes:
>
> [snip]
>
> > + if (strbuf_read_file(&ref_content, iter->path.buf, 0) < 0) {
> > + ret = fsck_report_ref(o, &report,
> > + FSCK_MSG_BAD_REF_CONTENT,
> > + "cannot read ref file");
> > + goto cleanup;
> > + }
> > +
>
> Shouldn't we use `die_errno` here instead? I mean, this is not really a
> bad ref content issue. If we don't want to die here, it would still
> probably be nice to get the actual issue using `strerror` instead and
> use that instead of the generic message we have here.
>
Well, I think I need to dive into the "open" system call here. Actually,
we have two opinions now. Junio thought that we should use
"fsck_report_ref" to report. Karthik, Patrick and I thought that we
should report using "*errno" because this is a general error.
Let me investigate what situations will make "open" system call fail.
Thus, we could fully explain which choice we will choose.
Thanks,
Jialuo
