tech-userlevel archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: FreeBSD/NetBSD makefs sync
On Oct 21, 9:14am, emaste%freebsd.org@localhost (Ed Maste) wrote:
-- Subject: Re: FreeBSD/NetBSD makefs sync
| Ok. We added -D in r247041 to treat duplicates as warnings instead of
| errors for mtree specs. It keeps the first, not last, entry. We use it
| to work around issues in our install infrastructure where the same
| file gets installed more than once, so it doesn't matter for this use
| which entry is kept. I can see how -r is useful, but -D without -r has
| a distinct use case so I imagine we'll keep it even after bringing -r
| over.
Yes, we had a few of those too :-). I am fine bringing -D in...
| The switch to -R and man page clean up is in review now at
| https://reviews.freebsd.org/D3959 if you'd like to take a look and/or
| bring into NetBSD.
|
| >>[1] https://lists.freebsd.org/pipermail/freebsd-arch/2015-October/017376.html
| >>[2] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203707
LGTM.
| It's a bit tricky because at least one other developer is working
| through outstanding PRs and Coverity reports against makefs and fixing
| issues now, and it will be some time still before anyone will have a
| chance to go through our history to find changes that aren't in
| NetBSD. Perhaps we can at least send you the new changes as we work
| through them so there's only a gap in the middle to investigate later.
Sure, just send them over.
| What's the most convenient way to get these changes to you? You could
| subscribe to the PRs or subscribe (in Phabricator) to commits to
| usr.sbin/makefs in our tree. Or we could send a list of the commits
| once this current round of work is done. I think there's somewhere
| around 10 issues reported in PRs from Coverity.
|
| One example committed yesterday:
| > Free buffer before returning from cd9660_write_path_table to avoid
| > leaking it after returning from the function
| https://svnweb.freebsd.org/base?view=revision&revision=289687
| https://svnweb.freebsd.org/base?view=revision&revision=289693
|
| I haven't checked if this one is already fixed in NetBSD, but a number
| of open PRs have confirmation that the issue exists in both places.
It probably is not fixed... I think that the best way is send me patches
via e-mail or send-pr and I will apply them.
christos
Home |
Main Index |
Thread Index |
Old Index