Util-Linux Archive on lore.kernel.org
 help / color / Atom feed
* Help with reviewing dosfstools patches
@ 2019-06-14 10:25 Pali Rohár
  2019-06-14 14:20 ` Enrico Weigelt, metux IT consult
  0 siblings, 1 reply; 8+ messages in thread
From: Pali Rohár @ 2019-06-14 10:25 UTC (permalink / raw)
  To: util-linux, linux-fsdevel, linux-kernel

Hello!

Can somebody help with reviewing existing patches / pull requests for
dosfstools project? https://github.com/dosfstools/dosfstools/pulls

Dosfstools contains linux userspace utilities for FAT file filesystems,
including mkfs and fsck. They are de-facto standard tools available in
any linux distribution supporting FAT file systems.

There are more patches for these utilities and due to lack of more
active developers, these patches are just waiting for review.

More end users are asking for releasing a new version of dosfstools
including of those waiting patches. So can somebody help with reviewing
them?

-- 
Pali Rohár
pali.rohar@gmail.com

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 10:25 Help with reviewing dosfstools patches Pali Rohár
@ 2019-06-14 14:20 ` Enrico Weigelt, metux IT consult
  2019-06-14 14:25   ` Pali Rohár
  2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
  0 siblings, 2 replies; 8+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-06-14 14:20 UTC (permalink / raw)
  To: Pali Rohár, util-linux, linux-fsdevel, linux-kernel

On 14.06.19 12:25, Pali Rohár wrote:
> Hello!
> 
> Can somebody help with reviewing existing patches / pull requests for
> dosfstools project? https://github.com/dosfstools/dosfstools/pulls

I'll have a look at it. Could you perhaps prepare a (rebased) patch
queue ?

Does the project already have a maillist ?


--mtx

-- 
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 14:20 ` Enrico Weigelt, metux IT consult
@ 2019-06-14 14:25   ` Pali Rohár
  2019-06-14 14:30     ` Christoph Hellwig
  2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
  1 sibling, 1 reply; 8+ messages in thread
From: Pali Rohár @ 2019-06-14 14:25 UTC (permalink / raw)
  To: Enrico Weigelt, metux IT consult; +Cc: util-linux, linux-fsdevel, linux-kernel

On Friday 14 June 2019 16:20:08 Enrico Weigelt, metux IT consult wrote:
> On 14.06.19 12:25, Pali Rohár wrote:
> > Hello!
> > 
> > Can somebody help with reviewing existing patches / pull requests for
> > dosfstools project? https://github.com/dosfstools/dosfstools/pulls
> 
> I'll have a look at it. Could you perhaps prepare a (rebased) patch
> queue ?
> 
> Does the project already have a maillist ?

No, there is no mailing list. Basically whole development is on github
via github pull requests where are also put review comments and where is
also whole discussion, including bug reports.

-- 
Pali Rohár
pali.rohar@gmail.com

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 14:25   ` Pali Rohár
@ 2019-06-14 14:30     ` Christoph Hellwig
  2019-06-14 19:08       ` Pali Rohár
  0 siblings, 1 reply; 8+ messages in thread
From: Christoph Hellwig @ 2019-06-14 14:30 UTC (permalink / raw)
  To: Pali Rohár
  Cc: Enrico Weigelt, metux IT consult, util-linux, linux-fsdevel,
	linux-kernel

On Fri, Jun 14, 2019 at 04:25:34PM +0200, Pali Rohár wrote:
> > Does the project already have a maillist ?
> 
> No, there is no mailing list. Basically whole development is on github
> via github pull requests where are also put review comments and where is
> also whole discussion, including bug reports.

That could explain why it is lacking qualified reviewers..

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 14:20 ` Enrico Weigelt, metux IT consult
  2019-06-14 14:25   ` Pali Rohár
@ 2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
  2019-06-14 18:45     ` Enrico Weigelt, metux IT consult
  2019-06-14 19:10     ` Pali Rohár
  1 sibling, 2 replies; 8+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-06-14 15:45 UTC (permalink / raw)
  To: Pali Rohár, util-linux, linux-fsdevel, linux-kernel

On 14.06.19 16:20, Enrico Weigelt, metux IT consult wrote:

<snip>

Currently working through your branches. Smells like they really deserve
a rebase and signed-off lines.


--mtx

-- 
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
@ 2019-06-14 18:45     ` Enrico Weigelt, metux IT consult
  2019-06-14 19:10     ` Pali Rohár
  1 sibling, 0 replies; 8+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-06-14 18:45 UTC (permalink / raw)
  To: Pali Rohár, util-linux, linux-fsdevel, linux-kernel

On 14.06.19 17:45, Enrico Weigelt, metux IT consult wrote:
> On 14.06.19 16:20, Enrico Weigelt, metux IT consult wrote:
> 
> <snip>
> 
> Currently working through your branches. Smells like they really deserve
> a rebase and signed-off lines.

rebased/applied your patches and got test failures:

XFAIL: check-dot_entries
========================

Test check-dot_entries
First fsck run to check and fix error...
fsck.fat 4.1+git (2017-01-24)
/DIR
  "." is not first entry. Can't fix this yet.
/DIR
  ".." is not second entry. Can't fix this yet.
check-dot_entries.img: 4 files, 3/63931 clusters
*** Error was not detected by fsck.
XFAIL check-dot_entries.fsck (exit status: 100)

XFAIL: check-huge
=================

Test check-huge
First fsck run to check and fix error...
Failed to read sector 167772191.
fsck.fat 4.1+git (2017-01-24)
Second fsck run to check if error was fixed...
Failed to read sector 167772191.
fsck.fat 4.1+git (2017-01-24)
*** Error was not fixed by fsck.
XFAIL check-huge.fsck (exit status: 1)


--mtx

-- 
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 14:30     ` Christoph Hellwig
@ 2019-06-14 19:08       ` Pali Rohár
  0 siblings, 0 replies; 8+ messages in thread
From: Pali Rohár @ 2019-06-14 19:08 UTC (permalink / raw)
  To: Christoph Hellwig
  Cc: Enrico Weigelt, metux IT consult, util-linux, linux-fsdevel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1733 bytes --]

On Friday 14 June 2019 07:30:52 Christoph Hellwig wrote:
> On Fri, Jun 14, 2019 at 04:25:34PM +0200, Pali Rohár wrote:
> > > Does the project already have a maillist ?
> > 
> > No, there is no mailing list. Basically whole development is on github
> > via github pull requests where are also put review comments and where is
> > also whole discussion, including bug reports.
> 
> That could explain why it is lacking qualified reviewers..

That is not my decision. Probably current maintainer of dosfstools like
it. But nowadays do not have much more time and gave me commit access.

Main benefit of github pull requests is that every one pull request
patch is automatically compiled and tested on i686, x86_64, x32, arm
and big endian powerpc systems via Travis CI. So author and also
reviewers/people with commit access immediately know if proposed change
could break something.

I like automated testing on more platforms, because lot of people either
do not know or do not have access to different systems just for testing.
And this can show possible problems...

Setting up a new mailing list, configuring some testing server, copying
existing patches from github to list, etc... is just tons of work for
which I basically do not have a time. Therefore I asked for help to
review existing stuff with minimal technical time setup.

I understand that it is not ideal for everybody, but for current
contributors it is probably better. I have already reviewed and merged
more patches, just those which are mine are left open, so more eyes can
look at them.

If you or somebody else have time and want to improve dosfstools
project, I'm not against it...

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Help with reviewing dosfstools patches
  2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
  2019-06-14 18:45     ` Enrico Weigelt, metux IT consult
@ 2019-06-14 19:10     ` Pali Rohár
  1 sibling, 0 replies; 8+ messages in thread
From: Pali Rohár @ 2019-06-14 19:10 UTC (permalink / raw)
  To: Enrico Weigelt, metux IT consult; +Cc: util-linux, linux-fsdevel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 677 bytes --]

On Friday 14 June 2019 17:45:20 Enrico Weigelt, metux IT consult wrote:
> On 14.06.19 16:20, Enrico Weigelt, metux IT consult wrote:
> 
> <snip>
> 
> Currently working through your branches. Smells like they really deserve
> a rebase and signed-off lines.

Every patch/pull request is mean to be based on current upstream
"master" branch.

As some of pull requests were opened long time ago, they are out-of-sync
from upstream "master" branch.

Currently I rebased and force-pushed those changes which had merge
conflict with upstream master branch.

And now on each pull request passed compilation and 'make check'.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, back to index

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-14 10:25 Help with reviewing dosfstools patches Pali Rohár
2019-06-14 14:20 ` Enrico Weigelt, metux IT consult
2019-06-14 14:25   ` Pali Rohár
2019-06-14 14:30     ` Christoph Hellwig
2019-06-14 19:08       ` Pali Rohár
2019-06-14 15:45   ` Enrico Weigelt, metux IT consult
2019-06-14 18:45     ` Enrico Weigelt, metux IT consult
2019-06-14 19:10     ` Pali Rohár

Util-Linux Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/util-linux/0 util-linux/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 util-linux util-linux/ https://lore.kernel.org/util-linux \
		util-linux@vger.kernel.org
	public-inbox-index util-linux

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.util-linux


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git