Fails when relative path is not prepended with "./"

[mkell85]

Doesn't play well with ack (or any other tool with similar output) because ack does not prepend a "./" to the file list returned from . E.g.

ack -l 'foo'
junk/foo/bar.txt

ack -l 'foo' | fpp <------- Does not work needs ./junk/foo/bar.txt instead.

[pedropedruzzi]

Describe what do not work.

[gsheld]

+ 1

[brentvatne]

Same problem here, on a React Native project I ran:

ag RCTNavItem node_modules | fpp

I then scrolled down to node_modules/react-native/React/Views/RCTNavItem.h:12:@interface RCTNavItem : UIView and pressed enter

It opened:
~/www/node_modules/react-native/React/Views/RCTNavItem.h

I wanted it to open the relative path:
./node_modules/react-native/React/Views/RCTNavItem.h

[dutzi]

+1 for command line svn

[brentvatne]

To anyone else having the issue I mentioned above with ag, for now it works fine to do ag searchterm path --ackmate | fpp

[pcottle]

Sorry for the ~/www fallback, that is vestigial and only relevant for facebook dev environments.

we only do a hail-mary fallback punt if you are working outside of a git or mercurial repo, since we parse the top-level path here:
https://github.com/facebook/PathPicker/blob/master/src/parse.py#L38-L69

so this shouldn't be superrrrr common, but i guess it's clearly happening.

Anyways this is resolved by e302f16, update your version manually or wait for the next release to fix this! thanks all for the report

[brentvatne]

great, thanks @pcottle! this was actually happening from within a git repo