Jump to content

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


RV_

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}

 

RV/Derek
http://www.rvroadie.com Email on the bottom of my website page.
Retired AF 1971-1998


When you see a worthy man, endeavor to emulate him. When you see an unworthy man, look inside yourself. - Confucius

 

“Those who can make you believe absurdities, can make you commit atrocities.” ... Voltaire

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.

Dutch
2001 GBM Landau 34' Class A
F-53 Chassis, Triton V10, TST TPMS
2011 Toyota RAV4 4WD/Remco pump
ReadyBrute Elite tow bar/brake system

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...
RVers Online University

mywaggle.com

campgroundviews.com

RV Destinations

Find out more or sign up for Escapees RV'ers Bootcamp.

Advertise your product or service here.

The Rvers- Now Streaming

RVTravel.com Logo



×
×
  • Create New...