Jump to content

Linux boosts Microsoft NTFS support as Linus Torvalds complains about GitHub merges


RV_
 Share

Recommended Posts

Excerpt:

"Better support for Microsoft's NTFS file system is coming to the Linux 5.15 kernel.

Linux creator Linus Torvalds has agreed to include Paragon Software's NTFS3 kernel driver, giving the Linux kernel 5.15 release improved support for Microsoft's NTFS file system. But he also had some process and security lessons to offer developers about how to code submissions to the kernel should be made. 

Paragon's NTFS driver will make working with Windows' NTFS drives in Linux an easier task --  ending decades of difficulties with Microsoft's proprietary file system that succeeded FAT. 

Torvalds late last week asked Paragon Software again whether the NTFS3 pull request would be submitted for Linux 5.15. 

Konstantin Komarov of Paragon Software replied upon Friday with the NTFS3 pull request but did so -- to the dismay of Torvalds -- via Github's web interface. 

"This is NTFS read-write driver. The current version works with normal/compressed/sparse files and supports acl, NTFS journal replaying," explained Komarov.

"Most of the code was in the Linux-next branch since Aug 13, but there are some patches that were in the Linux-next branch only for a couple of days. Hopefully, it is ok - no regression was detected in tests."

Via The Register, Torvalds offered Komarov a few pointers on how not to submit pull requests in the future, especially using GitHub's web interface for any code merges into the Linux kernel. 

"I notice that you have a GitHub merge commit in there," wrote Torvalds. 

He continued: "That's another of those things that I *really* don't want to see - GitHub creates absolutely useless garbage merges, and you should never ever use the GitHub interfaces to merge anything...GitHub is a perfectly fine hosting site, and it does a number of other things well too, but merges are not one of those things."

Torvalds' chief problem with it was that merges need "proper commit messages with information about is being merged and *why* you merge something." 

He continued: "But it also means proper authorship and committer information etc. All of which GitHub entirely screws up."

Source:

https://www.zdnet.com/article/linux-boosts-microsoft-ntfs-support-as-linus-torvalds-complains-about-github-merges/?ftag=TRE-03-10aaa6b&bhid={%24external_id}&mid={%24MESSAGE_ID}&cid={%24contact_id}&eh={%24CF_emailHash}

 

Link to comment
Share on other sites

Thanks for the heads up! I'll be glad to see the improved NTFS support in the next kernel release. Solid NTFS compliance has always been difficult in cross platform applications, and hopefully this will help the situation.

Linus has been unhappy with the GitHub merge capabilities for quite awhile with good reason. With any open source product, proper documentation and notation is critical for future improvements.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
 Share

×
×
  • Create New...