The documentation does not describe what output a script should give when used with bzr bisect run
Bug #538025 reported by
SirVer
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar Bisect Plugin |
Fix Released
|
Undecided
|
Gustaf Thorslund |
Bug Description
Should this be return values or output on stdio. Should this be "yes", "no", 0, 1 ? It would be nice to add this to the documentation.
Related branches
lp:~gthorslund/bzr-bisect/538025-doc-script-exit-code
- Martin Pool (community): Approve
-
Diff: 14 lines (+4/-0)1 file modified__init__.py (+4/-0)
Changed in bzr-bisect: | |
assignee: | nobody → Gustaf Thorslund (gthorslund) |
Changed in bzr-bisect: | |
status: | Confirmed → In Progress |
Changed in bzr-bisect: | |
status: | In Progress → Fix Released |
To post a comment you must log in.
Confirming, just bumped into that one. I had to read source to find out this:
if retcode == 0:
done = self._set_ state(None, 'yes')
elif retcode == 125:
break
else:
done = self._set_ state(None, 'no')
So it should return 0 for yes, 125 for error (WTF??!), anything else for no.