PYTHON IndexError: tuple index out of range

Question

Would really appreciate feedback on this issue

import subprocess

def main():
'''
Here's where the whole thing starts.
'''
#Edit this constant to change the file name in the git log command.
FILE_NAME = 'file1.xml'

#Do the git describe command to get the tag names.
gitDescribe = 'git describe --tags `git rev-list --tags --max-count=2`'
print ('Invoking: {0}'.format(gitDescribe))
p1 = subprocess.Popen(gitDescribe, shell=True, stdout=subprocess.PIPE)
output = p1.stdout.read()

#Get the first 2 tags from the output.
parsedOutput = output.split('\n')
tag1 = parsedOutput[0]
tag2 = parsedOutput[1]

print('First revision: {0}'.format(tag1))
print('Second revision: {1}'.format(tag2))
#Do the git log command for the revision comparison.
gitLog = 'git log {0}..{1} --pretty=format:"%an %h %ad %d %s" --date=short --topo-order --no-merges {2}'.format(tag1, tag2, FILE_NAME)
print('Invoking: {0}'.format(gitLog))
p2 = subprocess.Popen(gitLog, shell=True, stdout=subprocess.PIPE)
output = p2.stdout.read()
print(output)

if __name__ == "__main__":
    main()

...

bash-3.2$ python pygit5.py 
Invoking: git describe --tags `git rev-list --tags --max-count=2`

First revision: 14.5.5.1
Traceback (most recent call last):
File "pygit5.py", line 31, in <module>
main()
File "pygit5.py", line 22, in main
print('Second revision: {1}'.format(tag2))
IndexError: tuple index out of range

Daniel Roseman · Accepted Answer · 2014-06-11 15:55:28Z

13

tag2 is only a single value, like tag1, so you can't reference item[1]. No doubt you mean

print('Second revision: {0}'.format(tag2))

answered Jun 11, 2014 at 15:55

Daniel Roseman

602k68 gold badges910 silver badges923 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Sean Perry Over a year ago

Or possibly tag2 is a tuple and then *tag2 which would expand the tuple into arguments for the format call and {1} will use the second item from the tuple.

GleasonK · Accepted Answer · 2014-06-11 16:01:14Z

8

When using formatting as you have, remember that in most programming languages, counting starts from zero. So since tag2 only carries one value, the following line:

print('Second revision: {1}'.format(tag2))

Should really be:

print('Second revision: {0}'.format(tag2))

You can also leave it empty for simple scripts if using python 2.7+:

print('Second revision: {}'.format(tag2))

Or provide them in any order with named variables:

print('Second revision: {revisiontag}'.format(revisiontag=tag2))

edited Jun 11, 2014 at 16:01

GleasonK

1,0228 silver badges17 bronze badges

answered Jun 11, 2014 at 15:56

Simon Fraser

2,81821 silver badges26 bronze badges

1 Comment

Sean Perry Over a year ago

In Python 2.6 you have to put an index of some sort in the braces. The empty braces work in 2.7+ and 3.x.

Collectives™ on Stack Overflow

PYTHON IndexError: tuple index out of range

2 Answers 2

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related