https://www.theregister.com/2021/09/06/github_merges_useless_garbage_says/ [user] [user] Sign in The Register(r) -- Biting the hand that feeds IT [magn] [burg] [burg] Topics Security Off-Prem All Off-PremEdge + IoTChannelPaaS + IaaSSaaS (X) On-Prem All On-PremServersStorageNetworksHPCPersonal Tech (X) Software All SoftwareAI + MLApplicationsDatabasesDevOpsOSesVirtualization (X) Offbeat All OffbeatDebatesColumnistsScienceGeek's GuideBOFHLegalBootnotesSite NewsAbout Us (X) Vendor Voice All Vendor VoiceAdobeAmazon Web Services (AWS)Amazon Web Services (AWS) MigrationGoogle CloudGoogle Cloud's ApigeeGoogle Workspace NutanixRapid7Red hatSophosVeeamVirtru (X) Resources * Whitepapers * Webinars * Newsletters Situation Publishing * The Next Platform * Devclass * Blocks and Files Get our Weekly newsletter [front] Software GitHub merges 'useless garbage' says Linus Torvalds as new NTFS support added to Linux kernel 5.15 Also: Compiler warnings now treated as errors by default in kernel builds Tim Anderson Mon 6 Sep 2021 // 12:31 UTC 27 comment bubble on white --------------------------------------------------------------------- 27 comment bubble on white # reddit Twitter Facebook linkedin WhatsApp email [https://www.theregis] Copy Linus Torvalds will pull Paragon Software's NTFS driver into the 5.15 kernel source - but he complained about the use of a GitHub merge in the submission, saying that GitHub "creates absolutely useless garbage merges." Early last month Torvalds gave Paragon Software a nudge that it really should submit a pull request - an actual submission of code to be merged into the kernel source - in order for its read-write NTFS driver to be included in the forthcoming 5.15 release, for which the merge window is currently open. NTFS is the native Windows file system and Paragon's implementation will improve interoperability, versus the existing driver which has limited write support. On Friday Paragon duly submitted its pull request, saying: "Current version works with normal/compressed/sparse files and supports acl, NTFS journal replaying." [front] However, the company is still getting to grips with the submission process and Torvalds had several comments to make, intended to improve future pull requests. [front] [front] First, he said the pull request should have been signed. "In a perfect world, it would be a PGP signature that I can trace directly to you through the chain of trust, but I've never actually required that," he said. Second, he noted that the code in the pull request included merge commits done with the GitHub web user interface. "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," he said. [front] He added: "[G]ithub is a perfectly fine hosting site, and it does a number of other things well too, but merges is not one of those things." Torvalds has complained about aspects of GitHub before, saying in 2012: "I don't do github pull requests. github throws away all the relevant information, like having even a valid email address for the person asking me to pull. The diffstat is also deficient and useless." Note that the git request-pull command is different from the GitHub pull request feature. The ensuing forthright thread has more information on the subject. [front] Third, Paragon's repository has commit messages which lack information, like "Merge branch 'torvalds:master' into master." Torvalds said that "Linux kernel merges need to be done *properly*." He added: "That means proper commit messages with information about what is being merged and *why* you merge something. But it also means proper authorship and committer information etc. All of which github entirely screws up." Don't do this, said Linus Torvalds: a merge using the GitHub user interface Don't do this: A merge using the GitHub user interface The pull request was nevertheless merged. "The initial pull often has a few oddities and I'll accept them now, but for continued development you need to do things properly," said Torvalds. * Tachyum's Prodigy emulator achieves first boot, runs Linux and says 'hello, world' * When you finish celebrating Linux turning 30, try new Linux 5.14, says Linus Torvalds * 'Apps for GNOME' site aims to improve discovery of the project's best applications * 30 years of Linux: OS was successful because of how it was licensed, says Red Hat An example of a well-commented change came from Torvalds yesterday, when he amended the Makefile used for compiling the kernel to "enable '-Werror' by default for all kernel builds ... but make it a config option so that broken environments can disable it when required." The meaning of -Werror is "Make all warnings into errors," requiring stricter discipline for code that would otherwise compile but with warnings about suboptimal code. Torvalds added that "we... will disable specific over-eager warnings as required, if we can't fix them." "Hopefully this will mean that I get fewer pull requests that have new warnings that were not noticed by various automation we have in place... Knock wood," he said. (r) Get our Tech Resources #Share reddit Twitter Facebook linkedin WhatsApp email [https://www.theregis] Copy 27 Comments Similar topics * MORE * Linux * Linus Torvalds * Github Corrections Send us news --------------------------------------------------------------------- [front] Other stories you might like * Why we abandoned open source: LiveCode CEO on retreat despite successful kickstarter Open source business model problems for modern-day HyperCard Tim Anderson Mon 6 Sep 2021 // 17:31 UTC 7 comment bubble on white Interview An easy to use programming language that was made open source in 2014 after a successful crowdfunding campaign is going back to closed source after too many customers switched to using it for free. LiveCode Community, open source under the GPLv3, was archived on GitHub and future development will be closed source. In the official announcement, LiveCode CEO Kevin Miller says: "99 per cent of the work on LiveCode is done by our in-house team. With a large section of the LiveCode community using the free open source edition, the cost of maintaining this vs the reward for doing so is not a balanced equation." Continue reading * Dobler effect: Spinnaker Support snaps up rival database consultant Sets sights on third party support and managed services for Oracle, SAP and Microsoft Lindsay Clark Mon 6 Sep 2021 // 16:32 UTC 1 comment bubble on white SAP and Oracle services firm Spinnaker Support has launched into database management with the purchase of Dobler Consulting. The buy, which adds 50 full-time staff to the ERP support specialist, will allow Spinnaker to extend its horizon to include support and managed services for Oracle, Microsoft SQL Server, and SAP Sybase database technologies, as well as so-called NoSQL databases. Spinnaker, which also supports JD Edwards (acquired along with PeopleSoft by Oracle in 2003) and Salesforce, has also taken on board around 45 new consulting customers with the deal. Continue reading * After failing to make it to orbit, Firefly Aerospace asserts it has 'arrived' 'Arrived' back on the planet quite a bit sooner than expected, sure Richard Speed Mon 6 Sep 2021 // 15:47 UTC 4 comment bubble on white Firefly Aerospace has confirmed that one of its Reaver engines shut down shortly after its Alpha rocket left the pad last week, resulting in the destruction of the vehicle in spectacular fashion just after reaching supersonic velocity. In a sequence of tweets in which Firefly bravely asserted it had "arrived" as a company "capable of building and launching rockets" if not quite one that can deliver a payload to orbit, the team ticked off its achievements. The rocket left the pad as planned: check. There was no unwanted tipping or rotation as it picked up speed: check. All four engines burned as planned: Well... kind of. For 15 seconds. Continue reading * Report details how Airbus pilots saved the day when all three flight computers failed on landing Software 'enhancement' on the way after triple touchdown TITSUP Richard Speed Mon 6 Sep 2021 // 14:58 UTC 43 comment bubble on white Airbus is to implement a software update for its A330 aircraft following an incident in 2020 where all three primary flight computers failed during landing. The result was a loss of thrust reversers and autobrake systems and the pilots having to use manual braking to bring the aircraft, a China Airlines A330-302, to a halt just 30 feet before the end of the runway. The incident happened at Taipei Songshan Airport on 14 June 2020. The flight, CI202 from Shanghai with 87 passengers and nine cabin crew members, had been uneventful. The landing, however, was anything but. Continue reading * Jenkins struck by 'Confluenza' as US Cyber Command warns Atlassian flaw 'cannot wait' How was your weekend? Got some patching done? Richard Speed Mon 6 Sep 2021 // 13:51 UTC comment bubble on black The Jenkins team issued a reminder over the weekend that one should keep one's systems patched as it found itself with a compromised Confluence service. Although the affected instance of Confluence integrated with the company's identity system (which also handles the likes of Jira and Artifactory), the group said: "At this time we have no reason to believe that any Jenkins releases, plugins, or source code have been affected." It is, however, a bit alarming. Atlassian warned users of the injection vulnerability back in August. While the cloud-hosted version of Confluence was not affected, a server or data centre instance most definitely was. Things have escalated somewhat since then. Continue reading * Italian lunatic flies aeroplane through two motorway tunnels Sponsor Red Bull supposedly gives you wings, but that doesn't mean you have to use them everywhere Matt Dupuy Mon 6 Sep 2021 // 13:03 UTC 39 comment bubble on white While most of Europe was still in bed at the weekend, Italian stunt pilot Dario Costa got up early, climbed into his aeroplane and, apropos of nothing, flew it through two Turkish motorway tunnels, becoming the first person on Earth to do so. The flight, which took place through the Catalca Tunnels on the Northern Marmara Highway east of Istanbul, broke the world record for the longest tunnel ever flown through in an aeroplane, an incongruous title which up until this point has not been hotly contested. The flight - which began in the shorter Catalca-2 tunnel then emerged into the open air before again plunging into the longer 1,638m (5,374ft) Catalca-1 section - covered 2.26km in total (1.4 miles) and lasted for 43.44 seconds, which when you watch the video is a disturbingly long time for an aircraft to be underground [see below]. Continue reading * Facebook apologises after its AI system branded Black people as primates Plus: Google Health app for NHS hospitals discontinued, and Tesla under the spotlight Katyanna Quach Mon 6 Sep 2021 // 12:02 UTC 30 comment bubble on white In Brief Facebook has apologized for an "an unacceptable error" after its AI systems asked folks who watched a British video about a Black man if they wanted to view more content on "primates." A former Facebook employee spotted the prompt and reported it, and the biz said it was "looking into the root cause." "As we have said, while we have made improvements to our AI, we know it's not perfect, and we have more progress to make," a spokesperson told The New York Times. "We apologize to anyone who may have seen these offensive recommendations." Continue reading * UK's NHS hands Accenture another PS5m to run Test and Trace system for another year What happened to transferring PS1,000-per-day consultant work to the civil service? Lindsay Clark Mon 6 Sep 2021 // 11:29 UTC 6 comment bubble on white Flying in the face of a commitment to wean itself off consultants, the NHS Test and Trace programme has awarded Accenture a PS4.8m contract extension to keep its much-criticised COVID-19 Test and Trace system up and running for another year. Awarded without competition, the latest contract will see the outsourcing consultancy and professional services outfit maintain the data platform until 1 August 2022. Accenture is the third-highest ranked consultancy on the Test and Trace spending list, after Deloitte and IBM. Total spending expected to go Accenture's way, according to an NAO report published in June [PDF], would be PS30m, putting it a long way behind Deloitte, which is expected to bag PS298m from the service. Continue reading * Spraying a boot error up the bathroom wall Taking the Pis in Paris Richard Speed Mon 6 Sep 2021 // 10:47 UTC 13 comment bubble on white Bork!Bork!Bork! It is time for le Bork de Paris today as the curse of the cock-up fairy spreads beyond the shores of the UK to blight the Pis of France. The example comes from longtime bork-spotter Alastair Craft and was snapped in what we suspect is the gentleman's private room within a Parisian cafe. Continue reading * When the bits hit the fan: What to do when ransomware strikes Don't trust the insurer's techies, take the blame and other practical tips Dominic Connor Mon 6 Sep 2021 // 10:01 UTC 11 comment bubble on white Feature When I first became a company chief techie, the finance director patronisingly explained the basic asymmetry of prevention vs cure. Spending money on assets to stop an attack come out of capex, but spending after the disaster would be up to the insurer, with premiums deducted out of opex. Also, prevention costs reduced current bonuses. But according to Bill Mew, founder and CEO of Crisis Team, who advises companies on how to escape the hole they are in, if you're expecting cyber insurance to come to your rescue - don't. His experience is that if it is a small claim, you will probably get paid eventually, if only to keep you from complaining too loudly. Continue reading * Operating your network isn't enough. You need to develop (for) it Time to drive NetOps beyond the command line Joseph Martins Mon 6 Sep 2021 // 09:15 UTC Sponsored No-one said enterprise networking is easy, so it's understandable that the traditional approach to managing switches has followed the maxim, "keep it simple, stupid". For years, NetOps teams have relied on vendors' own apps to manage their highly complex, and highly expensive, switches, or go on-device using a command line interface (CLI). Of course, they can turn to approved tools or third-party apps in conjunction with standard protocols, such as SNMP - if the supplier chooses to let them. Even then, each approach delivers, at best, a subtly different selection of data. Oh, and this "simple, stupid" approach means management is never going to be real time. Continue reading ABOUT US* * Who we are * Under the hood * Contact us * Advertise with us * Seeking client-side dev MORE CONTENT* * Latest News * Popular Stories * Forums * Whitepapers * Webinars SITUATION PUBLISHING* * The Next Platform * DevClass * Blocks and Files * Continuous Lifecycle London * M-cubed Situation Publishing The Register - Independent news and views for the tech community. Part of Situation Publishing SIGN UP TO OUR DAILY NEWSLETTER Subscribe Twitter Facebook LinkedIn feeds no-js Biting the hand that feeds IT (c) 1998-2021 Do not sell my personal information Cookies Privacy Ts&Cs