Hey J. R. Okajima,

Thanks again for the quick follow up!

On Sun, Oct 10, 2021 at 10:07 PM <hooanon...@gmail.com> wrote:
>
> Mauricio Faria de Oliveira:
> > So, could you please run it through your internal test suite?
>
> I've fixed the missing path.mnt problem, slightly tested with RW ntfs-3g
> branch, and seen many issues as much as I have to stop the test.
>
> The fundamental problem around RW fuse branch didn't change.
[...]
> It is a big problem for aufs, and I gave up handling it.  The decision
[...]

There's a misunderstanding here, apparently. Sorry if this wasn't
clear, but the request wasn't for testing w/ fuse; it was for testing
more code paths to check for no regressions. The code path that
depended on fuse is only the path used to reproduce the problem.

So, using your existing/current method (non-fuse based) that
cover most of the code in a stable manner, is what I asked for. :)

The point is to verify whether the patchset fixes that problem/path,
without breaking other things (more probably w/ the changes for
other functions.)  So if your test units show a 'pass/fail' as result,
having no 'pass' turning to 'fail' would be good enough.

> I will fix the original bug (path.mnt) and release it next Monday.  It
> will be slightly different from your patch, but I believe that I should
> keep your name as the original authour.

Please don't feel like keeping me as the patchset author -- it's your
work and approach to the problem. If you find your patches similar
to mine, I'm OK with a simple mention like 'based on patchset [1]
from full-name <email>'.

I just would like to understand, for my own education, if there's any
technical problem or other reason for not taking this v2 patchset,

You didn't object to v1 and just suggested changing something,
and now on v2 (w/ the changes) you say you'll do it another way.
(absolutely no problem w/ that, but I'd like to know, if at all possible.)

And again, if at all possible, how the different patch would help w/
the problem in regression testing w/ fuse (which isn't a requirement,
actually, as mentioned above), in case that was considered as a
problem for this patchset.

Thanks!
Mauricio



>
>
> Thank you
> J. R. Okajima



-- 
Mauricio Faria de Oliveira

Reply via email to