Bug 461357 - Code cleanup - tabs vs. spaces
Code cleanup - tabs vs. spaces
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: anaconda (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Anaconda Maintenance Team
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-09-06 09:34 EDT by Patryk Zawadzki
Modified: 2008-11-21 00:19 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-11-21 00:19:53 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Cosmetics - replace tabs with spaces (375.05 KB, patch)
2008-09-06 09:34 EDT, Patryk Zawadzki
no flags Details | Diff

  None (edit)
Description Patryk Zawadzki 2008-09-06 09:34:15 EDT
Created attachment 315951 [details]
Cosmetics - replace tabs with spaces

I've spent most of the last 3 weeks on and off hacking on a PLD installer based on Anaconda. Some of these changes could be pushed upstream but it's hard for me to produce small feature patches as:

* currently git anaconda uses mixed tabs and spaces (I think we all agree that's bad practice)

* all my editors have tabs are set to be 4 characters wide (as I work on a lot of python apps that use tabs instead of spaces and it's a lot more readable this way)

* the two above result in unreadable code so my git repo uses only spaces

* the consequence is that patches would be rather dirty (ignoring whitespace changes results in some indentation changes not being included) :)

Attached is a cosmetic patch that replaces tabs with eight spaces in all .py files (I did this by hand not to replace tabs in string constants). I know this kind of patches is generally not something maintainers love but it would make it much easier to keep our cloned trees in sync with upstream (and provide clean patches).
Comment 1 David Cantrell 2008-11-21 00:19:53 EST
We've discussed cleaning up the tabs vs. spaces in the code, but we have decided to not do a huge change across the code base, but rather clean up functions as we work on them.  New code will be following a policy of 4-space indentation.

Note You need to log in before you can comment on or make changes to this bug.