Reviewing merge request #15: Fix config parser regex to match lines with multiple leading spaces

As the commit says, the current option regex only matches zero or one leading space, although the comments state that any number of leading spaces are allowed. I'm going to be patching this in Fedora so that our config files will be parsed correctly. Many of them have leading spaces as opposed to a single leading tab.

Requested by: jkeating
Status: Open
Source repository: jkeatings-bugfixes:master
Target repository: mainline:master
Created at: February 16th, 2011 20:48

Commits that would be merged:

Version 1

Version 1

5eb7fd3
efbf8d1

Match any number of leading spaces in config values

Showing

Showing 5eb7fd3-efbf8d1

Comments
Include comments entered inline

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository

One way to merge in these changes is to pull them into a local branch for review, verifying the changes and then merge to back to master.

# Check out a new branch for integration
git checkout -b merge-requests/15

# Fetch the merge request into this branch
git pull \
  git://gitorious.org/git-python/mainline.git \
  refs/merge-requests/15

# Show the commits, assess they are okay
git log --pretty=oneline --abbrev-commit master..merge-requests/15

# To apply the changes to your branch:
git checkout master
git merge merge-requests/15
git push origin master

Reviewing merge request #15: Fix config parser regex to match lines with multiple leading spaces

Commits that would be merged:

Comments Include comments entered inline

Add a new comment:

Gitorious

Comments
Include comments entered inline