location of jmeter from OS call contains trailing newline that wrecks jmeter invocation

romanchyla / solrjmeter

Performance measurements of SOLR

BSD 3-Clause "New" or "Revised" License

28 stars 12 forks source link

location of jmeter from OS call contains trailing newline that wrecks jmeter invocation #3

Open vvolkman opened 9 years ago

vvolkman commented 9 years ago

Use of rstrip() will remove the newline which causes commands to be split def check_prerequisities(options): jmeter = None try: jmeter = get_output(['which', 'jmeter']).rstrip() # add rstrip() here

romanchyla commented 9 years ago

Thank you for the report, i've just fixed it and pushed new branch python3 - not having too much time to test it myself, would you mind trying it?