795329 - Add 'which' method to mozfile

Status: RESOLVED WONTFIX

(Testing :: Mozbase, defect)

Description

[Andrew Halberstadt [:ahal]]

I've needed this in a few places, and I'd like to use it when finding the proper adb binary in b2g automation (since adb tends to get installed all over the place).

https://hg.mozilla.org/build/mozharness/file/bb5df91e065e/mozharness/base/script.py#l302

Comment 1

[Andrew Halberstadt [:ahal]]

Attachment: Patch 1.0 - Add which method

Comment 2

[Jeff Hammel]

Comment on attachment 665911 [details] [diff] [review]
Patch 1.0 - Add which method

+deps = ['mozinfo >= 0.3.3']

Any particular reason for this version number?  Since we don't have a versioning story yet I'd prefer to leave it unpegged as mozinfo.isWin has been there for time immemorial

Comment 3

[Jeff Hammel]

I'm not opposed to taking this if we really need it, but there is a 3rd party which, http://pypi.python.org/pypi/which , which is also in-tree, https://mxr.mozilla.org/mozilla-central/source/python/which/ . :ahal, what software were you planning on using this for?  Is the 3rd party `which` sufficient?

Comment 4

[Andrew Halberstadt [:ahal]]

I guess that is fine. Is the third party which already available in the m-c virtualenv?

Comment 5

[Andrew Halberstadt [:ahal]]

On the other hand, this might be something that could eventually be removed from the tree and consolidated into mozbase. Not that I'm volunteering or anything :p

Comment 6

[Jeff Hammel]

yep: https://mxr.mozilla.org/mozilla-central/source/build/virtualenv/packages.txt#14

not sure how/if it can be used from packaged tests, but i'm sure that is doable

Comment 7

[Jeff Hammel]

(In reply to Andrew Halberstadt [:ahal] from comment #5)
> On the other hand, this might be something that could eventually be removed
> from the tree and consolidated into mozbase. Not that I'm volunteering or
> anything :p

In general, I would like us to use 3rd party packages if/when possible.  Hopefully this can cut down our maintenance burden

Comment 8

[Jeff Hammel]

Comment on attachment 665911 [details] [diff] [review]
Patch 1.0 - Add which method

I'm going to take this down for review until we figure out if we want this.  If there is a needed use case, I think we should take it

Comment 9

[Jeff Hammel]

Ping....any reason not to close this?