927119 - mozrunner should require mozcrash 0.9

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Ed Morley [:emorley]]

Not sure if this really makes a difference (losing track of where we use mozrunner vs ?) but guess it doesn't hurt to bump this requirement as well.

Comment 1

[Ed Morley [:emorley]]

Attachment: Make mozrunner require mozcrash-0.9 or above

Comment 2

[Jeff Hammel]

fine by me; though, to be fair,

1. not sure what exactly it fixes?
2. i'd argue that normally the time to do this is with a mozrunner version bump (weak opinion)

Feel free to checkin, if desired

Comment 3

[Ed Morley [:emorley]]

(In reply to Jeff Hammel [:jhammel] from comment #2)
> 1. not sure what exactly it fixes?

I don't know what uses mozrunner - I was concerned that something that uses it may end up relying on newly added functionality, so would rather bump the version requirement just in case.

> 2. i'd argue that normally the time to do this is with a mozrunner version
> bump (weak opinion)

I wasn't going to re-mirror mozrunner, just check this in and leave for the next normal version bump.

Comment 4

[Ed Morley [:emorley]]

Thank you :-)

https://github.com/mozilla/mozbase/commit/61861b5c1606458ee5ccf75ee6e7b46f7464488e