Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It seems to be mostly a cleanup script relating to CVS usage. Also: what the hell is DEADJOE?
Comment | File | Size | Author |
---|---|---|---|
#2 | remove_codeclear-1282524-1.patch | 743 bytes | skottler |
Comments
Comment #1
TR CreditAttribution: TR commentedThis is not mainly CVS usage. clean-code.sh has some utility in removing files that are a normal side-effect of development. *.rej and *.orig are still generated as part of the patch process under Git, for example.
However, I do agree that it is highly questionable whether this should be in core. Perhaps we should have a developer tools module for this stuff. Oh wait, that's drush ... Why not just turn this into a drush script? I'd be willing to do the work if there's agreement that this should be removed from core.
From http://www.juniper.net/security/auto/vulnerabilities/vuln1959.html :
I have no idea why we should be cleaning up after the joe editor in core. Why not also remove .swp files from vi, etc.?
Comment #2
skottler CreditAttribution: skottler commentedI agree that this script is outdated. Here is a patch to remove it completely.
Comment #3
pillarsdotnet CreditAttribution: pillarsdotnet commented+1
Comment #4
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks.
Comment #6
jpklein CreditAttribution: jpklein commentedFYI: "DEADJOE" files are created when a session with the *nix text-editor joe is terminated with unsaved changes. :)